Closed Bug 1503852 Opened 2 years ago Closed 2 years ago
.https .html | HTMLLink Element with rel=prefetch fetches with an empty string Request .destination - expected TIMEOUT when Gecko 65 merges to Beta on 2018-12-03
Bug 1503852 - Split out prefetch test from fetch-destination.https.html, and disable for Firefox. r?jgraham
47 bytes, text/x-phabricator-request
|Details | Review|
[Tracking Requested - why for this release]: Regression from Bug 1501044 Central as beta simulation: https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=testfailed%2Cbusted%2Cexception&revision=7f1e42a7de915ec3547461676e7fabe9751d8431&selectedJob=209135029 Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=209135029&repo=try&lineNumber=6133 Log snippet: [task 2018-11-01T11:49:50.862Z] 11:49:50 INFO - TEST-PASS | /fetch/api/request/destination/fetch-destination.https.html | HTMLLinkElement with rel=preload and as=manifest fetches with a "manifest" Request.destination [task 2018-11-01T11:49:50.862Z] 11:49:50 INFO - TEST-UNEXPECTED-PASS | /fetch/api/request/destination/fetch-destination.https.html | HTMLLinkElement with rel=prefetch fetches with an empty string Request.destination - expected TIMEOUT [task 2018-11-01T11:49:50.863Z] 11:49:50 INFO - TEST-INFO | expected TIMEOUT [task 2018-11-01T11:49:50.864Z] 11:49:50 INFO - TEST-UNEXPECTED-OK | /fetch/api/request/destination/fetch-destination.https.html | expected TIMEOUT [task 2018-11-01T11:49:50.864Z] 11:49:50 INFO - TEST-INFO expected TIMEOUT | took 2049ms
Component: DOM → web-platform-tests
Product: Core → Testing
From the behavior, it ignores the browser.tabs.remote.separatePrivilegedContentProcess set to true in bug 1501044.
Component: web-platform-tests → DOM: Networking
Product: Testing → Core
A more robust solution is probably to disable the preloaded about:newtab for this test. That way, it's far less likely to pass accidentally, instead of timing out (which it should be doing, due to bug 1500089. Beta simulation try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d1d9830ae24a936bbb6e082daadb736cfe3cc7bb mozilla-central try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=15982c4cbdcf0a3f9e82fb80d2eacae1e021e28d
See Also: → 1500089
Cripes, it still fails. :( What would be truly lovely is the ability to just _disable_ this subtest, but (unless jgraham can correct me here), there's no way to disable a subtest, right?
Flags: needinfo?(mconley) → needinfo?(james)
The prefetch test is pretty flake-y, and when it passes, it's only by accident. See bug 1500089 for details.
Try is still not happy with this. :( Apparently, even though I've set this test as disabled, it can still time out? https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=superseded%2Ctestfailed%2Cbusted%2Cexception%2Csuccess%2Cretry%2Cusercancel%2Crunning%2Cpending%2Crunnable&selectedJob=213439682&revision=886fc096da34c06036aadbea407984c6ba7f188f
Yeah, if the test timing out is causing the entire file to time out there isn't a good and easy solution (subtests and the whole file have different statuses). At the moment the only possibilities are 1) Disable the whole file 2) Move the problematic subset out into a seperate test file and disable that subtest Obviously the second one is better since we avoid losing coverage for the other tests, but it's also more work.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/d534baf41277 Split out prefetch test from fetch-destination.https.html, and disable for Firefox. r=jgraham
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/14244 for changes under testing/web-platform/tests
Can't merge web-platform-tests PR due to failing upstream checks: Github PR https://github.com/web-platform-tests/wpt/pull/14244 * Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/YtaP2931RwOwwo-RrMtfoQ)
Verified fixed with today's beta simulations: https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=f3fdf9e4c1c469123991673f5b14a84c8279109b
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.