Closed Bug 1591588 Opened 5 years ago Closed 5 years ago

Remove the negative index check from GetWholeText

Categories

(Core :: DOM: Core & HTML, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: bgrins, Assigned: emmamalysz)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

From https://phabricator.services.mozilla.com/D50650#inline-307138:

Can you please file a followup to remove the negative index check from GetWholeText? The implementation no longer relies on that check anyway.

Once test_bug444030.xhtml goes away in Bug 1587142 we can make this change.

Attachment #9104803 - Attachment description: Bug 1591588, remove the negative index check from within GetWholeText → Bug 1591588, remove ComputeIndexOf call and negative index check from within GetWholeText
Pushed by bgrinstead@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4139bfe5e7f9 remove ComputeIndexOf call and negative index check from within GetWholeText r=bzbarsky
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Assignee: nobody → emalysz
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: