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..a1c51529ab2fdc01ce30e2865b38766b177d5329 100644
--- a/third_party/WebKit/public/platform/WebURLRequest.h
+++ b/third_party/WebKit/public/platform/WebURLRequest.h
@@ -136,13 +136,16 @@ 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. A value of zero for Previews state means to let the browser
+  // process decide whether or not to request Preview types.
+  enum PreviewsState {

[RyanSturm, 2016/12/06 19:39:30]

maybe nit: I can't find anything in the style guide, but it might be more clear
to replace various 0's throughout the CL with:
PreviewsUnspecified and defined PreviewsUnspecified as 0 here. Same for content
side of things if you decide to change it.

[megjablon, 2016/12/06 21:23:47]

I see this done elsewhere throughout the code, so I don't think there's a style
issue with it. Done.

+    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
+                            // the resource.
   };
 
   // Indicates which types of ServiceWorkers should skip handling this request.
@@ -285,10 +288,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 int getPreviewsState() const;
+  BLINK_PLATFORM_EXPORT void setPreviewsState(int);
 
   // Extra data associated with the underlying resource request. Resource
   // requests can be copied. If non-null, each copy of a resource requests