Closed Bug 1377980 Opened 7 years ago Closed 6 years ago

Remove nsIDOMRange

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: MatsPalmgren_bugz, Assigned: bzbarsky)

References

(Blocks 2 open bugs)

Details

Attachments

(4 files)

part 1. Remove use of nsIDOMRange in xpidl
(deleted), patch
mccr8
: review+
Details | Diff | Splinter Review
part 2. Remove most C++ use of nsIDOMRange
(deleted), patch
mccr8
: review+
Details | Diff | Splinter Review
part 3. Remove nsIDOMRange use in JS
(deleted), patch
mccr8
: review+
Details | Diff | Splinter Review
part 4. Remove nsIDOMRange
(deleted), patch
mccr8
: review+
Details | Diff | Splinter Review
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)
Depends on: 1447121
Thanks for the heads-up!
Blocks: 1132934
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.
Depends on: 1447889
Depends on: 1447890
This will be a lot simpler/cleaner once bug 1444991 is fixed.
Depends on: 1444991
Assignee: nobody → bzbarsky
Attachment #8976390 - Flags: review?(continuation)
Attachment #8976391 - Flags: review?(continuation)
Attachment #8976392 - Flags: review?(continuation)
Attached patch part 4. Remove nsIDOMRange (deleted) — — Splinter Review
Attachment #8976393 - Flags: review?(continuation)
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.

Attachment

General

Creator:
Created:
Updated:
Size: