Closed
Bug 1618543
Opened 5 years ago
Closed 4 years ago
[rel=preload] Let `fetch()` use "fetch" preloads
Categories
(Core :: DOM: Networking, enhancement, P1)
Core
DOM: Networking
Tracking
()
RESOLVED
FIXED
mozilla78
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: mayhemer, Assigned: mayhemer)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
Part of Design documentation for rel=preload
There are two main points to fulfill:
- At the place
fetch
opens its HTTP channel, look for an existing preload in Document.PreloadService using the key as defined in the design doc above. The "as" part of the key shall be "fetch". The exact content for the attributes part of the key is yet to be determined for fetch. If a prelaod is found in the PreloadService, then instead of opening a new channel, use the AsyncConsume method of the preload to get the data. If not, or if AsyncConsume fails, open a new HTTP channel as we do now. FetchDriver has to implement PreloaderBase and after it opens its HTTP channel, register itself in Document.PreloadService so that later added <link preload> tags get the onload/onerror notifications and don't open a second channel.
This goes against the finding on how Chrome behaves. A used fetch preload is removed from the preload hash table, following preloads create a new network request.
Assignee | ||
Updated•5 years ago
|
Component: DOM: Core & HTML → DOM: Networking
Assignee | ||
Updated•5 years ago
|
Priority: -- → P2
Whiteboard: [necko-triaged]
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Priority: P2 → P1
Assignee | ||
Comment 1•4 years ago
|
||
Pushed by honzab.moz@firemni.cz:
https://hg.mozilla.org/integration/autoland/rev/4068b1f7a903
Let `fetch()` use "fetch" preloads, r=baku
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox78:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in
before you can comment on or make changes to this bug.
Description
•