1618292 - [rel=preload] Update `ScriptLoader` to use `PreloaderBase` and `Document.PreloadService`

Status: RESOLVED FIXED

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

Description

[Honza Bambas (:mayhemer)]

Part of Design documentation for rel=preload

Please follow this “script” type design.

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

Note that script 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 script loader itself.

Comment 1

[Honza Bambas (:mayhemer)]

Stealing from the DOM team. I want to verify the API and have an example implementation other can refer to.

Comment 2

[Honza Bambas (:mayhemer)]

Attachment: Bug 1618292 - Make ScriptLoadRequest derive and use PreloaderBase to support new preload as speculative load feature, r=smaug

Depends on D67480

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7541ff8e630
Make ScriptLoadRequest derive and use PreloaderBase to support new preload as speculative load feature, r=smaug

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/a7541ff8e630