1576832 - Intermittent [TV] browser/base/content/test/siteProtections/browser_protections_UI.js | The tracking protection icon state has been changed to disabled. -

Status: RESOLVED FIXED

(Firefox :: Protections UI, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: aiakab [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=263643952&repo=autoland
Full log: https://queue.taskcluster.net/v1/task/POqIxdRMTvqHrKOhHVTq1A/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/POqIxdRMTvqHrKOhHVTq1A/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1

---

[task 2019-08-27T06:26:49.502Z] 06:26:49 INFO - Leaving test bound testNumberOfBlockedTrackers
[task 2019-08-27T06:26:49.502Z] 06:26:49 INFO - Entering test bound testSubViewTelemetry
[task 2019-08-27T06:26:49.503Z] 06:26:49 INFO - Buffered messages logged at 06:26:37
[task 2019-08-27T06:26:49.506Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded trackers telemetry event -
[task 2019-08-27T06:26:49.507Z] 06:26:49 INFO - Buffered messages logged at 06:26:38
[task 2019-08-27T06:26:49.508Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded subview_settings telemetry event -
[task 2019-08-27T06:26:49.508Z] 06:26:49 INFO - Buffered messages logged at 06:26:39
[task 2019-08-27T06:26:49.509Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded social telemetry event -
[task 2019-08-27T06:26:49.509Z] 06:26:49 INFO - Buffered messages logged at 06:26:40
[task 2019-08-27T06:26:49.509Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded subview_settings telemetry event -
[task 2019-08-27T06:26:49.510Z] 06:26:49 INFO - Buffered messages logged at 06:26:41
[task 2019-08-27T06:26:49.513Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded cookies telemetry event -
[task 2019-08-27T06:26:49.520Z] 06:26:49 INFO - Buffered messages logged at 06:26:43
[task 2019-08-27T06:26:49.520Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded subview_settings telemetry event -
[task 2019-08-27T06:26:49.521Z] 06:26:49 INFO - Buffered messages logged at 06:26:44
[task 2019-08-27T06:26:49.521Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded cryptominers telemetry event -
[task 2019-08-27T06:26:49.524Z] 06:26:49 INFO - Buffered messages logged at 06:26:45
[task 2019-08-27T06:26:49.524Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded subview_settings telemetry event -
[task 2019-08-27T06:26:49.525Z] 06:26:49 INFO - Buffered messages logged at 06:26:46
[task 2019-08-27T06:26:49.528Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded fingerprinters telemetry event -
[task 2019-08-27T06:26:49.531Z] 06:26:49 INFO - Buffered messages logged at 06:26:47
[task 2019-08-27T06:26:49.532Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | recorded subview_settings telemetry event -
[task 2019-08-27T06:26:49.532Z] 06:26:49 INFO - Leaving test bound testSubViewTelemetry
[task 2019-08-27T06:26:49.532Z] 06:26:49 INFO - Entering test bound testQuickSwitchTabAfterTogglingTPSwitch
[task 2019-08-27T06:26:49.533Z] 06:26:49 INFO - Buffered messages logged at 06:26:48
[task 2019-08-27T06:26:49.533Z] 06:26:49 INFO - TEST-PASS | browser/base/content/test/siteProtections/browser_protections_UI.js | The tracking protection icon state is still enabled. -
[task 2019-08-27T06:26:49.534Z] 06:26:49 INFO - Buffered messages finished
[task 2019-08-27T06:26:49.534Z] 06:26:49 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/siteProtections/browser_protections_UI.js | The tracking protection icon state has been changed to disabled. -
[task 2019-08-27T06:26:49.534Z] 06:26:49 INFO - Stack trace:
[task 2019-08-27T06:26:49.535Z] 06:26:49 INFO - chrome://mochikit/content/browser-test.js:test_ok:1580
[task 2019-08-27T06:26:49.535Z] 06:26:49 INFO - chrome://mochitests/content/browser/browser/base/content/test/siteProtections/browser_protections_UI.js:testQuickSwitchTabAfterTogglingTPSwitch:614
[task 2019-08-27T06:26:49.536Z] 06:26:49 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1350
[task 2019-08-27T06:26:49.536Z] 06:26:49 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1385
[task 2019-08-27T06:26:49.536Z] 06:26:49 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1207
[task 2019-08-27T06:26:49.537Z] 06:26:49 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:805
[task 2019-08-27T06:26:49.537Z] 06:26:49 INFO - Leaving test bound testQuickSwitchTabAfterTogglingTPSwitch
[task 2019-08-27T06:26:49.538Z] 06:26:49 INFO - GECKO(1085) | JavaScript error: resource://testing-common/PromiseTestUtils.jsm, line 112: uncaught exception: Object
[task 2019-08-27T06:26:49.538Z] 06:26:49 INFO - Console message: [JavaScript Error: "uncaught exception: Object" {file: "resource://testing-common/PromiseTestUtils.jsm" line: 112}]
[task 2019-08-27T06:26:49.539Z] 06:26:49 INFO - GECKO(1085) | MEMORY STAT | vsize 3092MB | residentFast 489MB | heapAllocated 204MB
[task 2019-08-27T06:26:49.539Z] 06:26:49 INFO - TEST-OK | browser/base/content/test/siteProtections/browser_protections_UI.js | took 22860ms

Comment 1

[Arthur Iakab [arthur_iakab]]

These failures started from this push
https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=263643952&resultStatus=testfailed%2Cbusted%2Cexception&revision=d3025ffd55d1038d74ca9979a039d5669f6931ad&searchStr=tv

Nihanth Subramanya can you please take a look?

Comment 2

[(inactive) Nihanth Subramanya [:nhnt11]]

The failure is a bit misleading, because the assertion's [1] info message is wrong (I think it was copied over from another one and wasn't updated properly, and I missed it in review). I believe the issue is that we assume that selectedBrowser gives us the correct browser reference after we set selectedTab. But the issue is that setting selectedTab only sets the tab, and gBrowser.selectedBrowser is updated via an event callback [2] after a select event on the tabpanels element. This means it's async and therefore we cannot rely on it to work immediately after setting selectedTab. One solution is to simply pass tabTwo.linkedBrowser to ContentBlockingAllowList.includes() - I prefer this. The other solution is to spin the event loop before accessing selectedBrowser.

Tim, want to take this follow-up? We should also update the assertion's info message while we're here. Thanks!

I took the liberty of changing the regressed-by field to bug 1575788.

[1] https://searchfox.org/mozilla-central/rev/325c1a707819602feff736f129cb36055ba6d94f/browser/base/content/test/siteProtections/browser_protections_UI.js#613
[2] https://searchfox.org/mozilla-central/rev/325c1a707819602feff736f129cb36055ba6d94f/browser/base/content/tabbrowser.js#5107

Comment 3

[Tim Huang[:timhuang]]

Attachment: Bug 1576832 - Fix the info and check of the ETP state to use the right browser object for the test 'browser_protections_UI.js' r?nhnt11!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2187d722dc38
Fix the info and check of the ETP state to use the right browser object for the test 'browser_protections_UI.js' r=nhnt11

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/2187d722dc38

Comment 6

[Intermittent Failures Robot]

8 failures in 5047 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 8

Platform breakdown:

• linux64: 1
• macosx1014-64: 1
• windows7-32: 1
• windows10-64: 1
• macosx1014-64-shippable: 1
• linux64-shippable: 1
• windows10-64-shippable: 1
• windows7-32-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1576832&startday=2019-08-26&endday=2019-09-01&tree=all