Change Lo-Fi bool to bitmask to support multiple Previews types

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 c58a4e491222d25c0643a790d77ce7e836e6c9af..d5d78aa389ad625b9bbd2d29cd871e7aa6ea17b8 100644
--- a/third_party/WebKit/public/platform/WebURLRequest.h
+++ b/third_party/WebKit/public/platform/WebURLRequest.h
@@ -136,13 +136,17 @@ class WebURLRequest {
     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.
+  // The Previews state which determines whether to request a Preview version of
+  // the resource.
+  enum PreviewsState {
+    PreviewsUnspecified = 0,  // Let the browser process decide whether or
+                              // not to request Preview types.
+    ServerLoFiOn = 1 << 0,    // Request a Lo-Fi version of the resource
+                              // from the server.
+    ClientLoFiOn = 1 << 1,    // Request a Lo-Fi version of the resource
+                              // from the client.
+    PreviewsOff = 1 << 2      // Request a normal (non-Preview) version of

[nasko, 2016/12/12 23:05:41]

Can we add static checks to ensure the two enums don't get out of sync? An
example you can refer to is
https://cs.chromium.org/chromium/src/content/renderer/render_thread_impl.cc?r....

[megjablon, 2016/12/13 03:03:14]

Excellent idea. I added this to render_frame_impl.cc. Please let me know if it
should live elsewhere.

+                              // the resource.
   };
 
   // Indicates which types of ServiceWorkers should skip handling this request.
@@ -285,10 +289,10 @@ class WebURLRequest {
   BLINK_PLATFORM_EXPORT FetchRedirectMode getFetchRedirectMode() const;
   BLINK_PLATFORM_EXPORT void setFetchRedirectMode(FetchRedirectMode);
 
-  // The LoFi state which determines whether to request a Lo-Fi version of the
-  // resource.
-  BLINK_PLATFORM_EXPORT LoFiState getLoFiState() const;
-  BLINK_PLATFORM_EXPORT void setLoFiState(LoFiState);
+  // The Previews state which determines whether to request a Preview version of
+  // the resource.
+  BLINK_PLATFORM_EXPORT PreviewsState getPreviewsState() const;
+  BLINK_PLATFORM_EXPORT void setPreviewsState(PreviewsState);
 
   // Extra data associated with the underlying resource request. Resource
   // requests can be copied. If non-null, each copy of a resource requests