Don't wait for AppCache for link rel preloads

third_party/WebKit/Source/core/loader/DocumentLoader.cpp

Index: third_party/WebKit/Source/core/loader/DocumentLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/DocumentLoader.cpp b/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
index ba24f4d6e21fde07e25248049e60d76cc9ebd5b4..f971afbe9ae6ea122f4001843815484f6a8690c1 100644
--- a/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
@@ -208,6 +208,15 @@ Resource* DocumentLoader::startPreload(Resource::Type type, FetchRequest& reques
     return resource;
 }
 
+void DocumentLoader::dispatchLinkHeaderPreloads(ViewportDescriptionWrapper* viewport, LinkLoader::MediaPreloadPolicy mediaPolicy)
+{
+    String linkHeader = response().httpHeaderField(HTTPNames::Link);
+    if (!linkHeader.isEmpty()) {

[Yoav Weiss, 2016/07/22 16:03:46]

Why is this wrapper needed? There's already an empty check inside
loadLinksFromHeader

[Charlie Harrison, 2016/07/22 16:54:24]

Ah good catch. Removed. I still kept this helper function to avoid awkward
callsites reaching deep into document()->loader().

+        LinkLoader::loadLinksFromHeader(linkHeader, response().url(),
+            m_frame->document(), NetworkHintsInterfaceImpl(), LinkLoader::OnlyLoadResources, mediaPolicy, viewport);
+    }
+}
+
 void DocumentLoader::didChangePerformanceTiming()
 {
     if (frame() && frame()->isMainFrame() && m_state >= Committed) {