[DRP] Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc

Index: components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc
diff --git a/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc b/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc
index 7fbb47b28f1b9eece3e365049e29ff0dfe84ffa6..2558c959651f272bbf0d3a9c3a9b8638c160d62f 100644
--- a/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc
+++ b/components/data_reduction_proxy/content/browser/data_reduction_proxy_message_filter.cc
@@ -28,8 +28,8 @@ bool DataReductionProxyMessageFilter::OnMessageReceived(
     const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(DataReductionProxyMessageFilter, message)
-  IPC_MESSAGE_HANDLER(DataReductionProxyViewHostMsg_DataReductionProxyStatus,
-                      OnDataReductionProxyStatus)
+    IPC_MESSAGE_HANDLER(DataReductionProxyViewHostMsg_IsDataReductionProxy,
+                        OnIsDataReductionProxy)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
@@ -38,25 +38,20 @@ bool DataReductionProxyMessageFilter::OnMessageReceived(
 void DataReductionProxyMessageFilter::OverrideThreadForMessage(
     const IPC::Message& message, content::BrowserThread::ID* thread) {
   if (message.type() ==
-      DataReductionProxyViewHostMsg_DataReductionProxyStatus::ID) {
+      DataReductionProxyViewHostMsg_IsDataReductionProxy::ID) {
       *thread = content::BrowserThread::IO;
   }
 }
 
-void DataReductionProxyMessageFilter::OnDataReductionProxyStatus(
+void DataReductionProxyMessageFilter::OnIsDataReductionProxy(
     const net::HostPortPair& proxy_server,
-    bool* is_data_reduction_proxy,
-    LoFiStatus* lofi_status) {
+    bool* is_data_reduction_proxy) {
   DCHECK(is_data_reduction_proxy);

[bengr, 2015/10/14 22:34:10]

DCHECKing whether something is null isn't needed because the program will crash
anyway.

[megjablon, 2015/10/15 03:44:06]

Done.

-  DCHECK(lofi_status);
   *is_data_reduction_proxy = false;
-  *lofi_status = LOFI_STATUS_TEMPORARILY_OFF;
   if (!config_)
     return;
   *is_data_reduction_proxy =
       config_->IsDataReductionProxy(proxy_server, nullptr);
-  if (*is_data_reduction_proxy)
-    *lofi_status = config_->GetLoFiStatus();
 }
 
 }  // namespace data_reduction_proxy