Closed Bug 1195474 Opened 9 years ago Closed 7 years ago

ASSERTION: Parser and editor disagree on blockness: colgroup: 'aIsBlock', file nsHTMLEditor.cpp, line 730 in layout/tables/crashtests/759249-1.html and layout/xul/crashtests/415394-1.xhtml when run in e10s mode

Categories

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

Product:
Core
Component:
DOM: Editor
Platform:
Unspecified
Linux
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
e10s + ---
firefox43 --- affected

People

(Reporter: RyanVM, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, Whiteboard: [leave open]) 

11:51:15 INFO - [Child 1933] ###!!! ASSERTION: Parser and editor disagree on blockness: col: 'aIsBlock', file /builds/slave/m-in-l64-d-0000000000000000000/build/src/editor/libeditor/nsHTMLEditor.cpp, line 730
11:51:15 INFO - #01: nsHTMLEditor::NodeIsBlockStatic(nsINode const*) [editor/libeditor/nsHTMLEditor.cpp:776]
11:51:15 INFO - #02: nsEditor::GetLeftmostChild(nsINode*, bool) [editor/libeditor/nsEditor.cpp:3278]
11:51:15 INFO - #03: nsEditor::GetNextNode(nsINode*, int, bool, bool) [editor/libeditor/nsEditor.cpp:3110]
11:51:15 INFO - #04: nsHTMLEditor::GetNextHTMLNode(nsINode*, int, bool) [editor/libeditor/nsHTMLEditor.cpp:4085]
11:51:15 INFO - #05: nsHTMLEditRules::GetPromotedPoint(nsHTMLEditRules::RulesEndpoint, nsIDOMNode*, int, EditAction, nsCOMPtr<nsIDOMNode>*, int*) [editor/libeditor/nsHTMLEditRules.cpp:5618]
11:51:15 INFO - #06: nsHTMLEditRules::PromoteRange(nsRange&, EditAction) [editor/libeditor/nsHTMLEditRules.cpp:5740]
11:51:15 INFO - #07: nsHTMLEditRules::AfterEditInner(EditAction, short) [editor/libeditor/nsHTMLEditRules.cpp:459]
11:51:15 INFO - #08: nsHTMLEditRules::AfterEdit(EditAction, short) [editor/libeditor/nsHTMLEditRules.cpp:397]
11:51:15 INFO - #09: nsHTMLEditor::EndOperation() [editor/libeditor/nsHTMLEditor.cpp:3430]
11:51:15 INFO - #10: nsEditor::DeleteNode(nsINode*) [editor/libeditor/nsEditor.cpp:1574]
11:51:15 INFO - #11: nsEditor::DeleteNode(nsIDOMNode*) [editor/libeditor/nsEditor.cpp:1549]
11:51:15 INFO - #12: nsHTMLEditor::DeleteNode(nsIDOMNode*) [editor/libeditor/nsHTMLEditor.cpp:3126]
11:51:15 INFO - #13: nsHTMLEditRules::DocumentModifiedWorker() [editor/libeditor/nsHTMLEditRules.cpp:9045]
11:51:15 INFO - #14: nsRunnableMethodImpl<void (nsHTMLEditRules::*)(), true>::Run() [xpcom/glue/nsThreadUtils.h:871]
11:51:15 INFO - #15: nsContentUtils::RemoveScriptBlocker() [xpcom/glue/nsCOMPtr.h:1066]
11:51:15 INFO - #16: nsDocument::EndUpdate(unsigned int) [dom/base/nsDocument.cpp:4901]
11:51:15 INFO - #17: nsHTMLDocument::EndUpdate(unsigned int) [dom/html/nsHTMLDocument.cpp:2511]
11:51:15 INFO - #18: nsContentSink::NotifyAppend(nsIContent*, unsigned int) [dom/base/mozAutoDocUpdate.h:40]
11:51:15 INFO - #19: nsXMLContentSink::HandleEndElement(char16_t const*, bool) [dom/xml/nsXMLContentSink.cpp:1115]
11:51:15 INFO - #20: nsExpatDriver::HandleEndElement(char16_t const*) [parser/htmlparser/nsExpatDriver.cpp:403]
11:51:15 INFO - #21: doContent [parser/expat/lib/xmlparse.c:2522]
11:51:15 INFO - #22: contentProcessor [parser/expat/lib/xmlparse.c:2068]
11:51:15 INFO - #23: doProlog [parser/expat/lib/xmlparse.c:4047]
11:51:15 INFO - #24: prologProcessor [parser/expat/lib/xmlparse.c:3783]
11:51:15 INFO - #25: MOZ_XML_Parse [parser/expat/lib/xmlparse.c:1526]
11:51:15 INFO - #26: nsExpatDriver::ParseBuffer(char16_t const*, unsigned int, bool, unsigned int*) [parser/htmlparser/nsExpatDriver.cpp:1023]
11:51:15 INFO - #27: nsExpatDriver::ConsumeToken(nsScanner&, bool&) [parser/htmlparser/nsExpatDriver.cpp:1117]
11:51:15 INFO - #28: nsParser::Tokenize(bool) [parser/htmlparser/nsParser.cpp:1944]
11:51:15 INFO - #29: nsParser::ResumeParse(bool, bool, bool) [parser/htmlparser/nsParser.cpp:1464]
11:51:15 INFO - #30: nsParser::OnDataAvailable(nsIRequest*, nsISupports*, nsIInputStream*, unsigned long, unsigned int) [parser/htmlparser/nsParser.cpp:1841]
11:51:15 INFO - #31: nsBaseChannel::OnDataAvailable(nsIRequest*, nsISupports*, nsIInputStream*, unsigned long, unsigned int) [netwerk/base/nsBaseChannel.cpp:833]
11:51:15 INFO - #32: nsInputStreamPump::OnStateTransfer() [netwerk/base/nsInputStreamPump.cpp:606]
11:51:15 INFO - #33: nsInputStreamPump::OnInputStreamReady(nsIAsyncInputStream*) [netwerk/base/nsInputStreamPump.cpp:432]
11:51:15 INFO - #34: nsInputStreamReadyEvent::Run() [xpcom/io/nsStreamUtils.cpp:93]
11:51:15 INFO - #35: nsThread::ProcessNextEvent(bool, bool*) [xpcom/threads/nsThread.cpp:864]
11:51:15 INFO - #36: NS_ProcessNextEvent(nsIThread*, bool) [xpcom/glue/nsThreadUtils.cpp:277]
11:51:15 INFO - #37: mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:96]
11:51:15 INFO - #38: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:235]
11:51:15 INFO - #39: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:520]
11:51:15 INFO - #40: nsBaseAppShell::Run() [widget/nsBaseAppShell.cpp:158]
11:51:15 INFO - #41: XRE_RunAppShell [toolkit/xre/nsEmbedFunctions.cpp:785]
11:51:15 INFO - #42: mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:259]
11:51:15 INFO - #43: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:235]
11:51:15 INFO - #44: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:520]
11:51:15 INFO - #45: XRE_InitChildProcess [toolkit/xre/nsEmbedFunctions.cpp:625]
11:51:15 INFO - #46: content_process_main(int, char**) [ipc/contentproc/plugin-container.cpp:240]
11:51:15 INFO - #47: libc.so.6 + 0x2176d
11:51:15 INFO - #48: _start
See Also: → 163838
Blocks: e10s-tests
tracking-e10s: --- → +
With the patch from bug 1245092, this seems to get reverted back to how it was before:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2b466c5dafec&filter-searchStr=win%207%20debug%20crashtest

Unless someone objects, I'll undo the changes made here and land it alongside that other bug.
Nvm, I read the log backwards.. My patch makes winWidget have the same number of assertions as gtkWidget.
Priority: -- → P3
This got fixed by bug 1395828.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
(In reply to Ryan VanderMeulen [:RyanVM] from comment #0)
> 11:51:15 INFO - [Child 1933] ###!!! ASSERTION: Parser and editor disagree on
> blockness: col: 'aIsBlock', file

For the record, parser was right and editor was wrong: 'col' is not a block element :)
You need to log in before you can comment on or make changes to this bug.