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)
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
Comment 1•10 months ago
|
||
:mkaply, since you are the author of the regressor, bug 1973827, could you take a look?
For more information, please visit BugBot documentation.
Comment 2•10 months ago
|
||
Set release status flags based on info from the regressing bug 1973827
Comment 3•10 months ago
|
||
I've verified that backing my patch out doesn't fix this.
I'm not sure what actually caused it....
Comment 4•10 months ago
|
||
I thought maybe my other tracking protection patch was related, but it's not.
I'm at a loss.
Comment 5•10 months ago
|
||
All tests in browser/components/enterprisepolicies/tests/browser/ are showing this.
Comment 6•10 months ago
|
||
I'm guessing it was actually caused by this:
Updated•10 months ago
|
Comment 7•10 months ago
|
||
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
Comment 8•10 months ago
|
||
These prefs were recently added (bug 1974449) and one pref was already added to the ignore list in that bug.
| Comment hidden (Intermittent Failures Robot) |
Comment 10•9 months ago
|
||
(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.
Comment 11•9 months ago
|
||
Set release status flags based on info from the regressing bug 1816549
| Comment hidden (Intermittent Failures Robot) |
Description
•