Closed
Bug 1446851
Opened 7 years ago
Closed 7 years ago
Get rid of nsIDOMWheelEvent
Categories
(Core :: DOM: Events, enhancement, P2)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
42.36 KB,
patch
|
qdot
:
review+
|
Details | Diff | Splinter Review |
No description provided.
![]() |
Assignee | |
Comment 1•7 years ago
|
||
We can't include WheelEventBinding.h in MouseEvents.h because that produces
this include loop:
WheelEventBinding.h -> MouseEventBinding.h -> UIEventBinding.h ->
nsGlobalWindow.h -> nsGlobalWindowInner.h -> nsRefreshDriver.h ->
AnimationEventDispatcher.h -> AnimationComparator.h -> Animation.h ->
EffectCompositor.h -> PseudoElementHashEntry.h -> Element.h ->
PointerEventHandler.h -> MouseEvents.h -> WheelEventBinding.h
MozReview-Commit-ID: 5KNwH69aJYW
Attachment #8960073 -
Flags: review?(kyle)
Updated•7 years ago
|
Priority: -- → P2
Updated•7 years ago
|
Attachment #8960073 -
Flags: review?(kyle) → review+
Updated•7 years ago
|
Summary: Get rid of nsIDOWheelEvent → Get rid of nsIDOMWheelEvent
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ff93374ac3d2
Get rid of nsIDOMWheelEvent. r=qdot
Comment 3•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in
before you can comment on or make changes to this bug.
Description
•