[DRP] Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/content_lofi_decider.h

+// 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_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
+#define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
+
+#include "components/data_reduction_proxy/core/common/lofi_decider.h"
+#include "base/macros.h"
+
+namespace net {
+class URLRequest;
+}
+
+namespace data_reduction_proxy {
+
+// Handles getting if a request is in Lo-Fi mode.

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

How about:

// Class responsible for deciding whether a request
// should be requested with low fidelity (Lo-Fi) or
// not.

Also, please say why this lives in content, and mention any assumptions about
threading and lifetime.

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

Done.

+class ContentLoFiDecider : public LoFiDecider {
+ public:
+  ContentLoFiDecider();
+  ~ContentLoFiDecider() override;
+
+  bool IsUsingLoFiMode(const net::URLRequest& request) const override;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ContentLoFiDecider);
+};
+
+}  // namespace data_reduction_proxy
+
+#endif  // COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_