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

Status: RESOLVED FIXED

(Toolkit :: Password Manager, defect)

Description

[Matthew N. [:MattN]]

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.

Comment 3

[Matthew N. [:MattN]]

Attachment: 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/

Comment 4

[Matthew N. [:MattN]]

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/

Comment 5

[Justin Dolske [:Dolske]]

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

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/c0fcde897668

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c0fcde897668