Closed
Bug 842132
Opened 11 years ago
Closed 11 years ago
Crash with contentEditable, selection.deleteFromDocument with overlapping selection ranges
Categories
(Core :: DOM: Selection, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox20 | --- | unaffected |
firefox21 | --- | fixed |
People
(Reporter: jruderman, Assigned: smaug)
References
Details
(Keywords: crash, regression, testcase)
Crash Data
Attachments
(4 files)
541 bytes,
application/xhtml+xml
|
Details | |
11.19 KB,
text/plain
|
Details | |
2.02 KB,
patch
|
tbsaunde
:
review+
|
Details | Diff | Splinter Review |
3.47 KB,
patch
|
Details | Diff | Splinter Review |
Regression from bug 841706?
Reporter | ||
Comment 1•11 years ago
|
||
[@ nsFrameSelection::DeleteFromDocument]
Comment 2•11 years ago
|
||
On Windows: bp-9211abf3-9353-4811-9a58-83f2d2130217
Crash Signature: [@ nsFrameSelection::DeleteFromDocument() ]
status-firefox20:
--- → unaffected
status-firefox21:
--- → affected
OS: Mac OS X → All
Hardware: x86_64 → All
Version: Trunk → 21 Branch
Assignee | ||
Comment 3•11 years ago
|
||
Assignee: nobody → bugs
Attachment #714943 -
Flags: review?(trev.saunders)
Comment 4•11 years ago
|
||
Comment on attachment 714943 [details] [diff] [review] patch I guess this fine to bring fix the crash and keep old behavior for this release. It would be nice if you made the test case into a test. thanks for working on this!
Attachment #714943 -
Flags: review?(trev.saunders) → review+
Assignee | ||
Comment 5•11 years ago
|
||
I'll push the test as a crash test if that works... testing.
Assignee | ||
Comment 6•11 years ago
|
||
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/645cb4fbf390
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 8•11 years ago
|
||
Backed out for build bustage. https://hg.mozilla.org/mozilla-central/rev/4514f27f41ff https://tbpl.mozilla.org/php/getParsedLog.php?id=19845950&tree=Firefox cc1plus: warnings being treated as errors ../../../../editor/libeditor/base/nsEditor.cpp: In member function 'nsresult nsEditor::CreateTxnForDeleteSelection(nsIEditor::EDirection, EditAggregateTxn**, nsINode**, int32_t*, int32_t*)': ../../../../editor/libeditor/base/nsEditor.cpp:4569:67: error: comparison between signed and unsigned integer expressions make[7]: *** [nsEditor.o] Error 1
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 9•11 years ago
|
||
Maybe this works. https://hg.mozilla.org/mozilla-central/rev/148ea95584b4
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•