Consistently use LoFi for an entire page

components/data_reduction_proxy/content/browser/content_lofi_decider.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #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() {}
 
 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;
 }
 
 }  // namespace data_reduction_proxy