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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc

Issue 286013002: Added alternative configuration for the data reduction proxy (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: DCHECK fix Created 6 years, 7 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/browser/data_reduction_proxy_config_service_unittest.cc
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc b/components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc
index 657308a181596a3a58ae35f84661b5877be84df2..d5624f09608719b38423ebe4bc21907bba9b2469 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc
@@ -207,7 +207,8 @@ TEST_F(DataReductionProxyConfigServiceTest, TrackerEnable) {
tracker.Enable(false,
false,
"https://foo.com:443",
- "http://bar.com:80");
+ "http://bar.com:80",
+ "");
task_runner_->RunUntilIdle();
Mock::VerifyAndClearExpectations(&observer);
@@ -231,7 +232,8 @@ TEST_F(DataReductionProxyConfigServiceTest, TrackerEnableRestricted) {
tracker.Enable(true,
false,
"https://foo.com:443",
- "http://bar.com:80");
+ "http://bar.com:80",
+ "");
task_runner_->RunUntilIdle();
Mock::VerifyAndClearExpectations(&observer);

Powered by Google App Engine
This is Rietveld 408576698