Closed Bug 1377980 Opened 2 years ago Closed Last year

Remove nsIDOMRange

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: mats, Assigned: bzbarsky)

References

(Blocks 2 open bugs)

Details

Attachments

(4 files)

I think we should replace all uses of nsIDOMRange to use nsRange instead
(or mozilla::dom::Range after it's renamed).
Blocks: 1377981
Priority: -- → P3
Depends on: 1395080
Adrian, are you working on this on?  If not, I might grab it...
Flags: needinfo?(adrian.wielgosik)
I'm not, feel free to take it. Just mind one small nsIDOMRange conversion already done in bug 1447121.
Flags: needinfo?(adrian.wielgosik)
Thanks for the heads-up!
OK.  I'm going to file some bugs blocking this one, because the patch queue was getting ridiculous and there are intermediate states that are worth landing.
This will be a lot simpler/cleaner once bug 1444991 is fixed.
Depends on: 1444991
Assignee: nobody → bzbarsky
Attachment #8976390 - Flags: review?(continuation) → review+
Attachment #8976391 - Flags: review?(continuation) → review+
Attachment #8976392 - Flags: review?(continuation) → review+
Attachment #8976393 - Flags: review?(continuation) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ea8b11ebde55
part 1.  Remove use of nsIDOMRange in xpidl.  r=mcc8
https://hg.mozilla.org/integration/mozilla-inbound/rev/352f6f0a552a
part 2.  Remove most C++ use of nsIDOMRange.  r=mccr8
https://hg.mozilla.org/integration/mozilla-inbound/rev/cbd8ba61e49c
part 3.  Remove nsIDOMRange use in JS.  r=mccr8
https://hg.mozilla.org/integration/mozilla-inbound/rev/0846a3df31b4
part 4.  Remove nsIDOMRange.  r=mccr8
You need to log in before you can comment on or make changes to this bug.