Closed
Bug 535632
Opened 15 years ago
Closed 15 years ago
Crash [@ nsEditor::GetIndexOf]
Categories
(Core :: DOM: Editor, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.3a1
People
(Reporter: jruderman, Assigned: masayuki)
References
Details
(Keywords: assertion, crash, testcase)
Crash Data
Attachments
(2 files)
|
147 bytes,
application/xhtml+xml
|
Details | |
|
2.17 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
###!!! ASSERTION: null content in nsEditor::GetIndexOf: 'content', file /Users/jruderman/mozilla-central/editor/libeditor/base/nsEditor.cpp, line 4045
###!!! ASSERTION: You can't dereference a NULL nsCOMPtr with operator->().: 'mRawPtr != 0', file ../../../dist/include/nsCOMPtr.h, line 796
|
Assignee | |
Comment 1•15 years ago
|
||
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•15 years ago
|
Attachment #418333 -
Flags: review?(Olli.Pettay)
|
|
Assignee | |
Comment 2•15 years ago
|
||
Comment on attachment 418333 [details] [diff] [review]
Patch v1.0
Olli:
Can you review this? I'm not sure who is good editor reviewer...
At the testcase, the outdenting tries to get the node index of the body element. However, its parent is document node, therefore, it's crashed.
Attachment #418333 -
Flags: review?(Olli.Pettay) → review+
|
|
Assignee | |
Comment 3•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/35e6583db6ac
Should this be fixed on the branch? I'm not sure whether this is important.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
|
||
Updated•13 years ago
|
Crash Signature: [@ nsEditor::GetIndexOf]
You need to log in
before you can comment on or make changes to this bug.
Description
•