Closed Bug 1453487 Opened 6 years ago Closed 6 years ago

Make nsIDOMEventListener noscript

Categories

(Core :: DOM: Events, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(3 files)

Figured out how to do this.
MozReview-Commit-ID: Es4fEdGDzbx
Attachment #8967180 - Flags: review?(bugs)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
MozReview-Commit-ID: 4OjGuYkpXMM
Attachment #8967183 - Flags: review?(bugs)
MozReview-Commit-ID: 9jakAl7hMbb
Attachment #8967184 - Flags: review?(bugs)
Attachment #8967183 - Flags: review?(bugs) → review+
Attachment #8967184 - Flags: review?(bugs) → review+
Comment on attachment 8967180 [details] [diff] [review]
part 1.  Stop using XPCWrappedNative for nsIDOMEventListener

>+   * Takes the given listener, wraps it in a filter that filters out events from
>+   * dynamic docShells. and
, and
or
. And
Attachment #8967180 - Flags: review?(bugs) → review+
>, and

Indeed, fixed.
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a5d3181f3b27
part 1.  Stop using XPCWrappedNative for nsIDOMEventListener.  r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a9b18f3f4ac
part 2.  Remove useless implements="nsIDOMEventListener" from XBL bindings.  r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/03a535f247d8
part 3.  Make nsIDOMEventListener noscript.  r=smaug
Priority: -- → P3
https://hg.mozilla.org/mozilla-central/rev/a5d3181f3b27
https://hg.mozilla.org/mozilla-central/rev/2a9b18f3f4ac
https://hg.mozilla.org/mozilla-central/rev/03a535f247d8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: