Closed Bug 995757 Opened 5 years ago Closed 5 years ago
IDOMMoz Application Event
No description provided.
Attachment #8405867 - Flags: review?(bzbarsky)
Comment on attachment 8405867 [details] [diff] [review] Patch (v1) r=me
Attachment #8405867 - Flags: review?(bzbarsky) → review+
Backed out for cross-platform bustage. https://hg.mozilla.org/integration/mozilla-inbound/rev/bb6713d8c5eb https://tbpl.mozilla.org/php/getParsedLog.php?id=37724973&tree=Mozilla-Inbound
Seems like the dependency rules for regenerating GeneratedEvents.cpp are insufficient and need to be fixed first.
Well, MozApplicationEvent is in simple_events here <http://mxr.mozilla.org/mozilla-central/source/js/xpconnect/src/event_impl_gen.conf.in#21> and this code <http://mxr.mozilla.org/mozilla-central/source/js/xpconnect/src/event_impl_gen.py#113> clearly tries to load the IDL file. I don't think this is a dependency issue.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
I thought we could do events with WEbIDL-only codegen now?
Yes. New events should use WebIDL-only codegen. Some legacy stuff is still using IDL+WebIDL setup.
(In reply to comment #7) > Yes. New events should use WebIDL-only codegen. > Some legacy stuff is still using IDL+WebIDL setup. Can we move MozApplicationEvent to be WebIDL only (and how)? If that is feasible we could still remove this IDL file I think.
You need to log in before you can comment on or make changes to this bug.