Move adding Lo-Fi directives from DRPRequestOptions to ContentLoFiDecider

components/data_reduction_proxy/content/browser/content_lofi_decider.h

Index: components/data_reduction_proxy/content/browser/content_lofi_decider.h
diff --git a/components/data_reduction_proxy/content/browser/content_lofi_decider.h b/components/data_reduction_proxy/content/browser/content_lofi_decider.h
index 222fedde8a7d04e129adb9702d2917135c702313..7c5ec04aea84f0f8f1eb3593d52e98bec6bf142e 100644
--- a/components/data_reduction_proxy/content/browser/content_lofi_decider.h
+++ b/components/data_reduction_proxy/content/browser/content_lofi_decider.h
@@ -10,10 +10,14 @@
 
 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
@@ -26,6 +30,9 @@ class ContentLoFiDecider : public LoFiDecider {
 
   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);
 };