Closed Bug 1618293 Opened 4 years ago Closed 4 years ago

[rel=preload] Update `css::Loader` to use `PreloaderBase` and `Document.PreloadService`

Categories

(Core :: DOM: Core & HTML, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Part of Design documentation for rel=preload

Please follow this “style” type design.

This depends on the PreloaderBase class being implemented. The idea is to let the style load handler derive from it and let the loader register it in Document.PreloadService.

Note that style loader will never be called more than once to start a speculative load, neither preload or regular, so there is no need to add any more coalescing than we already have in the HTML5 parser (the caller to the loader) and the style loader itself.

Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Depends on: 1625770
Regressions: 1635417
Pushed by honzab.moz@firemni.cz:
https://hg.mozilla.org/integration/autoland/rev/2b5acb03fcf0
Make css::Loader derive and use PreloaderBase to support new preload as speculative load feature, r=smaug
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: