Closed Bug 1843036 Opened 1 year ago Closed 1 year ago

Intermittent dom/security/test/https-only/browser_save_as.js | single tracking bug

Categories

(Core :: DOM: Security, defect, P5)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1841650
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)

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 - 

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: nobody → mjurgens
Flags: needinfo?(mjurgens)
Whiteboard: [domsecurity-intermittent] → [domsecurity-intermittent][domsecurity-active]

Oh yes, I see. On it

Status: NEW → ASSIGNED
Flags: needinfo?(mjurgens)

This will lead to the test timing out if the expected URL isn't found.

Pushed by fbraun@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/08dc62129428 Ignore unexpected URLs instead of failing when monitoring traffic in `dom/security/test/https-only/browser_save_as.js` r=freddyb
Summary: Intermittent TV dom/security/test/https-only/browser_save_as.js | single tracking bug → Intermittent dom/security/test/https-only/browser_save_as.js | single tracking bug

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 }.

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Duplicate of bug: 1841650
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: