<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[210914] trunk</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/210914">210914</a></dd>
<dt>Author</dt> <dd>yoav@yoav.ws</dd>
<dt>Date</dt> <dd>2017-01-18 23:13:54 -0800 (Wed, 18 Jan 2017)</dd>
</dl>
<h3>Log Message</h3>
<pre>Add Link header support for preload.
https://bugs.webkit.org/show_bug.cgi?id=165521
Reviewed by Alex Christensen.
Source/WebCore:
This patch adds Link header support for preload, by adding a Link
header parser, and calling it from FrameLoader::receivedFirstData.
Test: http/tests/preload/download_resources_from_header.php
* CMakeLists.txt: Add LinkHeader files.
* WebCore.xcodeproj/project.pbxproj: Add LinkHeader files.
* loader/FrameLoader.cpp:
(WebCore::receivedFirstData): Trigger Link based preloading by calling loadLinksFromHeader.
* loader/LinkHeader.cpp: Added.
(WebCore::isWhitespace):
(WebCore::isValidURLChar):
(WebCore::isValidParameterNameChar):
(WebCore::isValidParameterValueEnd):
(WebCore::isValidParameterValueChar):
(WebCore::isExtensionParameter):
(WebCore::parseURL):
(WebCore::invalidParameterDelimiter):
(WebCore::validFieldEnd):
(WebCore::parseParameterDelimiter):
(WebCore::paramterNameFromString):
(WebCore::parseParameterName):
(WebCore::skipQuotesIfNeeded):
(WebCore::parseParameterValue):
(WebCore::LinkHeader::setValue):
(WebCore::findNextHeader):
(WebCore::LinkHeader::LinkHeader):
(WebCore::LinkHeaderSet::LinkHeaderSet):
(WebCore::LinkHeaderSet::init):
* loader/LinkHeader.h: Added.
(WebCore::LinkHeader::url):
(WebCore::LinkHeader::rel):
(WebCore::LinkHeader::as):
(WebCore::LinkHeader::mimeType):
(WebCore::LinkHeader::media):
(WebCore::LinkHeader::crossOrigin):
(WebCore::LinkHeader::valid):
(WebCore::LinkHeaderSet::begin):
(WebCore::LinkHeaderSet::end):
(WebCore::LinkHeaderSet::operator[]):
(WebCore::LinkHeaderSet::size):
* loader/LinkLoader.cpp:
(WebCore::LinkLoader::loadLinksFromHeader): Loads link preload directives that arrive from Link headers.
(WebCore::LinkLoader::preloadIfNeeded): Add return value.
* loader/LinkLoader.h:
* platform/network/HTTPHeaderNames.in: Add "Link" as a header name.
LayoutTests:
* http/tests/preload/download_resources_from_header_iframe-expected.txt: Added
* http/tests/preload/download_resources_from_header_iframe.html: Added
* http/tests/preload/download_resources_from_invalid_headers-expected: Added
* http/tests/preload/download_resources_from_invalid_headers.html: Added
* http/tests/preload/resources/download_resources_from_header.php: Added.
* http/tests/preload/resources/invalid_resources_from_header.php: Added.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreCMakeListstxt">trunk/Source/WebCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreWebCorexcodeprojprojectpbxproj">trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebCoreloaderFrameLoadercpp">trunk/Source/WebCore/loader/FrameLoader.cpp</a></li>
<li><a href="#trunkSourceWebCoreloaderLinkLoadercpp">trunk/Source/WebCore/loader/LinkLoader.cpp</a></li>
<li><a href="#trunkSourceWebCoreloaderLinkLoaderh">trunk/Source/WebCore/loader/LinkLoader.h</a></li>
<li><a href="#trunkSourceWebCoreplatformnetworkHTTPHeaderNamesin">trunk/Source/WebCore/platform/network/HTTPHeaderNames.in</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkLayoutTestshttptestspreloaddownload_resources_from_header_iframeexpectedtxt">trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe-expected.txt</a></li>
<li><a href="#trunkLayoutTestshttptestspreloaddownload_resources_from_header_iframehtml">trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe.html</a></li>
<li><a href="#trunkLayoutTestshttptestspreloaddownload_resources_from_invalid_headersexpectedtxt">trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers-expected.txt</a></li>
<li><a href="#trunkLayoutTestshttptestspreloaddownload_resources_from_invalid_headershtml">trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers.html</a></li>
<li><a href="#trunkLayoutTestshttptestspreloadresourcesdownload_resources_from_headerphp">trunk/LayoutTests/http/tests/preload/resources/download_resources_from_header.php</a></li>
<li><a href="#trunkLayoutTestshttptestspreloadresourcesinvalid_resources_from_headerphp">trunk/LayoutTests/http/tests/preload/resources/invalid_resources_from_header.php</a></li>
<li><a href="#trunkSourceWebCoreloaderLinkHeadercpp">trunk/Source/WebCore/loader/LinkHeader.cpp</a></li>
<li><a href="#trunkSourceWebCoreloaderLinkHeaderh">trunk/Source/WebCore/loader/LinkHeader.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/LayoutTests/ChangeLog 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -1,3 +1,17 @@
</span><ins>+2017-01-18 Yoav Weiss <yoav@yoav.ws>
+
+ Add Link header support for preload.
+ https://bugs.webkit.org/show_bug.cgi?id=165521
+
+ Reviewed by Alex Christensen.
+
+ * http/tests/preload/download_resources_from_header_iframe-expected.txt: Added
+ * http/tests/preload/download_resources_from_header_iframe.html: Added
+ * http/tests/preload/download_resources_from_invalid_headers-expected: Added
+ * http/tests/preload/download_resources_from_invalid_headers.html: Added
+ * http/tests/preload/resources/download_resources_from_header.php: Added.
+ * http/tests/preload/resources/invalid_resources_from_header.php: Added.
+
</ins><span class="cx"> 2017-01-18 Joseph Pecoraro <pecoraro@apple.com>
</span><span class="cx">
</span><span class="cx"> Web Inspector: console.table only works for the first 5 properties
</span></span></pre></div>
<a id="trunkLayoutTestshttptestspreloaddownload_resources_from_header_iframeexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe-expected.txt (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe-expected.txt (rev 0)
+++ trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe-expected.txt 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,18 @@
</span><ins>+CONSOLE MESSAGE: <link rel=preload> must have a valid `as` value
+
+
+--------
+Frame: '<!--framePath //<!--frame0-->-->'
+--------
+PASS internals.isPreloaded('../resources/dummy.js'); is true
+PASS internals.isPreloaded('../resources/dummy.css'); is true
+PASS internals.isPreloaded('../resources/square.png'); is true
+PASS internals.isPreloaded('../resources/Ahem.ttf'); is true
+PASS internals.isPreloaded('../resources/test.mp4'); is true
+PASS internals.isPreloaded('../security/resources/captions.vtt'); is true
+PASS internals.isPreloaded('../resources/dummy.xml?badvalue'); is false
+PASS internals.isPreloaded('../resources/dummy.xml'); is true
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestshttptestspreloaddownload_resources_from_header_iframehtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe.html (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe.html (rev 0)
+++ trunk/LayoutTests/http/tests/preload/download_resources_from_header_iframe.html 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,11 @@
</span><ins>+<!DOCTYPE html>
+<script>
+ if (window.internals)
+ window.internals.setLinkPreloadSupport(true);
+ if (window.testRunner) {
+ testRunner.dumpAsText();
+ testRunner.dumpChildFramesAsText();
+ }
+</script>
+<iframe src="resources/download_resources_from_header.php">
+
</ins></span></pre></div>
<a id="trunkLayoutTestshttptestspreloaddownload_resources_from_invalid_headersexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers-expected.txt (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers-expected.txt (rev 0)
+++ trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers-expected.txt 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,25 @@
</span><ins>+CONSOLE MESSAGE: Refused to load http://localhost:8000/preload/resources/square.png because it does not appear in the img-src directive of the Content Security Policy.
+
+
+--------
+Frame: '<!--framePath //<!--frame0-->-->'
+--------
+PASS internals.isPreloaded('../resources/dummy.js'); is true
+PASS internals.isPreloaded('../מש×בי×/dummy.css'); is false
+PASS internals.isPreloaded('../résôûrcès/square.png'); is false
+PASS internals.isPreloaded('../resources/Ahem{.ttf,.woff}'); is true
+PASS internals.isPreloaded('../resources/test.mp4'); is false
+PASS internals.isPreloaded('../resources/test .mp4'); is true
+PASS internals.isPreloaded('../security/resources/cap tions.vtt'); is true
+PASS internals.isPreloaded('../resources/dummy.xml?badvalue'); is false
+PASS internals.isPreloaded('../resources/dummy .xml'); is true
+PASS internals.isPreloaded('../resources/dummy.xml'); is false
+PASS internals.isPreloaded('../resources/dumm'); is false
+PASS internals.isPreloaded('http://localhost:8000/preload/resources/square.png'); is false
+PASS internals.isPreloaded('http://localhost:53/preload/resources/dummy.js'); is true
+PASS internals.isPreloaded('#foobar'); is false
+PASS internals.isPreloaded('../resources/Ahem.ttf'); is false
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestshttptestspreloaddownload_resources_from_invalid_headershtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers.html (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers.html (rev 0)
+++ trunk/LayoutTests/http/tests/preload/download_resources_from_invalid_headers.html 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,11 @@
</span><ins>+<!DOCTYPE html>
+<script>
+ if (window.internals)
+ window.internals.setLinkPreloadSupport(true);
+ if (window.testRunner) {
+ testRunner.dumpAsText();
+ testRunner.dumpChildFramesAsText();
+ }
+</script>
+<iframe src="resources/invalid_resources_from_header.php">
+
</ins></span></pre></div>
<a id="trunkLayoutTestshttptestspreloadresourcesdownload_resources_from_headerphp"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/resources/download_resources_from_header.php (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/resources/download_resources_from_header.php (rev 0)
+++ trunk/LayoutTests/http/tests/preload/resources/download_resources_from_header.php 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,26 @@
</span><ins>+<?php
+header("Link: <../resources/dummy.js>; rel=preload; as=script", false);
+header("Link:<../resources/dummy.css>; rel=preload; as=style", false);
+header("Link: <../resources/square.png>;rel=preload;as=image", false);
+header("Link: <../resources/Ahem.ttf>; rel=preload; as=font; crossorigin", false);
+header("Link: <../resources/test.mp4>; rel=preload; as=media", false);
+header("link: <../security/resources/captions.vtt>; rel=preload; as=track", false);
+header("Link: <../resources/dummy.xml?badvalue>; rel=preload; as=foobar", false);
+header("Link: <../resources/dummy.xml>; rel=preload", false);
+?>
+<!DOCTYPE html>
+<script src="/js-test-resources/js-test.js"></script>
+<script>
+ shouldBeTrue("internals.isPreloaded('../resources/dummy.js');");
+ shouldBeTrue("internals.isPreloaded('../resources/dummy.css');");
+ shouldBeTrue("internals.isPreloaded('../resources/square.png');");
+ shouldBeTrue("internals.isPreloaded('../resources/Ahem.ttf');");
+ shouldBeTrue("internals.isPreloaded('../resources/test.mp4');");
+ shouldBeTrue("internals.isPreloaded('../security/resources/captions.vtt');");
+ shouldBeFalse("internals.isPreloaded('../resources/dummy.xml?badvalue');");
+ shouldBeTrue("internals.isPreloaded('../resources/dummy.xml');");
+</script>
+<script>
+ if (window.internals)
+ window.internals.setLinkPreloadSupport(false);
+</script>
</ins></span></pre></div>
<a id="trunkLayoutTestshttptestspreloadresourcesinvalid_resources_from_headerphp"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/http/tests/preload/resources/invalid_resources_from_header.php (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/preload/resources/invalid_resources_from_header.php (rev 0)
+++ trunk/LayoutTests/http/tests/preload/resources/invalid_resources_from_header.php 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,46 @@
</span><ins>+<?php
+header("Link: < ../resources/dummy.js >; rel=preload; as=script", false);
+header("Link: <../משאבים/dummy.css>; rel=preload; as=style", false);
+header("Link: <../résôûrcès/dummy.css>; rel=preload; as=style", false);
+header("Link: <../resources/Ahem{.ttf,.woff}>; rel=preload; as=font; crossorigin", false);
+header("Link: <../resources/test\f.mp4>; rel=preload; as=media", false);
+header("Link: <../security/resources/cap\ttions.vtt>; rel=preload; as=track", false);
+header("Link: <../resources/dummy .xml>; rel=preload;", false);
+header("Link: <../resources/dumm>y.xml>; rel=preload", false);
+header("Content-Security-Policy: img-src 'none'", false);
+header("Link: <http://localhost:8000/preload/resources/square.png>; rel=preload; as=image", false);
+header("Link: <http://localhost:53/preload/resources/dummy.js>; rel=preload; as=script", false);
+header("Link: <#foobar>; rel=preload; as=style", false);
+header("Link: <>; rel=preload; as=style", false);
+header("Link: < \t>; rel=preload; as=style", false);
+header("Link: >; rel=preload; as=style", false);
+header("Link: <; rel=preload; as=style", false);
+header("Link: ; rel=preload; as=style", false);
+header("Link <../resources/Ahem.ttf>; rel=preload; as=font; crossorigin", false);
+header("Link: < ../resources/dummy.js?foobar >; rel=preload; as='", false);
+?>
+<!DOCTYPE html>
+<script src="/js-test-resources/js-test.js"></script>
+<script>
+ shouldBeTrue("internals.isPreloaded('../resources/dummy.js');");
+ shouldBeFalse("internals.isPreloaded('../משאבים/dummy.css');");
+ shouldBeFalse("internals.isPreloaded('../résôûrcès/square.png');");
+ // Invalid URLs get preloaded (and get terminated further down the stack)
+ shouldBeTrue("internals.isPreloaded('../resources/Ahem{.ttf,.woff}');");
+ shouldBeFalse("internals.isPreloaded('../resources/test.mp4');");
+ shouldBeTrue("internals.isPreloaded('../resources/test\f.mp4');");
+ shouldBeTrue("internals.isPreloaded('../security/resources/cap\ttions.vtt');");
+ shouldBeFalse("internals.isPreloaded('../resources/dummy.xml?badvalue');");
+ shouldBeTrue("internals.isPreloaded('../resources/dummy .xml');");
+ shouldBeFalse("internals.isPreloaded('../resources/dummy.xml');");
+ shouldBeFalse("internals.isPreloaded('../resources/dumm');");
+ shouldBeFalse("internals.isPreloaded('http://localhost:8000/preload/resources/square.png');");
+ // Invalid ports get preloaded (and get terminated further down the stack).
+ shouldBeTrue("internals.isPreloaded('http://localhost:53/preload/resources/dummy.js');");
+ shouldBeFalse("internals.isPreloaded('#foobar');");
+ shouldBeFalse("internals.isPreloaded('../resources/Ahem.ttf');");
+</script>
+<script>
+ if (window.internals)
+ window.internals.setLinkPreloadSupport(false);
+</script>
</ins></span></pre></div>
<a id="trunkSourceWebCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/CMakeLists.txt (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/CMakeLists.txt 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/CMakeLists.txt 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -1856,6 +1856,7 @@
</span><span class="cx"> loader/FrameLoaderStateMachine.cpp
</span><span class="cx"> loader/HistoryController.cpp
</span><span class="cx"> loader/ImageLoader.cpp
</span><ins>+ loader/LinkHeader.cpp
</ins><span class="cx"> loader/LinkLoader.cpp
</span><span class="cx"> loader/LinkPreloadResourceClients.cpp
</span><span class="cx"> loader/LoadTiming.cpp
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/ChangeLog 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -1,3 +1,57 @@
</span><ins>+2017-01-18 Yoav Weiss <yoav@yoav.ws>
+
+ Add Link header support for preload.
+ https://bugs.webkit.org/show_bug.cgi?id=165521
+
+ Reviewed by Alex Christensen.
+
+ This patch adds Link header support for preload, by adding a Link
+ header parser, and calling it from FrameLoader::receivedFirstData.
+
+ Test: http/tests/preload/download_resources_from_header.php
+
+ * CMakeLists.txt: Add LinkHeader files.
+ * WebCore.xcodeproj/project.pbxproj: Add LinkHeader files.
+ * loader/FrameLoader.cpp:
+ (WebCore::receivedFirstData): Trigger Link based preloading by calling loadLinksFromHeader.
+ * loader/LinkHeader.cpp: Added.
+ (WebCore::isWhitespace):
+ (WebCore::isValidURLChar):
+ (WebCore::isValidParameterNameChar):
+ (WebCore::isValidParameterValueEnd):
+ (WebCore::isValidParameterValueChar):
+ (WebCore::isExtensionParameter):
+ (WebCore::parseURL):
+ (WebCore::invalidParameterDelimiter):
+ (WebCore::validFieldEnd):
+ (WebCore::parseParameterDelimiter):
+ (WebCore::paramterNameFromString):
+ (WebCore::parseParameterName):
+ (WebCore::skipQuotesIfNeeded):
+ (WebCore::parseParameterValue):
+ (WebCore::LinkHeader::setValue):
+ (WebCore::findNextHeader):
+ (WebCore::LinkHeader::LinkHeader):
+ (WebCore::LinkHeaderSet::LinkHeaderSet):
+ (WebCore::LinkHeaderSet::init):
+ * loader/LinkHeader.h: Added.
+ (WebCore::LinkHeader::url):
+ (WebCore::LinkHeader::rel):
+ (WebCore::LinkHeader::as):
+ (WebCore::LinkHeader::mimeType):
+ (WebCore::LinkHeader::media):
+ (WebCore::LinkHeader::crossOrigin):
+ (WebCore::LinkHeader::valid):
+ (WebCore::LinkHeaderSet::begin):
+ (WebCore::LinkHeaderSet::end):
+ (WebCore::LinkHeaderSet::operator[]):
+ (WebCore::LinkHeaderSet::size):
+ * loader/LinkLoader.cpp:
+ (WebCore::LinkLoader::loadLinksFromHeader): Loads link preload directives that arrive from Link headers.
+ (WebCore::LinkLoader::preloadIfNeeded): Add return value.
+ * loader/LinkLoader.h:
+ * platform/network/HTTPHeaderNames.in: Add "Link" as a header name.
+
</ins><span class="cx"> 2017-01-18 Filip Pizlo <fpizlo@apple.com>
</span><span class="cx">
</span><span class="cx"> JSSegmentedVariableObject and its subclasses should have a sane destruction story
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -5708,6 +5708,8 @@
</span><span class="cx"> CB38FD5B1CD2325B00592A3F /* JSPerformanceResourceTiming.h in Headers */ = {isa = PBXBuildFile; fileRef = CB38FD591CD2314500592A3F /* JSPerformanceResourceTiming.h */; };
</span><span class="cx"> CB61C7931D755207001ADC7D /* LinkPreloadResourceClients.cpp in Sources */ = {isa = PBXBuildFile; fileRef = CBB6B2D11CB7ADC6009EDE1A /* LinkPreloadResourceClients.cpp */; };
</span><span class="cx"> CB8CF0181A9358D4000D510B /* Microtasks.cpp in Sources */ = {isa = PBXBuildFile; fileRef = CB8CF0151A934B43000D510B /* Microtasks.cpp */; };
</span><ins>+ CBA9DC0A1DF44DF10005675C /* LinkHeader.cpp in Sources */ = {isa = PBXBuildFile; fileRef = CBA9DC081DF44DC40005675C /* LinkHeader.cpp */; };
+ CBA9DC0B1DF44DF40005675C /* LinkHeader.h in Headers */ = {isa = PBXBuildFile; fileRef = CBA9DC091DF44DC40005675C /* LinkHeader.h */; settings = {ATTRIBUTES = (Private, ); }; };
</ins><span class="cx"> CBB6B2D41CB7AE51009EDE1A /* LinkPreloadResourceClients.h in Headers */ = {isa = PBXBuildFile; fileRef = CBB6B2D21CB7ADD0009EDE1A /* LinkPreloadResourceClients.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> CBC2D22F1CE5B89D00D1880B /* ResourceTimingInformation.cpp in Sources */ = {isa = PBXBuildFile; fileRef = CBC2D22D1CE5B77400D1880B /* ResourceTimingInformation.cpp */; };
</span><span class="cx"> CBC2D2301CE5B8A100D1880B /* ResourceTimingInformation.h in Headers */ = {isa = PBXBuildFile; fileRef = CBC2D22E1CE5B77D00D1880B /* ResourceTimingInformation.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="lines">@@ -13637,6 +13639,8 @@
</span><span class="cx"> CB38FD581CD2314500592A3F /* JSPerformanceResourceTiming.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSPerformanceResourceTiming.cpp; sourceTree = "<group>"; };
</span><span class="cx"> CB38FD591CD2314500592A3F /* JSPerformanceResourceTiming.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSPerformanceResourceTiming.h; sourceTree = "<group>"; };
</span><span class="cx"> CB8CF0151A934B43000D510B /* Microtasks.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = Microtasks.cpp; sourceTree = "<group>"; };
</span><ins>+ CBA9DC081DF44DC40005675C /* LinkHeader.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = LinkHeader.cpp; sourceTree = "<group>"; };
+ CBA9DC091DF44DC40005675C /* LinkHeader.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = LinkHeader.h; sourceTree = "<group>"; };
</ins><span class="cx"> CBB6B2D11CB7ADC6009EDE1A /* LinkPreloadResourceClients.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = LinkPreloadResourceClients.cpp; sourceTree = "<group>"; };
</span><span class="cx"> CBB6B2D21CB7ADD0009EDE1A /* LinkPreloadResourceClients.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = LinkPreloadResourceClients.h; sourceTree = "<group>"; };
</span><span class="cx"> CBC2D22D1CE5B77400D1880B /* ResourceTimingInformation.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = ResourceTimingInformation.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -22332,6 +22336,8 @@
</span><span class="cx"> BCB16BFB0979C38700467741 /* loader */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ CBA9DC081DF44DC40005675C /* LinkHeader.cpp */,
+ CBA9DC091DF44DC40005675C /* LinkHeader.h */,
</ins><span class="cx"> 1A8F6BB00DB55CDC001DB794 /* appcache */,
</span><span class="cx"> 512DD8E80D91E691000F89EE /* archive */,
</span><span class="cx"> A8D2B2521287A56000AF4DDA /* cache */,
</span><span class="lines">@@ -25814,6 +25820,7 @@
</span><span class="cx"> A8EA7CAF0A192B9C00A8EF5F /* HTMLHRElement.h in Headers */,
</span><span class="cx"> A871DE270A152AC800B12A68 /* HTMLHtmlElement.h in Headers */,
</span><span class="cx"> A871DE2A0A152AC800B12A68 /* HTMLIFrameElement.h in Headers */,
</span><ins>+ CBA9DC0B1DF44DF40005675C /* LinkHeader.h in Headers */,
</ins><span class="cx"> A8EA7D2D0A19385500A8EF5F /* HTMLImageElement.h in Headers */,
</span><span class="cx"> A8EA7D2B0A19385500A8EF5F /* HTMLImageLoader.h in Headers */,
</span><span class="cx"> A81369CC097374F600D74463 /* HTMLInputElement.h in Headers */,
</span><span class="lines">@@ -30589,6 +30596,7 @@
</span><span class="cx"> 1402645E0AFDC19B005919E2 /* LoggingMac.mm in Sources */,
</span><span class="cx"> 1A0409F31A43675C009E47F3 /* MachSendRight.cpp in Sources */,
</span><span class="cx"> 932AD70517EFA2C30038F8FF /* MainFrame.cpp in Sources */,
</span><ins>+ CBA9DC0A1DF44DF10005675C /* LinkHeader.cpp in Sources */,
</ins><span class="cx"> 7AE6C93B1BE0C60100E19E03 /* MainThreadSharedTimer.cpp in Sources */,
</span><span class="cx"> 7AE6C9381BE0C5C800E19E03 /* MainThreadSharedTimerCF.cpp in Sources */,
</span><span class="cx"> 1A8F6BC50DB55CDC001DB794 /* ManifestParser.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceWebCoreloaderFrameLoadercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/FrameLoader.cpp 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -79,6 +79,7 @@
</span><span class="cx"> #include "IgnoreOpensDuringUnloadCountIncrementer.h"
</span><span class="cx"> #include "InspectorController.h"
</span><span class="cx"> #include "InspectorInstrumentation.h"
</span><ins>+#include "LinkLoader.h"
</ins><span class="cx"> #include "LoadTiming.h"
</span><span class="cx"> #include "LoaderStrategy.h"
</span><span class="cx"> #include "Logging.h"
</span><span class="lines">@@ -659,6 +660,9 @@
</span><span class="cx"> ASSERT(m_frame.document());
</span><span class="cx"> auto& document = *m_frame.document();
</span><span class="cx">
</span><ins>+ ASSERT(m_frame.document());
+ LinkLoader::loadLinksFromHeader(m_documentLoader->response().httpHeaderField(HTTPHeaderName::Link), m_frame.document()->url(), *m_frame.document());
+
</ins><span class="cx"> double delay;
</span><span class="cx"> String urlString;
</span><span class="cx"> if (!parseHTTPRefresh(m_documentLoader->response().httpHeaderField(HTTPHeaderName::Refresh), delay, urlString))
</span></span></pre></div>
<a id="trunkSourceWebCoreloaderLinkHeadercpp"></a>
<div class="addfile"><h4>Added: trunk/Source/WebCore/loader/LinkHeader.cpp (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/LinkHeader.cpp (rev 0)
+++ trunk/Source/WebCore/loader/LinkHeader.cpp 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,365 @@
</span><ins>+/*
+ * Copyright 2015 The Chromium Authors. All rights reserved.
+ * Copyright (C) 2016 Akamai Technologies Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "LinkHeader.h"
+
+#include "ParsingUtilities.h"
+
+namespace WebCore {
+
+// LWSP definition in https://www.ietf.org/rfc/rfc0822.txt
+template <typename CharacterType>
+static bool isSpaceOrTab(CharacterType chr)
+{
+ return (chr == ' ') || (chr == '\t');
+}
+
+template <typename CharacterType>
+static bool isNotURLTerminatingChar(CharacterType chr)
+{
+ return (chr != '>');
+}
+
+template <typename CharacterType>
+static bool isValidParameterNameChar(CharacterType chr)
+{
+ // A separator, CTL or '%', '*' or '\'' means the char is not valid.
+ // Definition as attr-char at https://tools.ietf.org/html/rfc5987
+ // CTL and separators are defined in https://tools.ietf.org/html/rfc2616#section-2.2
+ // Valid chars are alpha-numeric and any of !#$&+-.^_`|~"
+ if ((chr >= '^' && chr <= 'z') || (chr >= 'A' && chr <= 'Z') || (chr >= '0' && chr <= '9') || (chr >= '!' && chr <= '$') || chr == '&' || chr == '+' || chr == '-' || chr == '.')
+ return true;
+ return false;
+}
+
+template <typename CharacterType>
+static bool isParameterValueEnd(CharacterType chr)
+{
+ return chr == ';' || chr == ',';
+}
+
+template <typename CharacterType>
+static bool isParameterValueChar(CharacterType chr)
+{
+ return !isSpaceOrTab(chr) && !isParameterValueEnd(chr);
+}
+
+// Verify that the parameter is a link-extension which according to spec doesn't have to have a value.
+static bool isExtensionParameter(LinkHeader::LinkParameterName name)
+{
+ return name >= LinkHeader::LinkParameterUnknown;
+}
+
+// Before:
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+//
+// After (if successful: otherwise the method returns false)
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+template <typename CharacterType>
+static std::optional<String> findURLBoundaries(CharacterType*& position, CharacterType* const end)
+{
+ ASSERT(position <= end);
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ if (!skipExactly<CharacterType>(position, end, '<'))
+ return std::nullopt;
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+
+ CharacterType* urlStart = position;
+ skipWhile<CharacterType, isNotURLTerminatingChar>(position, end);
+ CharacterType* urlEnd = position;
+ skipUntil<CharacterType>(position, end, '>');
+ if (!skipExactly<CharacterType>(position, end, '>'))
+ return std::nullopt;
+
+ return String(urlStart, urlEnd - urlStart);
+}
+
+template <typename CharacterType>
+static bool invalidParameterDelimiter(CharacterType*& position, CharacterType* const end)
+{
+ ASSERT(position <= end);
+ return (!skipExactly<CharacterType>(position, end, ';') && (position < end) && (*position != ','));
+}
+
+template <typename CharacterType>
+static bool validFieldEnd(CharacterType*& position, CharacterType* const end)
+{
+ ASSERT(position <= end);
+ return (position == end || *position == ',');
+}
+
+// Before:
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+//
+// After (if successful: otherwise the method returns false, and modifies the isValid boolean accordingly)
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+template <typename CharacterType>
+static bool parseParameterDelimiter(CharacterType*& position, CharacterType* const end, bool& isValid)
+{
+ ASSERT(position <= end);
+ isValid = true;
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ if (invalidParameterDelimiter(position, end)) {
+ isValid = false;
+ return false;
+ }
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ if (validFieldEnd(position, end))
+ return false;
+ return true;
+}
+
+static LinkHeader::LinkParameterName paramterNameFromString(String name)
+{
+ if (equalLettersIgnoringASCIICase(name, "rel"))
+ return LinkHeader::LinkParameterRel;
+ if (equalLettersIgnoringASCIICase(name, "anchor"))
+ return LinkHeader::LinkParameterAnchor;
+ if (equalLettersIgnoringASCIICase(name, "crossorigin"))
+ return LinkHeader::LinkParameterCrossOrigin;
+ if (equalLettersIgnoringASCIICase(name, "title"))
+ return LinkHeader::LinkParameterTitle;
+ if (equalLettersIgnoringASCIICase(name, "media"))
+ return LinkHeader::LinkParameterMedia;
+ if (equalLettersIgnoringASCIICase(name, "type"))
+ return LinkHeader::LinkParameterType;
+ if (equalLettersIgnoringASCIICase(name, "rev"))
+ return LinkHeader::LinkParameterRev;
+ if (equalLettersIgnoringASCIICase(name, "hreflang"))
+ return LinkHeader::LinkParameterHreflang;
+ if (equalLettersIgnoringASCIICase(name, "as"))
+ return LinkHeader::LinkParameterAs;
+ return LinkHeader::LinkParameterUnknown;
+}
+
+// Before:
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+//
+// After (if successful: otherwise the method returns false)
+//
+// <cat.jpg>; rel=preload
+// ^ ^
+// position end
+template <typename CharacterType>
+static bool parseParameterName(CharacterType*& position, CharacterType* const end, LinkHeader::LinkParameterName& name)
+{
+ ASSERT(position <= end);
+ CharacterType* nameStart = position;
+ skipWhile<CharacterType, isValidParameterNameChar>(position, end);
+ CharacterType* nameEnd = position;
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ bool hasEqual = skipExactly<CharacterType>(position, end, '=');
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ name = paramterNameFromString(String(nameStart, nameEnd - nameStart));
+ if (hasEqual)
+ return true;
+ bool validParameterValueEnd = (position == end) || isParameterValueEnd(*position);
+ return validParameterValueEnd && isExtensionParameter(name);
+}
+
+// Before:
+//
+// <cat.jpg>; rel="preload"; type="image/jpeg";
+// ^ ^
+// position end
+//
+// After (if the parameter starts with a quote, otherwise the method returns false)
+//
+// <cat.jpg>; rel="preload"; type="image/jpeg";
+// ^ ^
+// position end
+template <typename CharacterType>
+static bool skipQuotesIfNeeded(CharacterType*& position, CharacterType* const end, bool& completeQuotes)
+{
+ ASSERT(position <= end);
+ unsigned char quote;
+ if (skipExactly<CharacterType>(position, end, '\''))
+ quote = '\'';
+ else if (skipExactly<CharacterType>(position, end, '"'))
+ quote = '"';
+ else
+ return false;
+
+ while (!completeQuotes && position < end) {
+ skipUntil(position, end, static_cast<CharacterType>(quote));
+ if (*(position - 1) != '\\')
+ completeQuotes = true;
+ completeQuotes = skipExactly(position, end, static_cast<CharacterType>(quote)) && completeQuotes;
+ }
+ return true;
+}
+
+// Before:
+//
+// <cat.jpg>; rel=preload; foo=bar
+// ^ ^
+// position end
+//
+// After (if successful: otherwise the method returns false)
+//
+// <cat.jpg>; rel=preload; foo=bar
+// ^ ^
+// position end
+template <typename CharacterType>
+static bool parseParameterValue(CharacterType*& position, CharacterType* const end, String& value)
+{
+ ASSERT(position <= end);
+ CharacterType* valueStart = position;
+ CharacterType* valueEnd = position;
+ bool completeQuotes = false;
+ bool hasQuotes = skipQuotesIfNeeded(position, end, completeQuotes);
+ if (!hasQuotes)
+ skipWhile<CharacterType, isParameterValueChar>(position, end);
+ valueEnd = position;
+ skipWhile<CharacterType, isSpaceOrTab>(position, end);
+ if ((!completeQuotes && valueStart == valueEnd) || (position != end && !isParameterValueEnd(*position))) {
+ value = String("");
+ return false;
+ }
+ if (hasQuotes)
+ ++valueStart;
+ if (completeQuotes)
+ --valueEnd;
+ ASSERT(valueEnd >= valueStart);
+ value = String(valueStart, valueEnd - valueStart);
+ return !hasQuotes || completeQuotes;
+}
+
+void LinkHeader::setValue(LinkParameterName name, String value)
+{
+ switch (name) {
+ case LinkParameterRel:
+ if (!m_rel)
+ m_rel = value;
+ break;
+ case LinkParameterAnchor:
+ m_isValid = false;
+ break;
+ case LinkParameterCrossOrigin:
+ m_crossOrigin = value;
+ break;
+ case LinkParameterAs:
+ m_as = value;
+ break;
+ case LinkParameterType:
+ m_mimeType = value;
+ break;
+ case LinkParameterMedia:
+ m_media = value;
+ break;
+ case LinkParameterTitle:
+ case LinkParameterRev:
+ case LinkParameterHreflang:
+ case LinkParameterUnknown:
+ // These parameters are not yet supported, so they are currently ignored.
+ break;
+ }
+ // FIXME: Add support for more header parameters as neccessary.
+}
+
+template <typename CharacterType>
+static void findNextHeader(CharacterType*& position, CharacterType* const end)
+{
+ ASSERT(position <= end);
+ skipUntil<CharacterType>(position, end, ',');
+ skipExactly<CharacterType>(position, end, ',');
+}
+
+template <typename CharacterType>
+LinkHeader::LinkHeader(CharacterType*& position, CharacterType* const end)
+{
+ ASSERT(position <= end);
+ auto urlResult = findURLBoundaries(position, end);
+ if (urlResult == std::nullopt) {
+ m_isValid = false;
+ findNextHeader(position, end);
+ return;
+ }
+ m_url = urlResult.value();
+
+ while (m_isValid && position < end) {
+ if (!parseParameterDelimiter(position, end, m_isValid)) {
+ findNextHeader(position, end);
+ return;
+ }
+
+ LinkParameterName parameterName;
+ if (!parseParameterName(position, end, parameterName)) {
+ findNextHeader(position, end);
+ m_isValid = false;
+ return;
+ }
+
+ String parameterValue;
+ if (!parseParameterValue(position, end, parameterValue) && !isExtensionParameter(parameterName)) {
+ findNextHeader(position, end);
+ m_isValid = false;
+ return;
+ }
+
+ setValue(parameterName, parameterValue);
+ }
+ findNextHeader(position, end);
+}
+
+LinkHeaderSet::LinkHeaderSet(const String& header)
+{
+ if (header.isNull())
+ return;
+
+ if (header.is8Bit())
+ init(header.characters8(), header.length());
+ else
+ init(header.characters16(), header.length());
+}
+
+template <typename CharacterType>
+void LinkHeaderSet::init(CharacterType* headerValue, size_t length)
+{
+ CharacterType* position = headerValue;
+ CharacterType* const end = headerValue + length;
+ while (position < end)
+ m_headerSet.append(LinkHeader(position, end));
+}
+
+} // namespace WebCore
+
</ins></span></pre></div>
<a id="trunkSourceWebCoreloaderLinkHeaderh"></a>
<div class="addfile"><h4>Added: trunk/Source/WebCore/loader/LinkHeader.h (0 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/LinkHeader.h (rev 0)
+++ trunk/Source/WebCore/loader/LinkHeader.h 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -0,0 +1,87 @@
</span><ins>+/*
+ * Copyright 2015 The Chromium Authors. All rights reserved.
+ * Copyright (C) 2016 Akamai Technologies Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#pragma once
+
+#include <wtf/text/WTFString.h>
+
+namespace WebCore {
+
+class LinkHeader {
+public:
+ template <typename CharacterType>
+ LinkHeader(CharacterType*& position, CharacterType* const end);
+
+ const String& url() const { return m_url; }
+ const String& rel() const { return m_rel; }
+ const String& as() const { return m_as; }
+ const String& mimeType() const { return m_mimeType; }
+ const String& media() const { return m_media; }
+ const String& crossOrigin() const { return m_crossOrigin; }
+ bool valid() const { return m_isValid; }
+
+ enum LinkParameterName {
+ LinkParameterRel,
+ LinkParameterAnchor,
+ LinkParameterTitle,
+ LinkParameterMedia,
+ LinkParameterType,
+ LinkParameterRev,
+ LinkParameterHreflang,
+ // Beyond this point, only link-extension parameters
+ LinkParameterUnknown,
+ LinkParameterCrossOrigin,
+ LinkParameterAs,
+ };
+
+private:
+ void setValue(LinkParameterName, String value);
+
+ String m_url;
+ String m_rel;
+ String m_as;
+ String m_mimeType;
+ String m_media;
+ String m_crossOrigin;
+ bool m_isValid { true };
+};
+
+class LinkHeaderSet {
+public:
+ LinkHeaderSet(const String& header);
+
+ Vector<LinkHeader>::const_iterator begin() const { return m_headerSet.begin(); }
+ Vector<LinkHeader>::const_iterator end() const { return m_headerSet.end(); }
+
+private:
+ template <typename CharacterType>
+ void init(CharacterType* headerValue, size_t length);
+
+ Vector<LinkHeader> m_headerSet;
+};
+
+} // namespace WebCore
+
</ins></span></pre></div>
<a id="trunkSourceWebCoreloaderLinkLoadercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/loader/LinkLoader.cpp (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/LinkLoader.cpp 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/loader/LinkLoader.cpp 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -43,6 +43,7 @@
</span><span class="cx"> #include "Frame.h"
</span><span class="cx"> #include "FrameLoaderClient.h"
</span><span class="cx"> #include "FrameView.h"
</span><ins>+#include "LinkHeader.h"
</ins><span class="cx"> #include "LinkPreloadResourceClients.h"
</span><span class="cx"> #include "LinkRelAttribute.h"
</span><span class="cx"> #include "RuntimeEnabledFeatures.h"
</span><span class="lines">@@ -83,6 +84,24 @@
</span><span class="cx"> m_cachedLinkResource = nullptr;
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+void LinkLoader::loadLinksFromHeader(const String& headerValue, const URL& baseURL, Document& document)
+{
+ if (headerValue.isEmpty())
+ return;
+ LinkHeaderSet headerSet(headerValue);
+ for (auto& header : headerSet) {
+ if (!header.valid() || header.url().isEmpty() || header.rel().isEmpty())
+ continue;
+
+ LinkRelAttribute relAttribute(header.rel());
+ URL url(baseURL, header.url());
+ // Sanity check to avoid re-entrancy here.
+ if (equalIgnoringFragmentIdentifier(url, baseURL))
+ continue;
+ preloadIfNeeded(relAttribute, url, document, header.as(), header.crossOrigin(), nullptr, nullptr);
+ }
+}
+
</ins><span class="cx"> std::optional<CachedResource::Type> LinkLoader::resourceTypeFromAsAttribute(const String& as)
</span><span class="cx"> {
</span><span class="cx"> if (as.isEmpty())
</span><span class="lines">@@ -139,21 +158,22 @@
</span><span class="cx"> return nullptr;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void LinkLoader::preloadIfNeeded(const LinkRelAttribute& relAttribute, const URL& href, Document& document, const String& as, const String& crossOriginMode)
</del><ins>+std::unique_ptr<LinkPreloadResourceClient> LinkLoader::preloadIfNeeded(const LinkRelAttribute& relAttribute, const URL& href, Document& document, const String& as, const String& crossOriginMode, LinkLoader* loader, LinkLoaderClient* client)
</ins><span class="cx"> {
</span><span class="cx"> if (!document.loader() || !relAttribute.isLinkPreload)
</span><del>- return;
</del><ins>+ return nullptr;
</ins><span class="cx">
</span><span class="cx"> ASSERT(RuntimeEnabledFeatures::sharedFeatures().linkPreloadEnabled());
</span><span class="cx"> if (!href.isValid()) {
</span><span class="cx"> document.addConsoleMessage(MessageSource::Other, MessageLevel::Error, String("<link rel=preload> has an invalid `href` value"));
</span><del>- return;
</del><ins>+ return nullptr;
</ins><span class="cx"> }
</span><span class="cx"> auto type = LinkLoader::resourceTypeFromAsAttribute(as);
</span><span class="cx"> if (!type) {
</span><span class="cx"> document.addConsoleMessage(MessageSource::Other, MessageLevel::Error, String("<link rel=preload> must have a valid `as` value"));
</span><del>- m_client.linkLoadingErrored();
- return;
</del><ins>+ if (client)
+ client->linkLoadingErrored();
+ return nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> ResourceRequest resourceRequest(document.completeURL(href));
</span><span class="lines">@@ -164,8 +184,9 @@
</span><span class="cx"> linkRequest.setAsPotentiallyCrossOrigin(crossOriginMode, document);
</span><span class="cx"> CachedResourceHandle<CachedResource> cachedLinkResource = document.cachedResourceLoader().preload(type.value(), WTFMove(linkRequest));
</span><span class="cx">
</span><del>- if (cachedLinkResource)
- m_preloadResourceClient = createLinkPreloadResourceClient(*cachedLinkResource, *this, type.value());
</del><ins>+ if (cachedLinkResource && loader)
+ return createLinkPreloadResourceClient(*cachedLinkResource, *loader, type.value());
+ return nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool LinkLoader::loadLink(const LinkRelAttribute& relAttribute, const URL& href, const String& as, const String& crossOrigin, Document& document)
</span><span class="lines">@@ -178,8 +199,11 @@
</span><span class="cx"> document.frame()->loader().client().prefetchDNS(href.host());
</span><span class="cx"> }
</span><span class="cx">
</span><del>- if (m_client.shouldLoadLink())
- preloadIfNeeded(relAttribute, href, document, as, crossOrigin);
</del><ins>+ if (m_client.shouldLoadLink()) {
+ auto resourceClient = preloadIfNeeded(relAttribute, href, document, as, crossOrigin, this, &m_client);
+ if (resourceClient)
+ m_preloadResourceClient = WTFMove(resourceClient);
+ }
</ins><span class="cx">
</span><span class="cx"> #if ENABLE(LINK_PREFETCH)
</span><span class="cx"> if ((relAttribute.isLinkPrefetch || relAttribute.isLinkSubresource) && href.isValid() && document.frame()) {
</span></span></pre></div>
<a id="trunkSourceWebCoreloaderLinkLoaderh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/loader/LinkLoader.h (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/LinkLoader.h 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/loader/LinkLoader.h 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -53,6 +53,7 @@
</span><span class="cx">
</span><span class="cx"> bool loadLink(const LinkRelAttribute&, const URL&, const String& as, const String& crossOrigin, Document&);
</span><span class="cx"> static std::optional<CachedResource::Type> resourceTypeFromAsAttribute(const String& as);
</span><ins>+ static void loadLinksFromHeader(const String& headerValue, const URL& baseURL, Document&);
</ins><span class="cx">
</span><span class="cx"> WeakPtr<LinkLoader> createWeakPtr() { return m_weakPtrFactory.createWeakPtr(); }
</span><span class="cx"> void triggerEvents(const CachedResource&);
</span><span class="lines">@@ -59,7 +60,7 @@
</span><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> void notifyFinished(CachedResource&) override;
</span><del>- void preloadIfNeeded(const LinkRelAttribute&, const URL& href, Document&, const String& as, const String& crossOriginMode);
</del><ins>+ static std::unique_ptr<LinkPreloadResourceClient> preloadIfNeeded(const LinkRelAttribute&, const URL& href, Document&, const String& as, const String& crossOriginMode, LinkLoader*, LinkLoaderClient*);
</ins><span class="cx">
</span><span class="cx"> LinkLoaderClient& m_client;
</span><span class="cx"> CachedResourceHandle<CachedResource> m_cachedLinkResource;
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformnetworkHTTPHeaderNamesin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/network/HTTPHeaderNames.in (210913 => 210914)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/network/HTTPHeaderNames.in 2017-01-19 05:26:49 UTC (rev 210913)
+++ trunk/Source/WebCore/platform/network/HTTPHeaderNames.in 2017-01-19 07:13:54 UTC (rev 210914)
</span><span class="lines">@@ -68,6 +68,7 @@
</span><span class="cx"> Last-Event-ID
</span><span class="cx"> Last-Modified
</span><span class="cx"> Location
</span><ins>+Link
</ins><span class="cx"> Origin
</span><span class="cx"> Ping-From
</span><span class="cx"> Ping-To
</span></span></pre>
</div>
</div>
</body>
</html>