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

third_party/WebKit/public/platform/WebURLRequest.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 123 matching lines @@
         ReportIntent, // Report metrics with UI action displayed intent.
     };
 
     // The LoFi state which determines whether to request a Lo-Fi version of the resource.
     enum LoFiState {
         LoFiUnspecified, // Let the browser process decide whether or not to request the Lo-Fi version.
         LoFiOff, // Request a normal (non-Lo-Fi) version of the resource.
         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() { }
     };
 
     ~WebURLRequest() { reset(); }
 
     WebURLRequest() : m_private(0) { }
     WebURLRequest(const WebURLRequest& r) : m_private(0) { assign(r); }
     WebURLRequest& operator=(const WebURLRequest& r)
@@ skipping 94 matching lines @@
     // If true, the response body will be downloaded to a file managed by the
     // WebURLLoader. See WebURLResponse::downloadedFilePath.
     BLINK_PLATFORM_EXPORT bool downloadToFile() const;
     BLINK_PLATFORM_EXPORT void setDownloadToFile(bool);
 
     // True if the requestor wants to receive the response body as a stream.
     BLINK_PLATFORM_EXPORT bool useStreamOnResponse() const;
     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;
     BLINK_PLATFORM_EXPORT void setShouldResetAppCache(bool);
 
     // The request mode which will be passed to the ServiceWorker.
     BLINK_PLATFORM_EXPORT FetchRequestMode getFetchRequestMode() const;
     BLINK_PLATFORM_EXPORT void setFetchRequestMode(FetchRequestMode);
 
     // The credentials mode which will be passed to the ServiceWorker.
@@ skipping 46 matching lines @@
 protected:
     BLINK_PLATFORM_EXPORT void assign(WebURLRequestPrivate*);
 
 private:
     WebURLRequestPrivate* m_private;
 };
 
 } // namespace blink
 
 #endif