If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

RESOLVED FIXED in Firefox 55

Status

()

Firefox
Preferences
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: mconley, Assigned: mconley)

Tracking

unspecified
Firefox 56
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox55 fixed, firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 months ago
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 -
(Assignee)

Comment 1

2 months ago
The new organization is, I believe, supposed to ship in 56, so I think we only need this for mozilla-beta (55).
status-firefox55: --- → affected
status-firefox56: --- → wontfix
Comment hidden (mozreview-request)

Comment 3

2 months ago
mozreview-review
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
Attachment #8887612 - Flags: review?(jaws) → review+
Comment hidden (obsolete)
(Assignee)

Comment 5

2 months ago
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.
status-firefox56: wontfix → affected

Comment 6

2 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/4de1b6c489fc

I've got it queued to land on inbound too once the trees reopen.
status-firefox55: affected → fixed
Flags: in-testsuite+
Keywords: checkin-needed

Comment 7

2 months ago
Pushed by ryanvm@gmail.com:
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

2 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c5cd17f7ca31
Status: NEW → RESOLVED
Last Resolved: 2 months ago
status-firefox56: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
You need to log in before you can comment on or make changes to this bug.