Closed Bug 1451426 Opened 2 years ago Closed 2 years ago

add an iframe event listener leak test

Categories

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

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: bkelly, Assigned: bkelly)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 3 obsolete files)

Continuation of bug 1450358.
This tests a nested iframe for event listener leaks.
Attachment #8965006 - Flags: review?(amarchesini)
I forgot to hg add the test in the previous patch.
Attachment #8965006 - Attachment is obsolete: true
Attachment #8965006 - Flags: review?(amarchesini)
Attachment #8965036 - Flags: review?(amarchesini)
Comment on attachment 8965037 [details] [diff] [review]
P2 Test that Notification does not leak via its event listeners. r=baku

Andrea, this tests the Notification API for event listener leaks.
Attachment #8965037 - Flags: review?(amarchesini)
Attachment #8965036 - Flags: review?(amarchesini) → review+
Attachment #8965037 - Flags: review?(amarchesini) → review+
It appears the test triggers some NS_ASSERTION() stuff in Notification...
Comment on attachment 8965037 [details] [diff] [review]
P2 Test that Notification does not leak via its event listeners. r=baku

Moved to bug 1451769.
Attachment #8965037 - Attachment is obsolete: true
Let's just make this about testing iframe.
Summary: MOAR event listener leak tests → add an iframe event listener leak test
Pushed by bkelly@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/08fa7a65ad2f
Test that iframe does not leak cotent windows through event listeners. r=baku
https://hg.mozilla.org/mozilla-central/rev/08fa7a65ad2f
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.