Closed Bug 1318096 Opened 3 years ago Closed 3 years ago

Remove some unused document/element/etc xpidl code

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(9 files)

We no longer have binary extensions, so we can nix some of this stuff.  Less codesize is good, right?
Blocks: 1318117
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8811486 - Flags: review?(nfroyd) → review+
Attachment #8811487 - Flags: review?(nfroyd) → review+
Attachment #8811488 - Flags: review?(nfroyd) → review+
Attachment #8811489 - Flags: review?(nfroyd) → review+
Attachment #8811490 - Flags: review?(nfroyd) → review+
Attachment #8811491 - Flags: review?(nfroyd) → review+
Attachment #8811492 - Flags: review?(nfroyd) → review+
Attachment #8811494 - Flags: review?(nfroyd) → review+
Comment on attachment 8811495 [details] [diff] [review]
part 9.  Remove XPCOM parentnode/childnode APIs

Review of attachment 8811495 [details] [diff] [review]:
-----------------------------------------------------------------

So great to have all these virtual methods cleaned out of the vtables.  Thank you!
Attachment #8811495 - Flags: review?(nfroyd) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d73246f35fb2
part 1.  Remove unused XPCOM querySelector(All) bits.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/39c742242411
part 2.  Remove the unused XPCOM pointerlock API.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c6f51d784da
part 3.  Remove the unused XPCOM fullscreen API.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/7ca78018918e
part 4.  Remove the unused XPCOM setCapture/removeCapture.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/11964097d033
part 5.  Remove XPCOM mozMatchesSelector.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/0f66a6a676ef
part 6.  Remove XPCOM document.compatMode.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/90d39fcf8890
part 7.  Remove XPCOM element scroll* APIs.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/cdb9cc8de7a0
part 8.  Remove XPCOM element client* APIs.  r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/a246493b8a5c
part 9.  Remove XPCOM parentnode/childnode APIs.  r=froydnj
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.