Change LinkPreload from a resource type to a separate flag |
|||||
Issue descriptionWe need to pipe a separate indication that the resource in question is a link preload (and kill the LinkPreload resource type while we're at it). Currently the `forPreload` flag is confusing link preload and preloadScanner based preloads. We need to split that up. From https://codereview.chromium.org/2219663005/#msg9. This is needed in particular to be able to tag any resource with the LOAD_PREFETCH network flag, regardless of its resource type.
,
Aug 16 2016
,
Aug 16 2016
,
Aug 16 2016
,
Aug 16 2016
As I read the code more carefully, it seems that LinkPrefetch is controlling the LOAD_PREFETCH flag, not LinkPreload. Maybe both should be removed?
,
Aug 16 2016
I started a CL: https://codereview.chromium.org/2249913003/ This is not ready for review at all: it merely compiles, but I don't know if it works yet.
,
Aug 16 2016
LinkPrefetch is the resource type for all <link rel=prefetch> resources. In that case, it makes sense to have a separate resource type, as there's a lot of behavior and priorities that are related to these fetches.
,
Aug 18 2016
Closing this bug, since it's conflating several issues (LinkPrefetch and LinkPreload) and is prone to confusion. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by droger@chromium.org
, Aug 16 2016