Closed
Bug 1318096
Opened 8 years ago
Closed 8 years ago
Remove some unused document/element/etc xpidl code
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
Attachments
(9 files)
8.38 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
4.73 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
7.49 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
5.03 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
2.53 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
1.92 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
7.90 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
5.93 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
10.64 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
We no longer have binary extensions, so we can nix some of this stuff. Less codesize is good, right?
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8811486 -
Flags: review?(nfroyd)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8811487 -
Flags: review?(nfroyd)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8811488 -
Flags: review?(nfroyd)
Assignee | ||
Comment 4•8 years ago
|
||
Attachment #8811489 -
Flags: review?(nfroyd)
Assignee | ||
Comment 5•8 years ago
|
||
Attachment #8811490 -
Flags: review?(nfroyd)
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8811491 -
Flags: review?(nfroyd)
Assignee | ||
Comment 7•8 years ago
|
||
Attachment #8811492 -
Flags: review?(nfroyd)
Assignee | ||
Comment 8•8 years ago
|
||
Attachment #8811494 -
Flags: review?(nfroyd)
Assignee | ||
Comment 9•8 years ago
|
||
Attachment #8811495 -
Flags: review?(nfroyd)
Updated•8 years ago
|
Attachment #8811486 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811487 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811488 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811489 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811490 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811491 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811492 -
Flags: review?(nfroyd) → review+
Updated•8 years ago
|
Attachment #8811494 -
Flags: review?(nfroyd) → review+
Comment 10•8 years ago
|
||
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+
Comment 11•8 years ago
|
||
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
Comment 12•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d73246f35fb2 https://hg.mozilla.org/mozilla-central/rev/39c742242411 https://hg.mozilla.org/mozilla-central/rev/7c6f51d784da https://hg.mozilla.org/mozilla-central/rev/7ca78018918e https://hg.mozilla.org/mozilla-central/rev/11964097d033 https://hg.mozilla.org/mozilla-central/rev/0f66a6a676ef https://hg.mozilla.org/mozilla-central/rev/90d39fcf8890 https://hg.mozilla.org/mozilla-central/rev/cdb9cc8de7a0 https://hg.mozilla.org/mozilla-central/rev/a246493b8a5c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Comment 13•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d73246f35fb2 https://hg.mozilla.org/mozilla-central/rev/39c742242411 https://hg.mozilla.org/mozilla-central/rev/7c6f51d784da https://hg.mozilla.org/mozilla-central/rev/7ca78018918e https://hg.mozilla.org/mozilla-central/rev/11964097d033 https://hg.mozilla.org/mozilla-central/rev/0f66a6a676ef https://hg.mozilla.org/mozilla-central/rev/90d39fcf8890 https://hg.mozilla.org/mozilla-central/rev/cdb9cc8de7a0 https://hg.mozilla.org/mozilla-central/rev/a246493b8a5c
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•