[ServiceWorker] Set FetchRequestMode and handle wasFetchedViaServiceWorker.

Source/core/loader/DocumentThreadableLoader.h

Index: Source/core/loader/DocumentThreadableLoader.h
diff --git a/Source/core/loader/DocumentThreadableLoader.h b/Source/core/loader/DocumentThreadableLoader.h
index d3c50b2d40bef1784881f0689298d6fdc67ae124..3693b9b82d1ba63b9250650c97b36c86ee99f91e 100644
--- a/Source/core/loader/DocumentThreadableLoader.h
+++ b/Source/core/loader/DocumentThreadableLoader.h
@@ -92,6 +92,8 @@ class DocumentThreadableLoader final : public ThreadableLoader, private Resource
 
         void didTimeout(Timer<DocumentThreadableLoader>*);
         void makeCrossOriginAccessRequest(const ResourceRequest&);
+        // Loads m_fallbackRequest.
+        void loadFallbackRequest();

[falken, 2014/10/09 05:38:34]

This is a pretty general name for a specific Service Worker thing. Maybe
"loadFallbackRequestForServiceWorker"

[horo, 2014/10/09 07:27:53]

Done.

         // Loads m_actualRequest.
         void loadActualRequest();
         // Clears m_actualRequest and reports access control check failure to
@@ -127,6 +129,9 @@ class DocumentThreadableLoader final : public ThreadableLoader, private Resource
         bool m_simpleRequest;
         bool m_async;
 
+        // Holds the original request for fallback of fetching via the
+        // ServiceWorker.

[falken, 2014/10/09 05:38:34]

Maybe, "Holds the original request for fallback in case the Service Worker does
not respond" is more clear.

[horo, 2014/10/09 07:27:53]

Done.

+        OwnPtr<ResourceRequest> m_fallbackRequest;

[falken, 2014/10/09 05:38:34]

As above, "m_fallbackRequestForServiceWorker" may be a better name.

[horo, 2014/10/09 07:27:53]

Done.

         // Holds the original request and options for it during preflight
         // request handling phase.
         OwnPtr<ResourceRequest> m_actualRequest;