browser_bug703210.js does not remove event listener

RESOLVED FIXED in mozilla11

Status

()

Core
XP Toolkit/Widgets: XUL
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: heycam, Assigned: heycam)

Tracking

unspecified
mozilla11
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
This causes problems for later tests that have popups.
(Assignee)

Comment 1

6 years ago
Created attachment 578473 [details] [diff] [review]
Remove event listener added in browser_bug703210.js.
Attachment #578473 - Flags: review?(masayuki)
(Assignee)

Updated

6 years ago
Blocks: 703176
(Assignee)

Updated

6 years ago
Assignee: nobody → cam
Status: NEW → ASSIGNED
Comment on attachment 578473 [details] [diff] [review]
Remove event listener added in browser_bug703210.js.

Wow, thank you.

Could you replace all |onPopupShowing| with |onPopupShown| before landing?
Attachment #578473 - Flags: review?(masayuki) → review+
(Assignee)

Comment 3

6 years ago
Sure thing, thanks for the quick review. :)
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/99860d7fd6ac

Comment 5

6 years ago
https://hg.mozilla.org/mozilla-central/rev/99860d7fd6ac
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.