Closed
Bug 1377980
Opened 7 years ago
Closed 6 years ago
Remove nsIDOMRange
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: MatsPalmgren_bugz, Assigned: bzbarsky)
References
(Blocks 2 open bugs)
Details
Attachments
(4 files)
(deleted),
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
(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).
Updated•7 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•7 years ago
|
||
Adrian, are you working on this on? If not, I might grab it...
Flags: needinfo?(adrian.wielgosik)
Comment 2•7 years ago
|
||
I'm not, feel free to take it. Just mind one small nsIDOMRange conversion already done in bug 1447121.
Flags: needinfo?(adrian.wielgosik)
Assignee | ||
Comment 3•7 years ago
|
||
Thanks for the heads-up!
Assignee | ||
Comment 4•7 years ago
|
||
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.
Assignee | ||
Comment 5•7 years ago
|
||
This will be a lot simpler/cleaner once bug 1444991 is fixed.
Depends on: 1444991
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bzbarsky
Assignee | ||
Comment 6•6 years ago
|
||
Attachment #8976390 -
Flags: review?(continuation)
Assignee | ||
Comment 7•6 years ago
|
||
Attachment #8976391 -
Flags: review?(continuation)
Assignee | ||
Comment 8•6 years ago
|
||
Attachment #8976392 -
Flags: review?(continuation)
Assignee | ||
Comment 9•6 years ago
|
||
Attachment #8976393 -
Flags: review?(continuation)
Updated•6 years ago
|
Attachment #8976390 -
Flags: review?(continuation) → review+
Updated•6 years ago
|
Attachment #8976391 -
Flags: review?(continuation) → review+
Updated•6 years ago
|
Attachment #8976392 -
Flags: review?(continuation) → review+
Updated•6 years ago
|
Attachment #8976393 -
Flags: review?(continuation) → review+
Comment 10•6 years ago
|
||
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
Comment 11•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ea8b11ebde55
https://hg.mozilla.org/mozilla-central/rev/352f6f0a552a
https://hg.mozilla.org/mozilla-central/rev/cbd8ba61e49c
https://hg.mozilla.org/mozilla-central/rev/0846a3df31b4
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•