Convert test_notifications_popup.html to b-c to work in e10s

RESOLVED FIXED in Firefox 49

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

(Depends on: 1 bug, Blocks: 1 bug)

unspecified
mozilla49
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(e10s+, firefox49 fixed)

Details

Attachments

(1 attachment)

This test is testing password manager doorhangers so can be moved to browser-chrome.

I noticed that this test is no longer testing what it is supposed to since we don't open all of the window.open calls in actual popups by default anymore.
Duplicate of this bug: 934057
Duplicate of this bug: 1137954
Created attachment 8744453 [details]
MozReview Request: Bug 1266825 - Convert test_notifications_popup.html to b-c to work in e10s. r=dolske

Review commit: https://reviewboard.mozilla.org/r/48551/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/48551/
Attachment #8744453 - Flags: review?(dolske)
Comment on attachment 8744453 [details]
MozReview Request: Bug 1266825 - Convert test_notifications_popup.html to b-c to work in e10s. r=dolske

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/48551/diff/1-2/
tracking-e10s: ? → +
Comment on attachment 8744453 [details]
MozReview Request: Bug 1266825 - Convert test_notifications_popup.html to b-c to work in e10s. r=dolske

https://reviewboard.mozilla.org/r/48551/#review45883
Attachment #8744453 - Flags: review?(dolske) → review+

Comment 7

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c0fcde897668
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Blocks: 1268163
You need to log in before you can comment on or make changes to this bug.