Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h

Index: components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h
diff --git a/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h b/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h
index 884cc1f23158ab8adf19d9b60bd13d7c093011b4..a2fa63076cc9c27301ce24a2ce0340c83ecd53cc 100644
--- a/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h
+++ b/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.h
@@ -6,7 +6,6 @@
 #define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_DATA_REDUCTION_PROXY_MESSAGE_FILTER_H_
 
 #include "base/macros.h"
-#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h"
 #include "content/public/browser/browser_message_filter.h"
 
 namespace net {
@@ -15,6 +14,7 @@ class HostPortPair;
 
 namespace data_reduction_proxy {
 
+class DataReductionProxyConfig;
 class DataReductionProxySettings;
 
 // An IPC listener to handle DataReductionProxy IPC messages from the renderer.
@@ -26,10 +26,8 @@ class DataReductionProxyMessageFilter
 
   // Sets |is_data_reduction_proxy| to true if the |proxy_server| corresponds to
   // a Data Reduction Proxy.
-  // Sets |lofi_response| to the current status of the LoFi.
   void OnDataReductionProxyStatus(const net::HostPortPair& proxy_server,

[bengr, 2015/08/25 00:00:01]

Why not just return a bool?

[megjablon, 2015/08/25 20:29:45]

This is how IPC return values work.

-                                  bool* is_data_reduction_proxy,
-                                  LoFiStatus* lofi_status);
+                                  bool* is_data_reduction_proxy);
 
  private:
   ~DataReductionProxyMessageFilter() override;