Closed Bug 1366869 Opened 4 years ago Closed 4 years ago

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

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: billm, Assigned: billm)

Details

Attachments

(1 file)

Attached 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: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.