Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc

Index: components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc
diff --git a/components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc b/components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc
index a6ebb262139a132c1c87ce54264dfadc4ce784b6..719648f559f3f0b671fc65a1c8e31a84e0be7007 100644
--- a/components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc
+++ b/components/data_reduction_proxy/content/browser/content_data_reduction_proxy_lofi_helper.cc
@@ -15,8 +15,12 @@ ContentDataReductionProxyLoFiHelper::~ContentDataReductionProxyLoFiHelper() {}
 
 bool ContentDataReductionProxyLoFiHelper::IsLoFi(
     net::URLRequest* request) const {
-  // TODO: return IsLoFi from ResourceRequestInfo.
-  return false;
+  const content::ResourceRequestInfo* request_info =
+      content::ResourceRequestInfo::ForRequest(request);

[mmenke, 2015/09/29 16:35:57]

ResourceRequestInfo will never exist at this point - we need to call this method
to figure out the value to set its IsLoFi bit in the first place, before we can
create it.

[megjablon, 2015/09/29 21:20:07]

This is called from DRPNetworkDelegate::OnBeforeSendProxyHeadersInternal. I
instrumented this and the ResourceRequestInfo exists from
ResourceDispatcherHostImpl::BeginRequest which does happen before.

[mmenke, 2015/09/30 19:33:41]

Sorry, I thought that this is what was being called from ShouldEnableLoFiMode
(Which seemed really odd to me) - I hadn't realized there was pre-existing code,
thought this added all of the lofi stuff.

+  if (request_info)
+    return request_info->IsLoFi();
+  else
+    return false;
 }
 
 }  // namespace data_reduction_roxy