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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h

Issue 981633002: Created new URLRequestContext for secure proxy check (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed the tests Created 5 years, 9 months 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 side-by-side diff with in-line comments
Download patch
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h
index 5343635bebb1ae74a9488036be7ebbaa1b81b1fd..1ca7b08ecf68c2ae5ca5874d67467d3a9466c73e 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h
@@ -57,8 +57,10 @@ class DataReductionProxyService : public base::NonThreadSafe,
// Requests the given |secure_proxy_check_url|. Upon completion, returns the
// results to the caller via the |fetcher_callback|. Virtualized for unit
// testing.
- virtual void SecureProxyCheck(const GURL& secure_proxy_check_url,
- FetcherResponseCallback fetcher_callback);
+ virtual void SecureProxyCheck(
+ const GURL& secure_proxy_check_url,
+ FetcherResponseCallback fetcher_callback,
+ scoped_refptr<base::SequencedTaskRunner> io_task_runner);
// Constructs statistics prefs. This should not be called if a valid
// statistics prefs is passed into the constructor.
@@ -84,6 +86,9 @@ class DataReductionProxyService : public base::NonThreadSafe,
const GURL& secure_proxy_check_url);
private:
+ void SecureProxyCheckOnIOThread(const GURL& secure_proxy_check_url,
bengr 2015/03/16 23:21:06 Please add a comment.
tbansal1 2015/03/19 22:43:57 Done.
+ FetcherResponseCallback fetcher_callback);
+
// net::URLFetcherDelegate:
void OnURLFetchComplete(const net::URLFetcher* source) override;

Powered by Google App Engine
This is Rietveld 408576698