Consistently use LoFi for an entire page

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Index: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
index b28332ea3809ee058b9b810f1b6d44be397b5319..35caa29445df4051bb0cb78ed8276cdb9210314b 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
@@ -34,6 +34,7 @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
+#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.h"
 #include "components/google/core/browser/google_util.h"
 #include "components/variations/net/variations_http_header_provider.h"
 #include "content/public/browser/browser_thread.h"
@@ -721,6 +722,18 @@ void ChromeResourceDispatcherHostDelegate::RequestComplete(
   }
 }
 
+bool ChromeResourceDispatcherHostDelegate::ShouldEnableLoFiMode(
+    net::URLRequest* url_request,
+    content::ResourceContext* resource_context) {
+  ProfileIOData* io_data = ProfileIOData::FromResourceContext(resource_context);
+  data_reduction_proxy::DataReductionProxyIOData* data_reduction_proxy_io_data =
+      io_data->data_reduction_proxy_io_data();
+
+  if (data_reduction_proxy_io_data)
+    return data_reduction_proxy_io_data->ShouldEnableLoFiMode(url_request);

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

This method doesn't seem to exist.

Also, I think the logic here makes more sense inlined than in a method of DRP's
catch-all class.  It doesn't, for instance, make any sense in Cronet's context,
I believe, and Cronet also uses DRP_io_data.

[megjablon, 2015/09/30 22:14:08]

Sorry this was split into two cls since it was getting enormous. The DRP
portions of it are in the other one and ShouldEnableLoFiMode is created there.
Inlining this here is not simple. There's a lot of code that is DRP specific and
would require more massive changes. Plus we may see value in adding this in
other contexts in the future.

+  return false;
+}
+
 // static
 void ChromeResourceDispatcherHostDelegate::
     SetExternalProtocolHandlerDelegateForTesting(