Move adding Lo-Fi directives from DRPRequestOptions to ContentLoFiDecider

components/data_reduction_proxy/content/browser/content_lofi_decider.h

 // 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.
 
 #ifndef COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
 #define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
 
 #include "components/data_reduction_proxy/core/common/lofi_decider.h"
 #include "base/macros.h"
 
 namespace net {
 class URLRequest;
+class HttpRequestHeaders;

[tbansal1, 2015/11/20 21:56:58]

Alphabetize.

[megjablon, 2015/11/20 23:14:50]

Done.

 }
 
 namespace data_reduction_proxy {
 
+extern const char kLoFiHeader[];
+extern const char kLoFiExperimentHeader[];
+
 // Class responsible for deciding whether a request should be requested with low
 // fidelity (Lo-Fi) or not. Relies on the Lo-Fi mode state stored in the
 // request's content::ResourceRequestInfo, which must be fetched using
 // content::ResourceRequestInfo::ForRequest. Owned by DataReductionProxyIOData
 // and should be called on the IO thread.
 class ContentLoFiDecider : public LoFiDecider {
  public:
   ContentLoFiDecider();
   ~ContentLoFiDecider() override;
 
   bool IsUsingLoFiMode(const net::URLRequest& request) const override;
 
+  bool MaybeEnableLoFiMode(const net::URLRequest& request,

[tbansal1, 2015/11/20 21:56:58]

// LoFiDecider implementation:


Also, this is not really enabling a mode..right? I would suggest
MaybeAddLoFiDirectiveToHeaders but it might be too long.

[megjablon, 2015/11/20 23:14:50]

SGTM. It's only called in one place so I don't think that's too long.

+                           net::HttpRequestHeaders* headers) const override;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(ContentLoFiDecider);
 };
 
 }  // namespace data_reduction_proxy
 
 #endif  // COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_