Open Bug 1715887 Opened 3 years ago Updated 3 years ago

Ensure /fetch/metadata/prefetch.https.sub.html passes with SHIP+BFCache

Categories

(Core :: DOM: Navigation, defect, P3)

defect

Tracking

()

Fission Milestone Future

People

(Reporter: smaug, Unassigned)

References

(Blocks 1 open bug)

Details

Severity: -- → S3
Fission Milestone: --- → M7a
Priority: -- → P2
Depends on: 1716437
Blocks: fission-bfcache
No longer blocks: 1715300
Depends on: 1716894

This is not seen as failing in latest try run https://treeherder.mozilla.org/jobs?repo=try&revision=76f19083ee9a7d7d7f38f568e2292c5ef30b74f1 . Peter is enabling bfcache in parent today (Bug 1715300). We'll confirm and close tomorrow if this doesn't fail anymore.

This doesn't need to be M7a. The test is failing always everywhere, but in different way with bfcache.
It has been disabled.

Fission Milestone: M7a → M8

Fission MVP might be even enough for this, or Future.

Fission Milestone: M8 → MVP
Assignee: nobody → ngogge
Status: NEW → ASSIGNED

This appears to be realted to the cache headers being set by the WPTs in record-header.py here.

I ran the tests with and without fission using:

 MOZ_LOG="nsPrefetch:5" ./mach web-platform-tests testing/web-platform/tests/fetch/metadata/prefetch.https.sub.html

and in both cases the following log can be observed:

 D/nsPrefetch document cannot be reused from cache; canceling prefetch

See here: https://searchfox.org/mozilla-central/source/uriloader/prefetch/nsPrefetchService.cpp#211-219

It appears that there is no issue with fission or the sec-fetch logic here but rather an issue with how we interpret the caching headers in the context of a prefetch.

It appears that there is no issue with fission or the sec-fetch logic here but rather an issue with how we interpret the caching headers in the context of a prefetch.

Based on this comment ^^, this is not a sec-fetch-*problem - un-assigning Niklas ...

Assignee: ngogge → nobody
Status: ASSIGNED → NEW

Moving from Fission MVP to Future backlog.

Fission is annotated as intermittently timing out on these tests:

https://searchfox.org/mozilla-central/rev/73330bf7355c0aef844c41d0d7eed2848e53c82f/testing/web-platform/meta/fetch/metadata/prefetch.https.sub.html.ini#3,6,11,16

We should either figure out why Fission is intermittently timing out or remove the annotations.

Fission Milestone: MVP → Future
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.