Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/content_lofi_decider.cc

Index: components/data_reduction_proxy/content/browser/content_lofi_decider.cc
diff --git a/components/data_reduction_proxy/content/browser/content_lofi_decider.cc b/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
index 6a83c75d90e1d177730e5d7c59c59656c22aab9c..cd628a7bc40fafc4d92772da8e6f12ed768f023f 100644
--- a/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
+++ b/components/data_reduction_proxy/content/browser/content_lofi_decider.cc
@@ -4,6 +4,9 @@
 
 #include "components/data_reduction_proxy/content/browser/content_lofi_decider.h"
 
+#include "content/public/browser/resource_request_info.h"
+#include "net/url_request/url_request.h"

[bengr, 2015/10/20 16:30:51]

It looks like you can forward declare the URLRequest instead.

[megjablon, 2015/10/20 17:46:35]

Done.

[bengr, 2015/10/21 20:58:25]

You should forward declare it.

[megjablon, 2015/10/21 21:38:10]

It's already forward declared in the header file.

+
 namespace data_reduction_proxy {
 
 ContentLoFiDecider::ContentLoFiDecider() {}
@@ -11,7 +14,10 @@ ContentLoFiDecider::ContentLoFiDecider() {}
 ContentLoFiDecider::~ContentLoFiDecider() {}
 
 bool ContentLoFiDecider::IsUsingLoFiMode(const net::URLRequest& request) const {
-  // TODO: will be check of ResourceRequestInfo::IsUsingLoFi
+  const content::ResourceRequestInfo* request_info =
+      content::ResourceRequestInfo::ForRequest(&request);
+  if (request_info)
+    return request_info->IsUsingLoFi();
   return false;
 }