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

content/public/common/previews_state.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_PUBLIC_COMMON_PREVIEWS_TYPE_H_
+#define CONTENT_PUBLIC_COMMON_PREVIEWS_TYPE_H_
+
+#include "content/common/content_export.h"
+
+namespace content {
+
+// The Previews state which determines whether to request a Preview version of
+// the resource. Previews are optimizations that change the format and
+// content of web pages to improve data savings and / or performance. This enum
+// determines which Preview types to request.

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

nit: Previews?

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

Done.

+enum PreviewsState {
+  PREVIEWS_UNSPECIFIED = 0,  // Let the browser process decide whether or
+                             // not to request Preview types.
+  SERVER_LOFI_ON = 1 << 0,   // Request a Lo-Fi version of the resource
+                             // from the server.
+  CLIENT_LOFI_ON = 1 << 1,   // Request a Lo-Fi version of the resource
+                             // from the client.
+  PREVIEWS_OFF = 1 << 2,     // Request a normal (non-Preview) version of
+                             // the resource.
+  MAX_PREVIEWS_STATE = 1 << 3

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

You could use LAST pattern, which will be equal to the PREVIEWS_OFF values,
since we don't have to allow 1 << 3 as a value valid.

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

Done.

+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_PREVIEWS_TYPE_H_