Split requestResource into request preparation and loading

third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h

Index: third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h
diff --git a/third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h b/third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h
index bfe2ae9c4937250cfbaad225fa0d67aba7496598..9f9797d0340b54ef5afa048e5402c17755dc827d 100644
--- a/third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h
+++ b/third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.h
@@ -171,6 +171,18 @@ class PLATFORM_EXPORT ResourceFetcher
       FetchRequest::DeferOption = FetchRequest::NoDefer,
       bool forPreload = false);
 
+  struct PrepareRequestReturnValues {

[Charlie Harrison, 2017/01/30 22:02:58]

delete unused struct.

+    bool isStaticData{false};
+    bool shouldContinue{false};
+  };
+
+  Resource* prepareRequest(FetchRequest&,

[Charlie Harrison, 2017/01/30 22:02:58]

Please document this method. It is a bit strange for a method to be called
"prepareRequest" but return a Resource*, so I think a little explanation is
necessary.

+                           const ResourceFactory&,
+                           const SubstituteData&,
+                           unsigned long identifier,
+                           bool& isStaticData,
+                           bool& shouldContinue);
+
   Resource* resourceForStaticData(const FetchRequest&,
                                   const ResourceFactory&,
                                   const SubstituteData&);