Closed Bug 152973 Opened 23 years ago Closed 23 years ago

Composer - Editing in <html source> tab, then switching tabs (Normal, Preview) crashes app

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
VERIFIED DUPLICATE of bug 141054
Milestone:
mozilla1.0.1

People

(Reporter: greggl, Assigned: kinmoz)

References

Details

(Keywords: crash, regression, Whiteboard: editorbase+)

Attachments

(2 files)

Include file for Minimal Test Case
25 bytes, text/html
Details
Minimal Test Case
166 bytes, text/html
Details
Composer - Editing in <html source> tab, then switching tabs (Normal, Preview) crashes application. Reproducible every time... 1. Open a web page in composer. 2. Edit the title of the page for example. 3. Try to save or change tabs to "Normal" or "Preview" 4. Application freezes or crashes Same as http://bugscape.netscape.com/show_bug.cgi?id=16902
nominating nsbeta1. Running branch build 20020618. Reproduced problem on both Win98 and Win2000. It happens on this page every time for me: http://softserv.mcom.com/plugins/productiondocs/prds/adpmigration.html 1. Open the page above in a branch build (20020618) 2. Select File|Edit Page 3. Change the title text in the <html> tab 4. Attempt to change tabs or save and I crash Here is the stack trace: FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1730] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ReResolveStyleContext [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1901] FrameManager::ComputeStyleChangeFor [d:\builds\seamonkey\mozilla\layout\html\base\src\nsFrameManager.cpp, line 1945] PresShell::ReconstructStyleData [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 5524] PresShell::StyleSheetAdded [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 5548] nsDocument::InsertStyleSheetAt [d:\builds\seamonkey\mozilla\content\base\src\nsDocument.cpp, line 1501] CSSLoaderImpl::InsertSheetInDoc [d:\builds\seamonkey\mozilla\content\html\style\src\nsCSSLoader.cpp, line 1184] CSSLoaderImpl::LoadStyleLink [d:\builds\seamonkey\mozilla\content\html\style\src\nsCSSLoader.cpp, line 1465] nsStyleLinkElement::UpdateStyleSheet [d:\builds\seamonkey\mozilla\content\base\src\nsStyleLinkElement.cpp, line 387] nsHTMLLinkElement::SetDocument [d:\builds\seamonkey\mozilla\content\html\content\src\nsHTMLLinkElement.cpp, line 111] nsGenericContainerElement::InsertChildAt [d:\builds\seamonkey\mozilla\content\base\src\nsGenericElement.cpp, line 3474] nsGenericElement::doInsertBefore [d:\builds\seamonkey\mozilla\content\base\src\nsGenericElement.cpp, line 2312] nsHTMLDelElement::InsertBefore [d:\builds\seamonkey\mozilla\content\html\content\src\nsHTMLDelElement.cpp, line 63] InsertElementTxn::DoTransaction [d:\builds\seamonkey\mozilla\editor\libeditor\base\InsertElementTxn.cpp, line 115] nsTransactionItem::DoTransaction [d:\builds\seamonkey\mozilla\editor\txmgr\src\nsTransactionItem.cpp, line 183] nsTransactionManager::BeginTransaction [d:\builds\seamonkey\mozilla\editor\txmgr\src\nsTransactionManager.cpp, line 1078] nsTransactionManager::DoTransaction [d:\builds\seamonkey\mozilla\editor\txmgr\src\nsTransactionManager.cpp, line 140] nsEditor::Do [d:\builds\seamonkey\mozilla\editor\libeditor\base\nsEditor.cpp, line 494] nsEditor::InsertNode [d:\builds\seamonkey\mozilla\editor\libeditor\base\nsEditor.cpp, line 1152] nsHTMLEditor::ReplaceHeadContentsWithHTML [d:\builds\seamonkey\mozilla\editor\libeditor\html\nsHTMLEditor.cpp, line 1645] nsHTMLEditor::RebuildDocumentFromSource [d:\builds\seamonkey\mozilla\editor\libeditor\html\nsHTMLEditor.cpp, line 1707] nsEditorShell::RebuildDocumentFromSource [d:\builds\seamonkey\mozilla\editor\composer\src\nsEditorShell.cpp, line 2226] XPTC_InvokeByIndex [d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcinvoke.cpp, line 106] XPCWrappedNative::CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednative.cpp, line 2028] XPC_WN_CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednativejsops.cpp, line 1267] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 790] js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2744] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] nsXPCWrappedJSClass::CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappedjsclass.cpp, line 1195] nsXPCWrappedJS::CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappedjs.cpp, line 430] PrepareAndDispatch [d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcstubs.cpp, line 117] SharedStub [d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcstubs.cpp, line 139] nsControllerCommandManager::DoCommand [d:\builds\seamonkey\mozilla\embedding\components\commandhandler\src\nsControllerCommandManager.cpp, line 190] nsComposerController::DoCommand [d:\builds\seamonkey\mozilla\editor\composer\src\nsComposerController.cpp, line 240] XPTC_InvokeByIndex [d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcinvoke.cpp, line 106] XPCWrappedNative::CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednative.cpp, line 2028] XPC_WN_CallMethod [d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednativejsops.cpp, line 1267] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 790] js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2744] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 881] JS_CallFunctionValue [d:\builds\seamonkey\mozilla\js\src\jsapi.c, line 3426] nsJSContext::CallEventHandler [d:\builds\seamonkey\mozilla\dom\src\base\nsJSEnvironment.cpp, line 1019] nsJSEventListener::HandleEvent [d:\builds\seamonkey\mozilla\dom\src\events\nsJSEventListener.cpp, line 182] nsEventListenerManager::HandleEventSubType [d:\builds\seamonkey\mozilla\content\events\src\nsEventListenerManager.cpp, line 1220] nsEventListenerManager::HandleEvent [d:\builds\seamonkey\mozilla\content\events\src\nsEventListenerManager.cpp, line 2219] nsXULElement::HandleDOMEvent [d:\builds\seamonkey\mozilla\content\xul\content\src\nsXULElement.cpp, line 3461] PresShell::HandleDOMEventWithTarget [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 6314] nsButtonBoxFrame::MouseClicked [d:\builds\seamonkey\mozilla\layout\xul\base\src\nsButtonBoxFrame.cpp, line 195] nsButtonBoxFrame::HandleEvent [d:\builds\seamonkey\mozilla\layout\xul\base\src\nsButtonBoxFrame.cpp, line 142] PresShell::HandleEventInternal [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 6283] PresShell::HandleEventWithTarget [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 6234] nsEventStateManager::CheckForAndDispatchClick [d:\builds\seamonkey\mozilla\content\events\src\nsEventStateManager.cpp, line 2640] nsEventStateManager::PostHandleEvent [d:\builds\seamonkey\mozilla\content\events\src\nsEventStateManager.cpp, line 1721] PresShell::HandleEventInternal [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 6287] PresShell::HandleEvent [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 6189] nsViewManager::HandleEvent [d:\builds\seamonkey\mozilla\view\src\nsViewManager.cpp, line 2086] nsView::HandleEvent [d:\builds\seamonkey\mozilla\view\src\nsView.cpp, line 306] nsViewManager::DispatchEvent [d:\builds\seamonkey\mozilla\view\src\nsViewManager.cpp, line 1897]
Keywords: nsbeta1
Keywords: regression
Whiteboard: editorbase
Daniel: Any idea what might be causing this style crash?
I don't get any crash/freeze with PR1 build 20020613.
I am able to reproduce this problem on the 06-20 1.0 branch build, but only on the URL specified in comment #5. My talkback ID: TB7543018G
I mean comment #1, it was comment 5 in the bugscape bug.
Assigning to Kin since I think Glazman is on vacation.
Assignee: syd → kin
Severity: normal → major
Keywords: crash
Whiteboard: editorbase → editorbase+
Target Milestone: --- → mozilla1.0.1
Status: NEW → ASSIGNED
Attached file Minimal Test Case —
I can't reproduce this crash on the TRUNK. I think this was fixed by dbaron's checkin for bug 141054. I'll have to verify that this stack matches one of the stacks in the various dups of that bug. To see the crash with this test case, load it in a Composer MOZILLA_1_0_BRANCH build, switch to HTML Source mode, add a space anywhere in the source, and switch back to Normal mode. You should then see the crash.
Yup this was due to the same thing that caused bug 141054. I just picked up dbaron's changes for bug 141054, which he checked into the MOZILLA_1_0_BRANCH last night, and now I can't reproduce this in my branch build anymore. *** This bug has been marked as a duplicate of 141054 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
bulk verification.
Status: RESOLVED → VERIFIED
*** Bug 142888 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: