475132 - "ASSERTION: selection could not be collapsed after insert" with contentEditable, execCommand, tables

Status: RESOLVED WORKSFORME

(Core :: DOM: Editor, defect)

Description

[Jesse Ruderman]

Attachment: testcase

Loading the testcase:
###!!! ASSERTION: selection could not be collapsed after insert.: '(NS_SUCCEEDED(result))', file editor/libeditor/base/CreateElementTxn.cpp, line 153
###!!! ASSERTION: transaction did not execute properly: '(NS_SUCCEEDED(result))', file editor/libeditor/base/nsEditor.cpp, line 709
###!!! ASSERTION: null node passed to nsEditor::Tag(): 'aNode', file editor/libeditor/base/nsEditor.cpp, line 3893

Unloading the testcase:
###!!! ASSERTION: Please remove this from the document properly: '!IsInDoc()', file content/base/src/nsGenericElement.cpp, line 1758

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Is this bug only about the assertion in the summary, or all of them?

Because the assertion in the summary happens a lot in tinderbox logs, randomly.  I suspect it may be related to cycle collection, though I don't have a useful stack for it.

Comment 2

[Jesse Ruderman]

I intended the bug to be about all of the assertions, but you can do what you want with it :)

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I filed bug 563980 to cover that assertion being frequently random on tinderbox.

Comment 4

[Jesse Ruderman]

Attachment: stack and info for "ASSERTION: Please remove this from the document properly"

Includes debugging output from the printf added in http://hg.mozilla.org/mozilla-central/rev/76e0ed39f6ad

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It seems to be the no pres shell case.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Er, meant that for bug 563980.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Bug 563980 should fix the last of the assertions (the one in the summary), but the others remain (although I don't see them every time I load the testcase).

Comment 8

[Jesse Ruderman]

WFM. I don't get any assertions when I load the testcase. I'll file new bugs if I come across them again ;)