Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(756)

Side by Side Diff: components/data_reduction_proxy/content/browser/content_lofi_decider.h

Issue 1463583003: Move adding Lo-Fi directives from DRPRequestOptions to ContentLoFiDecider (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressing tbansal comments Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_ 5 #ifndef COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
6 #define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_ 6 #define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_H_
7 7
8 #include "components/data_reduction_proxy/core/common/lofi_decider.h" 8 #include "components/data_reduction_proxy/core/common/lofi_decider.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 10
11 namespace net { 11 namespace net {
12 class HttpRequestHeaders;
12 class URLRequest; 13 class URLRequest;
13 } 14 }
14 15
15 namespace data_reduction_proxy { 16 namespace data_reduction_proxy {
16 17
17 // Class responsible for deciding whether a request should be requested with low 18 // Class responsible for deciding whether a request should be requested with low
18 // fidelity (Lo-Fi) or not. Relies on the Lo-Fi mode state stored in the 19 // fidelity (Lo-Fi) or not. Relies on the Lo-Fi mode state stored in the
19 // request's content::ResourceRequestInfo, which must be fetched using 20 // request's content::ResourceRequestInfo, which must be fetched using
20 // content::ResourceRequestInfo::ForRequest. Owned by DataReductionProxyIOData 21 // content::ResourceRequestInfo::ForRequest. Lo-Fi mode will not be enabled for
21 // and should be called on the IO thread. 22 // requests that don't have a ResourceRequestInfo, such as background requests.
23 // Owned by DataReductionProxyIOData and should be called on the IO thread.
22 class ContentLoFiDecider : public LoFiDecider { 24 class ContentLoFiDecider : public LoFiDecider {
23 public: 25 public:
24 ContentLoFiDecider(); 26 ContentLoFiDecider();
25 ~ContentLoFiDecider() override; 27 ~ContentLoFiDecider() override;
26 28
29 // LoFiDecider implementation:
27 bool IsUsingLoFiMode(const net::URLRequest& request) const override; 30 bool IsUsingLoFiMode(const net::URLRequest& request) const override;
31 bool MaybeAddLoFiDirectiveToHeaders(
32 const net::URLRequest& request,
33 net::HttpRequestHeaders* headers) const override;
28 34
29 private: 35 private:
30 DISALLOW_COPY_AND_ASSIGN(ContentLoFiDecider); 36 DISALLOW_COPY_AND_ASSIGN(ContentLoFiDecider);
31 }; 37 };
32 38
33 } // namespace data_reduction_proxy 39 } // namespace data_reduction_proxy
34 40
35 #endif // COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_ H_ 41 #endif // COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_DECIDER_ H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698