Closed
Bug 1158837
Opened 9 years ago
Closed 9 years ago
crash in nsINode::GetAsText()
Categories
(Core :: DOM: Editor, defect)
Tracking
()
VERIFIED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | verified |
People
(Reporter: phorea, Assigned: ehsan.akhgari)
References
Details
(Keywords: crash)
Crash Data
Attachments
(1 file)
1.26 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-e38cbe46-7ea3-4d39-bbc6-92f242150423. ============================================================= There are a few crashes in Socorro after bug 1146883 landed: https://crash-stats.mozilla.com/report/index/47db6d1c-2460-4c2b-b1d9-0b4932150424 https://crash-stats.mozilla.com/report/index/bc323988-f597-40f4-b4a0-20fdd2150423 https://crash-stats.mozilla.com/report/index/e38cbe46-7ea3-4d39-bbc6-92f242150423 No steps or relevant information until now on comments.
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(ehsan)
Assignee | ||
Comment 1•9 years ago
|
||
This seems to be caused by mRangeItem->startNode being null. I don't understand how this can be null except if the start part of the range inside the selection is null, which kind of makes no sense, but I think the selection may be in an inconsistent state at this point. But anyways, wallpapering over this seems totally worth it, since we'd just leave a bunch of 's in the DOM, which seems like a survivable consequence.
Assignee: nobody → ehsan
Flags: needinfo?(ehsan)
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8599639 -
Flags: review?(roc)
Attachment #8599639 -
Flags: review?(roc) → review+
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bdebff6d0770
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Comment 6•9 years ago
|
||
Socorro [1] shows zero crashes over the past 28 days, in versions 40+. [1] - https://crash-stats.mozilla.com/report/list?range_unit=days&range_value=28&signature=nsINode%3A%3AGetAsText%28%29#tab-sigsummary
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•