Intermittent dom/security/test/https-only/browser_save_as.js | single tracking bug
Categories
(Core :: DOM: Security, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox-esr115 | --- | unaffected |
firefox115 | --- | unaffected |
firefox116 | --- | unaffected |
firefox117 | --- | affected |
People
(Reporter: intermittent-bug-filer, Assigned: mjurgens)
References
(Regression)
Details
(Keywords: intermittent-failure, regression, test-verify-fail, Whiteboard: [domsecurity-intermittent][domsecurity-active])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Filed by: ctuns [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=422402688&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/eaML6RznRCmqjxR3rWo4xQ/runs/1/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/eaML6RznRCmqjxR3rWo4xQ/runs/1/artifacts/public/logs/live_backing.log&only_show_unexpected=1
[task 2023-07-12T12:03:58.967Z] 12:03:58 INFO - TEST-PASS | dom/security/test/https-only/browser_save_as.js | File should have been downloaded successfully -
[task 2023-07-12T12:03:58.968Z] 12:03:58 INFO - Wait until the menu is closed.
[task 2023-07-12T12:03:58.968Z] 12:03:58 INFO - Open a new tab for testing "Save link as" in context menu.
[task 2023-07-12T12:03:58.969Z] 12:03:58 INFO - Open the context menu.
[task 2023-07-12T12:03:58.969Z] 12:03:58 INFO - Triggering the save process.
[task 2023-07-12T12:03:58.976Z] 12:03:58 INFO - Waiting for the channel.
[task 2023-07-12T12:03:58.976Z] 12:03:58 INFO - Buffered messages finished
[task 2023-07-12T12:03:58.977Z] 12:03:58 INFO - TEST-UNEXPECTED-FAIL | dom/security/test/https-only/browser_save_as.js | URL of download should match - Got "http://127.0.0.1:8888/safebrowsing4-dummy/update&$req=ChUKE25hdmNsaWVudC1hdXRvLWZmb3gaCggFEAIiAiACKAEaCggBEAIiAiACKAEaCggDEAIiAiACKAEaCggHEAIiAiACKAEaCggJEAIiAiACKAE=", expected "https://example.org/"
[task 2023-07-12T12:03:58.978Z] 12:03:58 INFO - Stack trace:
[task 2023-07-12T12:03:58.978Z] 12:03:58 INFO - chrome://mochikit/content/browser-test.js:test_is:1608
[task 2023-07-12T12:03:58.979Z] 12:03:58 INFO - chrome://mochitests/content/browser/dom/security/test/https-only/browser_save_as.js:observer:45
[task 2023-07-12T12:03:58.979Z] 12:03:58 INFO - resource://gre/modules/UrlClassifierListManager.sys.mjs:PROT_ListManager.prototype.makeUpdateRequestForEntry_:642
[task 2023-07-12T12:03:58.980Z] 12:03:58 INFO - resource://gre/modules/UrlClassifierListManager.sys.mjs:PROT_ListManager.prototype.makeUpdateRequest_:608
[task 2023-07-12T12:03:58.981Z] 12:03:58 INFO - resource://gre/modules/UrlClassifierLib.sys.mjs:newfn:48
[task 2023-07-12T12:03:58.981Z] 12:03:58 INFO - Wait until the save is finished.
[task 2023-07-12T12:03:58.982Z] 12:03:58 INFO - MockFilePicker showCallback
[task 2023-07-12T12:03:58.982Z] 12:03:58 INFO - TEST-PASS | dom/security/test/https-only/browser_save_as.js | File should have been downloaded successfully -
Comment 1•1 year ago
|
||
Malte, this test seems to fail on and off due to monitoring all outgoing requests and sometimes catching safebrowsing (probably also telemetry).
Can you rewrite it so that it will keep waiting & skip unexpected URLs instead of failing on them?
Assignee | ||
Comment 2•1 year ago
|
||
Oh yes, I see. On it
Assignee | ||
Comment 3•1 year ago
|
||
This will lead to the test timing out if the expected URL isn't found.
Updated•1 year ago
|
Comment 5•1 year ago
|
||
Assignee | ||
Comment 6•1 year ago
|
||
Could that new error have anything to do with Bug 1841650? The flagged error is happening on the line BrowserTestUtils.removeTab(tab);
, and I see errors like JavaScript Error: "TypeError: can't access property "schemeIs", requestedBrowser.currentURI is undefined" {file: "resource:///modules/AsyncTabSwitcher.sys.mjs" line: 372 }
.
Assignee | ||
Updated•1 year ago
|
Comment 7•1 year ago
|
||
Set release status flags based on info from the regressing bug 1674859
Updated•1 year ago
|
Comment 9•1 year ago
|
||
bugherder |
Description
•