Open Bug 1975547 Opened 10 months ago Updated 7 days ago

Intermittent [TV] chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: browser.policies.alternatePath

Categories

(Firefox :: Enterprise Policies, defect, P5)

defect

Tracking

()

Tracking Status
firefox-esr128 --- unaffected
firefox-esr140 --- unaffected
firefox140 --- unaffected
firefox141 --- unaffected
firefox142 --- affected
firefox143 --- affected

People

(Reporter: intermittent-bug-filer, Unassigned)

References

(Regression)

Details

(Keywords: intermittent-failure, regression, test-verify-fail)

Filed by: smolnar [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=516230441&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/CzznFiypQwaG6uaVnRLGDg/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://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/CzznFiypQwaG6uaVnRLGDg/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


TEST-PASS | browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | screen checkbox should be disabled - 
[task 2025-07-03T19:00:01.227+00:00] 19:00:01     INFO - Buffered messages logged at 19:00:00
[task 2025-07-03T19:00:01.228+00:00] 19:00:01     INFO - Leaving test bound test_pageinfo_permissions
[task 2025-07-03T19:00:01.228+00:00] 19:00:01     INFO - TEST-PASS | browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | Engine is inactive at the end of the test - 
[task 2025-07-03T19:00:01.229+00:00] 19:00:01     INFO - Buffered messages finished
[task 2025-07-03T19:00:01.229+00:00] 19:00:01    ERROR - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: privacy.trackingprotection.allow_list.baseline.enabled
[task 2025-07-03T19:00:01.230+00:00] 19:00:01    ERROR - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: browser.policies.alternatePath
[task 2025-07-03T19:00:01.231+00:00] 19:00:01    ERROR - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: privacy.trackingprotection.allow_list.convenience.enabled
[task 2025-07-03T19:00:01.231+00:00] 19:00:01     INFO - GECKO(1022) | MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
[task 2025-07-03T19:00:01.232+00:00] 19:00:01     INFO - GECKO(1022) | MEMORY STAT heapAllocated not supported in this build configuration.
[task 2025-07-03T19:00:01.232+00:00] 19:00:01     INFO - GECKO(1022) | MEMORY STAT | vsize 120599095MB | residentFast 1345MB
[task 2025-07-03T19:00:01.233+00:00] 19:00:01     INFO - TEST-OK | browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | took 8002ms

:mkaply, since you are the author of the regressor, bug 1973827, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(mozilla)

Set release status flags based on info from the regressing bug 1973827

I've verified that backing my patch out doesn't fix this.

I'm not sure what actually caused it....

Flags: needinfo?(mozilla)

I thought maybe my other tracking protection patch was related, but it's not.

I'm at a loss.

All tests in browser/components/enterprisepolicies/tests/browser/ are showing this.

I'm guessing it was actually caused by this:

https://bugzilla.mozilla.org/show_bug.cgi?id=1816549

Regressed by: 1816549
No longer regressed by: 1973827

Here's a profile of running browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js locally with the "Preference Write" marker modified to capture stacks: https://share.firefox.dev/44iJiU3

The stack for setting these preferences is:

promise callback []
AsyncFunctionNext [self-hosted:800:27]
InterpretGeneratorResume [self-hosted:1332:34]
test_pageinfo_permissions [chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js:8:50]
withNewTab [resource://testing-common/BrowserTestUtils.sys.mjs:110:19]
openNewForegroundTab [resource://testing-common/BrowserTestUtils.sys.mjs:166:23]
switchTab [resource://testing-common/BrowserTestUtils.sys.mjs:357:12]
openNewForegroundTab/promises< [resource://testing-common/BrowserTestUtils.sys.mjs:214:57]
addTab [resource://testing-common/BrowserTestUtils.sys.mjs:2684:9]
addTab [chrome://browser/content/tabbrowser/tabbrowser.js:2668:11]
_kickOffBrowserLoad [chrome://browser/content/tabbrowser/tabbrowser.js:3457:24]
fixupAndLoadURIString [chrome://browser/content/tabbrowser/tabbrowser.js:8657:26]
_internalMaybeFixupLoadURI [chrome://browser/content/tabbrowser/tabbrowser.js:8666:31]
fixupAndLoadURIString [resource://gre/modules/RemoteWebNavigation.sys.mjs:150:24]
CanonicalBrowsingContext.fixupAndLoadURIString []
XPCWrappedJS method call []
registerAndRunExceptionListObserver [resource://gre/modules/UrlClassifierExceptionListService.sys.mjs:281:38]
lazyInit [resource://gre/modules/UrlClassifierExceptionListService.sys.mjs:190:17]
maybeMigrateCategoryPrefs [resource://gre/modules/UrlClassifierExceptionListService.sys.mjs:234:28]
nsIPrefBranch.setBoolPref []

So the code setting these prefs is https://searchfox.org/mozilla-central/rev/fca753067e53a53b0b8cd71e0c9bdc1d777a40ae/netwerk/url-classifier/UrlClassifierExceptionListService.sys.mjs#244-266

Because this code is unrelated to what the test actually does, I don't think changing the tests to reset the prefs in a cleanup function is useful, so I think we should just add these preferences to the ignore list. I'm doing it in https://phabricator.services.mozilla.com/D256280

These prefs were recently added (bug 1974449) and one pref was already added to the ignore list in that bug.

(In reply to Florian Quèze [:florian] from comment #7)

Because this code is unrelated to what the test actually does, I don't think changing the tests to reset the prefs in a cleanup function is useful, so I think we should just add these preferences to the ignore list. I'm doing it in https://phabricator.services.mozilla.com/D256280

The tracking protection preferences were added to the ignore list in https://phabricator.services.mozilla.com/D256280 (bug 1975980). browser.policies.alternatePath was added to the ignore list too because it was modified by many tests, but it may be worth checking if the test code could be changed to no longer leave a modified value in this pref.

Summary: Intermittent [TV] chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: privacy.trackingprotection.allow_list.baseline.enabled → Intermittent [TV] chrome://mochitests/content/browser/browser/components/enterprisepolicies/tests/browser/browser_policy_pageinfo_permissions.js | changed preference: browser.policies.alternatePath

Set release status flags based on info from the regressing bug 1816549

Whiteboard: [stockwell disable-recommended]
You need to log in before you can comment on or make changes to this bug.