nsEventDispatcher::CreateEvent should return nsDOMEvent instead of nsIDOMEvent

RESOLVED FIXED in Firefox 43

Status

()

enhancement
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: ayg, Assigned: ayg)

Tracking

2.0 Branch
mozilla43
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Posted patch Patch (obsolete) — Splinter Review
Assignee: nobody → ayg
Status: NEW → ASSIGNED
Attachment #8645686 - Flags: review?(bugs)
Comment on attachment 8645686 [details] [diff] [review]
Patch

There's an include order problem here, will fix and resubmit.
Attachment #8645686 - Attachment is obsolete: true
Attachment #8645686 - Flags: review?(bugs)
Posted patch PatchSplinter Review
Try is green, except for B2G-specific failures that might be fixed by this patch revision: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d546000d1301
Attachment #8646422 - Flags: review?(khuey)
Comment on attachment 8646422 [details] [diff] [review]
Patch

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

::: dom/events/AnimationEvent.h
@@ +54,5 @@
>  
> +already_AddRefed<mozilla::dom::AnimationEvent>
> +NS_NewDOMAnimationEvent(mozilla::dom::EventTarget* aOwner,
> +                        nsPresContext* aPresContext,
> +                        mozilla::InternalAnimationEvent* aEvent);

nit: \n before #endif, here and everywhere.

::: dom/events/CustomEvent.cpp
@@ +111,5 @@
>    VariantToJsval(aCx, mDetail, aRetval);
>  }
>  
> +already_AddRefed<CustomEvent>
> +NS_NewDOMCustomEvent(mozilla::dom::EventTarget* aOwner,

Just EventTarget?

::: dom/svg/SVGZoomEvent.cpp
@@ +93,5 @@
> +using namespace mozilla;
> +using namespace mozilla::dom;
> +
> +already_AddRefed<SVGZoomEvent>
> +NS_NewDOMSVGZoomEvent(mozilla::dom::EventTarget* aOwner,

just EventTarget?
Attachment #8646422 - Flags: review?(khuey) → review+
Flags: in-testsuite-
(I forgot the r=, oops.  We don't have a pre-commit hook for that?)
https://hg.mozilla.org/mozilla-central/rev/68f0b9143193
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.