Closed Bug 950659 Opened 6 years ago Closed 6 years ago

Remove nsIDOMMediaQueryList and nsIDOMMediaQueryListListener

Categories

(Core :: CSS Parsing and Computation, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(2 files)

Follow-up to bug 950523.
Assignee: nobody → ehsan
Attachment #8348025 - Flags: review?(bzbarsky)
Comment on attachment 8348025 [details] [diff] [review]
Remove the XPCOM versions of DOMQueryList::AddListener and RemoveListener; r=bzbarsky

Just remove them from the idl.  Why not?
Attachment #8348025 - Flags: review?(bzbarsky) → review+
Can do, I was a bit nervous doing that but I couldn't convince myself why :-)
Actually it just occurred to me that there should not be any reason why we can't kill all of these interfaces in one go!
Attached patch Kill'em allSplinter Review
What do you say to this, folded on top of the previous patch? :-)
Attachment #8348490 - Flags: review?(bzbarsky)
Comment on attachment 8348490 [details] [diff] [review]
Kill'em all

I approve of the preponderance of '-' at the starts of lines here.  Very good.
Attachment #8348490 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b7648edfcbb
Summary: Remove the XPCOM versions of DOMQueryList::AddListener and RemoveListener → Remove nsIDOMMediaQueryList and nsIDOMMediaQueryListListener
https://hg.mozilla.org/mozilla-central/rev/1b7648edfcbb
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.