"ASSERTION: Range index not returned"

RESOLVED FIXED in mozilla1.9.3a1

Status

()

Core
Selection
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: graememcc)

Tracking

(Depends on: 1 bug, {assertion, regression, testcase})

Trunk
mozilla1.9.3a1
x86
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
editor/libeditor/html/crashtests/336081-1.xhtml

triggers

###!!! ASSERTION: Range index not returned: 'rangeIndex >= 0', file /Users/jruderman/central/layout/generic/nsSelection.cpp, line 4990
Flags: in-testsuite+
(Assignee)

Comment 1

8 years ago
Created attachment 401669 [details] [diff] [review]
Patch v1

More fallout from bug 348681.

The "overlap" case in AddItem fails to correctly set aOutIndex, triggering the assertion.
Assignee: nobody → graememcc_firefox
Status: NEW → ASSIGNED
Attachment #401669 - Flags: review?(roc)
(Assignee)

Updated

8 years ago
Blocks: 348681
Keywords: regression
Attachment #401669 - Flags: review?(roc) → review+
(Assignee)

Comment 2

8 years ago
http://hg.mozilla.org/mozilla-central/rev/8cfc5c84bddf
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
(Assignee)

Updated

8 years ago
Attachment #401669 - Flags: approval1.9.2?
(Assignee)

Comment 3

8 years ago
Comment on attachment 401669 [details] [diff] [review]
Patch v1

Requesting a1.9.2 for a trivial fix: aside from the assertion spew in debug builds, in certain circumstances this could cause unpredictable behaviour when performing multiple selections, as failing to return the index of the new range would lead to the focus and anchor nodes being reported as null.
Comment on attachment 401669 [details] [diff] [review]
Patch v1

a1.9.2=dbaron

But you should add showfunc = 1 to the [diff] section of your ~/.hgrc .
Attachment #401669 - Flags: approval1.9.2? → approval1.9.2+
(Assignee)

Comment 5

8 years ago
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/8eb768b8bccd
status1.9.2: --- → beta1-fixed

Updated

6 years ago
Depends on: 673785

Updated

5 years ago
Depends on: 766181
You need to log in before you can comment on or make changes to this bug.