Closed Bug 1446527 Opened 3 years ago Closed 3 years ago

Get rid of nsIDOMUIEvent

Categories

(Core :: DOM: Events, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(7 files)

No description provided.
Depends on: 1446708
Depends on: 1446709
Assignee: nobody → bzbarsky
Component: DOM → DOM: Events
Depends on: 1446710
Depends on: 1446711
Priority: -- → P2
Blocks: 1447472
MozReview-Commit-ID: 86cMORXSV9u
Attachment #8960869 - Flags: review?(kyle)
MozReview-Commit-ID: GlTgti53Akj
Attachment #8960870 - Flags: review?(kyle)
MozReview-Commit-ID: 9KqSXRUIc8G
Attachment #8960871 - Flags: review?(kyle)
MozReview-Commit-ID: 6F9Q6kOogEo
Attachment #8960872 - Flags: review?(kyle)
MozReview-Commit-ID: IPzWAakqHN4
Attachment #8960873 - Flags: review?(kyle)
MozReview-Commit-ID: 7GbifGJQUtQ
Attachment #8960874 - Flags: review?(kyle)
We no longer have multiple inheritance from nsIDOMEvent anywhere.

This looks like it saves 20KB of codesize or so.

MozReview-Commit-ID: qJtM7QNu9z
Attachment #8960875 - Flags: review?(kyle)
Attachment #8960869 - Flags: review?(kyle) → review+
Attachment #8960870 - Flags: review?(kyle) → review+
Attachment #8960871 - Flags: review?(kyle) → review+
Attachment #8960872 - Flags: review?(kyle) → review+
Attachment #8960873 - Flags: review?(kyle) → review+
Attachment #8960874 - Flags: review?(kyle) → review+
Attachment #8960875 - Flags: review?(kyle) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/899c9f492988
part 1.  Remove rangeParent/rangeOffset members from nsIDOMUIEvent.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/b2169fd28b66
part 2.  Remove nsIDOMUIEvent::GetDetail.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/04ce1f71a72b
part 3.  Remove nsIDOMUIEvent::AsEvent.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/b3c4650657af
part 4.  Remove nsIDOMUIEvent::SCROLL_PAGE_* constants.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/be7e5f1bd0a1
part 5.  Remove unused nsIDOMUIEvent bits.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/357845bfcad2
part 6.  Remove nsIDOMUIEvent.  r=qdot
https://hg.mozilla.org/integration/mozilla-inbound/rev/dc93e64c63e9
part 7.  Remove the various NS_FORWARD_TO_EVENT bits.  r=qdot
You need to log in before you can comment on or make changes to this bug.