[Webkit-unassigned] [Bug 156334] Initial Link preload support

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=156334

--- Comment #24 from Yoav Weiss <yoav at yoav.ws> ---
(In reply to comment #20)
> We should consider putting this behind a runtime flag that is enabled for
> Safari Technology Preview and WebKit nightlies until there is more conscious
> around the feature. Especially in regards to the similar, but not mandatory,
> <link rel=prefetch>.

I have a runtime flag which prevents LinkRelAttribute::isLinkPreload from being true if the flag is false. That should be enough to turn off the feature.

I would have actually preferred that the flag would be false by default for WebKit nightly/Safari Technology Preview until I'd add support for the onload event (for Web compatibility reasons: things like http://filamentgroup.github.io/loadCSS/test/preload.html rely on the load event).

Is it possible to turn the flag on only for layout tests, but to keep it false for everything else?

Otherwise, I could add onload support to this patch, but it may increase its size.

> 
> Also we should do as was mentioned in the webkit-dev thread and log a
> warning to the console when a preloaded resource is not used (likely after a
> short timeout from the load event or something).

Sure. Should I add that mechanism as part of this patch? Or could it wait for a followup patch?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160407/afaff581/attachment.html>