1381944 - browser_notification_open_settings.js needs to work for both old and new about:preferences organizations

Status: RESOLVED FIXED

(Firefox :: Settings UI, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

We're getting this on mozilla-beta:

[task 2017-07-18T17:23:45.828672Z] 17:23:45     INFO - TEST-START | browser/base/content/test/alerts/browser_notification_open_settings.js
[task 2017-07-18T17:25:16.082997Z] 17:25:16     INFO - TEST-INFO | started process screentopng
[task 2017-07-18T17:25:17.692990Z] 17:25:17     INFO - TEST-INFO | screentopng: exit 0
[task 2017-07-18T17:25:17.713904Z] 17:25:17     INFO - Buffered messages logged at 17:23:45
[task 2017-07-18T17:25:17.714407Z] 17:25:17     INFO - Entering test bound test_settingsOpen_observer
[task 2017-07-18T17:25:17.717226Z] 17:25:17     INFO - Opening a dummy tab so openPreferences=>switchToTabHavingURI doesn't use the blank tab.
[task 2017-07-18T17:25:17.719591Z] 17:25:17     INFO - Buffered messages logged at 17:23:46
[task 2017-07-18T17:25:17.719962Z] 17:25:17     INFO - simulate a notifications-open-settings notification
[task 2017-07-18T17:25:17.720197Z] 17:25:17     INFO - Buffered messages finished
[task 2017-07-18T17:25:17.721584Z] 17:25:17     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/alerts/browser_notification_open_settings.js | Test timed out - 
[task 2017-07-18T17:25:17.721954Z] 17:25:17     INFO - GECKO(1098) | MEMORY STAT | vsize 20973762MB | residentFast 581MB
[task 2017-07-18T17:25:17.722357Z] 17:25:17     INFO - TEST-OK | browser/base/content/test/alerts/browser_notification_open_settings.js | took 90305ms
[task 2017-07-18T17:25:17.729358Z] 17:25:17     INFO - Not taking screenshot here: see the one that was previously logged
[task 2017-07-18T17:25:17.734731Z] 17:25:17     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/alerts/browser_notification_open_settings.js | Found a tab after previous test timed out: about:preferences#content - 
[task 2017-07-18T17:25:17.737737Z] 17:25:17     INFO - Not taking screenshot here: see the one that was previously logged
[task 2017-07-18T17:25:17.739900Z] 17:25:17     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/alerts/browser_notification_open_settings.js | Found a tab after previous test timed out: about:robots -

Comment 1

[Mike Conley (:mconley) (:⚙️)]

The new organization is, I believe, supposed to ship in 56, so I think we only need this for mozilla-beta (55).

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1381944 - Adjust browser_notification_open_settings.js to work with both old and new forms of about:preferences.

Review commit: https://reviewboard.mozilla.org/r/158496/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/158496/

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8887612 [details]
Bug 1381944 - Adjust browser_notification_open_settings.js to work with both old and new forms of about:preferences.

https://reviewboard.mozilla.org/r/158496/#review163736

Comment 4

[Mike Conley (:mconley) (:⚙️)]

Thanks!

This should land directly on mozilla-beta with a=test-only, and skip mozilla-central entirely.

Comment 5

[Mike Conley (:mconley) (:⚙️)]

Having spoken with jaws and RyanVM, landing this on mozilla-central too shouldn't be a big deal, and puts us in a defensive posture in the event that (knock on wood) the new re-org somehow slips 56.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/4de1b6c489fc

I've got it queued to land on inbound too once the trees reopen.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5cd17f7ca31
Adjust browser_notification_open_settings.js to work with both old and new forms of about:preferences. r=jaws

Comment 8

[Ryan VanderMeulen [:RyanVM]]

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