Make DISPATCH_NORMAL be the default flags argument for nsIEventTarget::Dispatch

RESOLVED FIXED in Firefox 55

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: billm, Assigned: billm)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Posted patch patchSplinter Review
I'm trying to make nsIEventTarget easier to use, and the requirement that you pass flags in is pretty annoying. This patch changes XPIDL to allow default parameter values to be specified (as long as they're identifiers).

I also added a new #define, NS_DECL_NSIEVENTTARGET_FULL, that does "using nsIEventTarget::Dispatch;". Originally this was more useful when I was trying to make this patch work in a different way. Now it's just a slight simplification. i can remove it if you like.
Attachment #8870132 - Flags: review?(nfroyd)
Comment on attachment 8870132 [details] [diff] [review]
patch

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

This seems like a nice cleanup, thank you.
Attachment #8870132 - Flags: review?(nfroyd) → review+
Pushed by wmccloskey@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9b7affc8fb26
Make DISPATCH_NORMAL be the default flags argument for nsIEventTarget::Dispatch (r=froydnj)
https://hg.mozilla.org/mozilla-central/rev/9b7affc8fb26
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.