<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Initial Link preload support"
href="https://bugs.webkit.org/show_bug.cgi?id=156334#c24">Comment # 24</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Initial Link preload support"
href="https://bugs.webkit.org/show_bug.cgi?id=156334">bug 156334</a>
from <span class="vcard"><a class="email" href="mailto:yoav@yoav.ws" title="Yoav Weiss <yoav@yoav.ws>"> <span class="fn">Yoav Weiss</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=156334#c20">comment #20</a>)
<span class="quote">> 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>.</span >
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 <a href="http://filamentgroup.github.io/loadCSS/test/preload.html">http://filamentgroup.github.io/loadCSS/test/preload.html</a> 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.
<span class="quote">>
> 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).</span >
Sure. Should I add that mechanism as part of this patch? Or could it wait for a followup patch?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>