[DRP] Consistently use LoFi for an entire page

components/data_reduction_proxy/core/common/lofi_decider.h

Index: components/data_reduction_proxy/core/common/lofi_decider.h
diff --git a/components/data_reduction_proxy/core/common/lofi_decider.h b/components/data_reduction_proxy/core/common/lofi_decider.h
new file mode 100644
index 0000000000000000000000000000000000000000..57b25d470fd6d0eeb65d36dcb6eb739232c2fb35
--- /dev/null
+++ b/components/data_reduction_proxy/core/common/lofi_decider.h
@@ -0,0 +1,25 @@
+// Copyright 2015 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 COMPONENTS_DATA_REDUCTION_PROXY_CORE_BROWSER_LOFI_DECIDER_H_
+#define COMPONENTS_DATA_REDUCTION_PROXY_CORE_BROWSER_LOFI_DECIDER_H_
+
+#include "base/macros.h"
+#include "net/url_request/url_request.h"
+
+namespace data_reduction_proxy {
+
+// Interface to determine if a request should be made for a low fidelity version
+// of the resource.
+class LoFiDecider {
+ public:
+  virtual ~LoFiDecider() {}
+
+  // Returns true when Lo-Fi mode is activated.
+  virtual bool IsUsingLoFiMode(const net::URLRequest& request) const = 0;

[bengr, 2015/10/12 21:06:41]

I really don't understand the name. I'm presuming this is called before the
request header is added. If so, this should be called ShouldUseLoFiMode().
Please explain in the comment and rename if needed.

[megjablon, 2015/10/12 23:08:05]

ShouldUseLoFiMode is reserved for when we call the logic that determines if we
should activate Lo-Fi mode. This is done on navigations from
ResourceDispatcherHostImpl. IsUsingLoFiMode is once we have set Lo-Fi mode to on
for a page and then that state is copied to all requests.

+};
+
+}  // namespace data_reduction_proxy
+
+#endif  // COMPONENTS_DATA_REDUCTION_PROXY_CORE_BROWSER_LOFI_DECIDER_H_