1618549 - [rel=preload] Better granularity for preload/speculative load deduplication

Status: NEW

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

Description

[Honza Bambas (:mayhemer)]

Part of Design documentation for rel=preload

Please see the implementation notes for Better deduplication of requests in HTML5 parser.

We want to make speculative request deduplication in the HTML5 parser be more fine grained. From mere URL to the full "key" as defined in the design document above.

Comment 1

[Honza Bambas (:mayhemer)]

I more tend to move the deduplication checks inside the loaders themselves. If left in the parser we would need to build the (expensive) key for a resource load twice: in the parser and in the loader. Moving this inside loaders would also make the code more clear about where the coalescing happens. Right now the loaders depend on the caller (the HTML5 parser) to do the coalescing job. Loaders count on their respective "Preload*" methods being called NO MORE than once for exactly the same resource. This should change.

Comment 2

[Honza Bambas (:mayhemer)]

Attachment: Bug 1618549 - Add preload coalescing essentials to PreloadService, r=smaug

Comment 3

[Honza Bambas (:mayhemer)]

Attachment: Bug 1618549 - Let ScriptLoader check for duplicate speculative loads using PreloadService, r=smaug

Depends on D73311

Comment 4

[Honza Bambas (:mayhemer)]

Attachment: Bug 1618549 - Let css::Loader check for duplicate speculative loads using PreloadService, r=smaug

Depends on D73312

Comment 5

[Honza Bambas (:mayhemer)]

Attachment: Bug 1618549 - Stop the HTML5 parser from preventing duplicated calls for preload of styles and scripts, r=smaug

Depends on D73313

Comment 6

[Manuel Bucher [:manuel]]

Attachment: Bug 1618549 - Fix fetch and font preloads not getting deduplicated r=#necko