Skip foreign fetch when the skipServiceWorker flag is set on a request.

third_party/WebKit/public/platform/WebURLRequest.h

Index: third_party/WebKit/public/platform/WebURLRequest.h
diff --git a/third_party/WebKit/public/platform/WebURLRequest.h b/third_party/WebKit/public/platform/WebURLRequest.h
index 09b58183fd9e0717ee776f2f549b0cd958e33800..643957d31691bc7fc484482c66b33735fc7794c9 100644
--- a/third_party/WebKit/public/platform/WebURLRequest.h
+++ b/third_party/WebKit/public/platform/WebURLRequest.h
@@ -141,6 +141,18 @@ public:
         LoFiOn, // Request a Lo-Fi version of the resource.
     };
 
+    // Whether or not a request should be handled by a ServiceWorker.

[kinuko, 2016/06/30 09:38:22]

ditto.  Also-- started to wonder if we could have just defined single enum in
public/platform and use it in content too... having dup'ed same enums at
multiple places with static_assert is awkward.

[Marijn Kruisselbrink, 2016/06/30 17:41:54]

We could... we'd need to move the enums we'd want to share to some header file
that doesn't also declare classes for content/browser/DEPS to be able to allow
usage of them of course, but that would be easy. All the different classes we
have to represent requests (and all the conversions between them) seem pretty
awkward too, although I'm sure there are good reasons for those.

Anyway, at least the enum part seems like something that would at least change
in some way when more stuff gets mojoified. Just not sure what the some way will
be (hopefully not just adding yet another enum and more conversions). It just
doesn't seem worth the effort now to unify all these enums when it is going to
change in some way anyway. But maybe mojoifying this is far enough away that
unifying enums first is going to make sense anyway, I don't know.

+    enum class SkipServiceWorker {
+        // Request can be handled both by a controlling same-origin worker and
+        // a cross-origin foreign fetch service worker.
+        None,
+        // Request should not be handled by a same-origin controlling worker,
+        // but can be intercepted by a foreign fetch service worker.
+        Controlling,
+        // Request should skip all possible service workers.
+        All
+    };
+
     class ExtraData {
     public:
         virtual ~ExtraData() { }
@@ -255,8 +267,8 @@ public:
     BLINK_PLATFORM_EXPORT void setUseStreamOnResponse(bool);
 
     // True if the request should not be handled by the ServiceWorker.
-    BLINK_PLATFORM_EXPORT bool skipServiceWorker() const;
-    BLINK_PLATFORM_EXPORT void setSkipServiceWorker(bool);
+    BLINK_PLATFORM_EXPORT SkipServiceWorker skipServiceWorker() const;
+    BLINK_PLATFORM_EXPORT void setSkipServiceWorker(SkipServiceWorker);
 
     // True if corresponding AppCache group should be resetted.
     BLINK_PLATFORM_EXPORT bool shouldResetAppCache() const;