Closed
Bug 1483727
Opened 6 years ago
Closed 6 years ago
Permafailing browser/base/content/test/trackingUI/browser_trackingUI_state.js | CB is disabled after setting the pref - when Gecko 63 merges to Beta on 2018-08-27
Categories
(Firefox :: Site Identity, defect, P1)
Firefox
Site Identity
Tracking
()
VERIFIED
FIXED
Firefox 63
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox61 | --- | unaffected |
firefox62 | --- | unaffected |
firefox63 | + | verified |
People
(Reporter: nataliaCs, Assigned: johannh)
References
Details
Attachments
(1 file)
[Tracking Requested - why for this release]:
Central as Beta:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ae5df0fec8ffc3539b5c97c2aef00ab852f57e1b&group_state=expanded&filter-searchStr=browser-chrome&selectedJob=194182701
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=194182701&repo=try&lineNumber=3733
[task 2018-08-15T22:08:06.429Z] 22:08:06 INFO - TEST-PASS | browser/base/content/test/trackingUI/browser_trackingUI_state.js | blocking not detected label is hidden -
[task 2018-08-15T22:08:06.430Z] 22:08:06 INFO - TEST-PASS | browser/base/content/test/trackingUI/browser_trackingUI_state.js | blocking detected label is visible -
[task 2018-08-15T22:08:06.431Z] 22:08:06 INFO - Buffered messages finished
[task 2018-08-15T22:08:06.433Z] 22:08:06 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/trackingUI/browser_trackingUI_state.js | CB is disabled after setting the pref -
[task 2018-08-15T22:08:06.434Z] 22:08:06 INFO - Stack trace:
[task 2018-08-15T22:08:06.435Z] 22:08:06 INFO - chrome://mochitests/content/browser/browser/base/content/test/trackingUI/browser_trackingUI_state.js:testNormalBrowsing:280
[task 2018-08-15T22:08:06.437Z] 22:08:06 INFO - Testing with Tracking Protection DISABLED.
[task 2018-08-15T22:08:06.438Z] 22:08:06 INFO - Load a test page not containing tracking elements
[task 2018-08-15T22:08:06.442Z] 22:08:06 INFO - Wait tab event: load
[task 2018-08-15T22:08:06.443Z] 22:08:06 INFO - Console message: [JavaScript Error: "TypeError: aWindow.docShell is null" {file: "resource://gre/modules/PrivateBrowsingUtils.jsm" line: 56}]
[task 2018-08-15T22:08:06.444Z] 22:08:06 INFO - Console message: [JavaScript Error: "TypeError: aWindow.docShell is null" {file: "resource://gre/modules/PrivateBrowsingUtils.jsm" line: 56}]
[task 2018-08-15T22:08:06.445Z] 22:08:06 INFO - Console message: [JavaScript Error: "TypeError: aWindow.docShell is null" {file: "resource://gre/modules/PrivateBrowsingUtils.jsm" line: 56}]
[task 2018-08-15T22:08:06.446Z] 22:08:06 INFO - Console message: [JavaScript Warning: "The resource at “http://trackertest.org/” was blocked because tracking protection is enabled." {file: "http://tracking.example.org/browser/browser/base/content/test/trackingUI/trackingPage.html" line: 0}]
[task 2018-08-15T22:08:06.447Z] 22:08:06 INFO - Tab event received: load
[task 2018-08-15T22:08:06.448Z] 22:08:06 INFO - Non-tracking content must not be blocked
[task 2018-08-15T22:08:06.449Z] 22:08:06 INFO - TEST-PASS | browser/base/content/test/trackingUI/browser_trackingUI_state.js | no trackers are detected -
Flags: needinfo?(jhofmann)
Comment 1•6 years ago
|
||
For the record, this is with the patch from bug 1482634 applied.
Assignee | ||
Comment 2•6 years ago
|
||
Ah, right, I see what I missed there...
Assignee: nobody → jhofmann
Status: NEW → ASSIGNED
Flags: needinfo?(jhofmann)
Priority: -- → P1
Reporter | ||
Updated•6 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment 4•6 years ago
|
||
Hi Johann, sorry for the short notice. The first merge day to beta is actually today? Can you take a look at this and bug 1484439, please?
Flags: needinfo?(jhofmann)
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Yup, sorry for that, I thought I had time until final merge...
Flags: needinfo?(jhofmann)
Comment 7•6 years ago
|
||
Comment on attachment 9003635 [details]
Bug 1483727 - Correctly test the TP UI state when CB UI is off. r=francois
François Marier [:francois] has approved the revision.
Attachment #9003635 -
Flags: review+
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2622040931e6
Correctly test the TP UI state when CB UI is off. r=francois
Comment 9•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Comment 10•6 years ago
|
||
Verified fixed in today's beta simulation: https://treeherder.mozilla.org/#/jobs?repo=try&revision=360732b72078525287a759529c1df1ef2e6b277b&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=usercancel&filter-resultStatus=runnable
Status: RESOLVED → VERIFIED
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•