Closed Bug 7211 Opened 25 years ago Closed 25 years ago

MLK: nsTransactionStack's nsDeque

Categories

(Core :: DOM: Editor, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: bruce, Assigned: kinmoz)

Details

Build from May 26, 1999. Solaris 2.6. Start and immediately exit apprunner -editor and this leaks. MLK: 256 bytes leaked at 0x96ec38 * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] __bUiLtIn_vEc_nEw [libgcc.a] __builtin_vec_new [rtlib.o] nsDeque::nsDeque(nsDequeFunctor*) [nsDeque.cpp:35] nsTransactionStack::nsTransactionStack() [nsTransactionStack.cpp:25] nsTransactionRedoStack::nsTransactionRedoStack() [nsTransactionStack.h:59] nsTransactionManager::nsTransactionManager(int) [nsTransactionManager.cpp:37] nsTransactionManagerFactory::CreateInstance(nsISupports*,const nsID&,void**) [nsTransactionManagerFactory.cpp:101] nsComponentManagerImpl::CreateInstance(const nsID&,nsISupports*,const nsID&,void**) [nsComponentManager.cpp:1118] nsEditor::EnableUndo(int) [nsEditor.cpp:490] nsTextEditor::EnableUndo(int) [nsTextEditor.cpp:855] nsHTMLEditor::EnableUndo(int) [nsHTMLEditor.cpp:316] nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:330] nsHTMLEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsHTMLEditor.cpp:118] nsEditorAppCore::InstantiateEditor(nsIDOMDocument*,nsIPresShell*) [nsEditorAppCore.cpp:335] nsEditorAppCore::DoEditorMode(nsIWebShell*) [nsEditorAppCore.cpp:387] nsEditorAppCore::PrepareDocumentForEditing() [nsEditorAppCore.cpp:681] nsEditorAppCore::OnEndDocumentLoad(nsIDocumentLoader*,nsIURL*,int) [nsEditorAppCore.cpp:1987] nsWebShell::OnEndDocumentLoad(nsIDocumentLoader*,nsIURL*,int) [nsWebShell.cpp:2486] nsDocLoaderImpl::FireOnEndDocumentLoad(nsIDocumentLoader*,int) [nsDocLoader.cpp:840] nsDocLoaderImpl::LoadURLComplete(nsIURL*,nsISupports*,int) [nsDocLoader.cpp:1018] nsDocumentBindInfo::OnStopBinding(nsIURL*,unsigned int,const unsigned short*) [nsDocLoader.cpp:1539] stub_complete(_NET_StreamClass*) [nsStubContext.cpp:772] net_ProcessFile [mkfile.c:1360] NET_ProcessNet [mkgeturl.c:3355] NET_PollSockets [mkselect.c:298] nsNetlibService::NetPollSocketsCallback(nsITimer*,void*) [nsNetService.cpp:1276] TimerImpl::FireTimeout() [nsTimer.cpp:73]
Summary: MLK: nsTransactionManagerFactory → MLK: nsTransactionStack's nsDeque
Assignee: kostello → kin
Target Milestone: M7
Reassigning to myself.
Status: NEW → ASSIGNED
accepting bug
Kin, I'm not sure if my fixes fixed this. You want to check?
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
There were several problems here. Simon fixed the problem where the appcore was not releasing the editor, and I fixed a problem where the editor was double refcounting the transaction manager in nsEditor::EnableUndo().
Hi Kin, can you verify this one and mark VERIFIED_FIXED ? thanks buddy!
I'll try and verify under Purify tonight.
Bruce/Kin, please verify when you get chance and mark verified-fixed...thanks!
someone please mark this on VERIFIED-FIXED...thanks!
I'm swamped with work at my real job. I'll get to this when I can. Patience! If patience isn't working, reassign the QA contact to me if you like.
no problem....thanks!
Kin, how easy would this be for you to VERIFY. I gotta get this bug of our VERIFICATION radar...Bruce will get to it eventually, but I don't wanna wait around too long for this one to get VERIFIED.. try the best you can...
Status: RESOLVED → VERIFIED
I'm gonna have to mark this VERIFIED_FIXED. I've waited a reasonable amount of time. I have to get this off the verification radar.
You need to log in before you can comment on or make changes to this bug.