[ServiceWorker] Plumbing the request mode from the renderer to the ServiceWorker. [1/2 blink]

Source/modules/serviceworkers/FetchRequestData.cpp

Index: Source/modules/serviceworkers/FetchRequestData.cpp
diff --git a/Source/modules/serviceworkers/FetchRequestData.cpp b/Source/modules/serviceworkers/FetchRequestData.cpp
index d9f7f5d2f03a03f757f7f12ad32b63a1599b17f6..63fc78a33baf966fee294ab4a3a5083bde27f7af 100644
--- a/Source/modules/serviceworkers/FetchRequestData.cpp
+++ b/Source/modules/serviceworkers/FetchRequestData.cpp
@@ -12,6 +12,7 @@
 #include "modules/serviceworkers/FetchHeaderList.h"
 #include "platform/network/ResourceRequest.h"
 #include "public/platform/WebServiceWorkerRequest.h"
+#include "public/platform/WebURLRequest.h"
 
 namespace blink {
 
@@ -39,6 +40,23 @@ FetchRequestData* FetchRequestData::create(const WebServiceWorkerRequest& webReq
         request->m_headerList->append(it->key, it->value);
     request->m_blobDataHandle = webRequest.blobDataHandle();
     request->m_referrer.setURL(webRequest.referrer());
+    switch (webRequest.mode()) {
+    case WebURLRequest::FetchRequestModeSameOrigin:
+        request->setMode(FetchRequestData::SameOriginMode);

[Mike West, 2014/09/25 12:35:53]

I think FrameRequestData should simply share the new platform-level enum. You
don't need to do that refactoring in this CL, but I think it's the right
direction to go.

[horo, 2014/09/25 14:02:16]

Acknowledged.

+        break;
+    case WebURLRequest::FetchRequestModeNoCORS:
+        request->setMode(FetchRequestData::NoCORSMode);
+        break;
+    case WebURLRequest::FetchRequestModeCORS:
+        request->setMode(FetchRequestData::CORSMode);
+        break;
+    case WebURLRequest::FetchRequestModeCORSWithForcedPreflight:
+        request->setMode(FetchRequestData::CORSWithForcedPreflight);
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+        break;
+    }
     return request;
 }