Closed Bug 995757 Opened 5 years ago Closed 5 years ago

Remove nsIDOMMozApplicationEvent

Categories

(Core :: DOM: Core & HTML, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED WONTFIX

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(1 file)

Attached patch Patch (v1)Splinter Review
No description provided.
Attachment #8405867 - Flags: review?(bzbarsky)
Blocks: 994964
Comment on attachment 8405867 [details] [diff] [review]
Patch (v1)

r=me
Attachment #8405867 - Flags: review?(bzbarsky) → review+
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.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.