Closed Bug 856960 Opened 11 years ago Closed 11 years ago

Remove some quickstubs

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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

Hmm.  So for the nsIDOMEvent bits, are all the event subclasses on WebIDL bindings?

If not, I don't think we should remove those, since we're not using WebIDL quickstubs for nsIDOMEvent either.  I'd like Olli to check that part over.

> -    'nsIDOMHTMLCommandElement.*',

Mind removing that interface as long as you're here?  There is no such beastie in the spec afaict.  Followup bug is fine for this.

r=me on the rest.
Attachment #732212 - Flags: review?(bzbarsky)
Attachment #732212 - Flags: review?(bugs)
Attachment #732212 - Flags: review+
Comment on attachment 732212 [details] [diff] [review]
Patch v1

I'd like to remove the event bits once all the events use WebIDL.
depending on reviews, later this week or so.
Attachment #732212 - Flags: review?(bugs)
https://hg.mozilla.org/mozilla-central/rev/94c8840a989c
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: