Closed Bug 1591588 Opened Last month Closed Last month

Remove the negative index check from GetWholeText

Categories

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

task
Not set

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: bgrins, Assigned: emalysz)

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: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.