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

Unified Diff: components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc

Issue 1248773002: Corrected the Data Reduction Proxy URL for configuration retrieval. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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
« no previous file with comments | « components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc b/components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc
index 7d9ced0c7a2bb58e95a0ee6c230e17551445df68..e4234254894c48136779e9022240f4635c4988ec 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc
@@ -382,34 +382,28 @@ TEST_F(DataReductionProxyParamsTest, GetConfigServiceURL) {
GURL expected;
} tests[] = {
{
- "Nothing set", "", "", GURL("https://datasaver.googleapis.com"),
+ "Nothing set", "", "",
+ GURL("https://datasaver.googleapis.com/v1/clientConfigs"),
},
{
- "Only command line set",
- "http://commandline.config-service/",
- "",
- GURL("http://commandline.config-service/"),
+ "Only command line set", "http://commandline.config-service/", "",
+ GURL("http://commandline.config-service/"),
},
{
- "Enabled group", "", "Enabled", GURL("http://enabled.config-service/"),
+ "Enabled group", "", "Enabled",
+ GURL("http://enabled.config-service/"),
},
{
- "Disabled group",
- "",
- "Disabled",
- GURL("http://disabled.config-service/"),
+ "Disabled group", "", "Disabled",
+ GURL("http://disabled.config-service/"),
},
{
- "Alternate enabled group",
- "",
- "EnabledOther",
- GURL("http://other.config-service/"),
+ "Alternate enabled group", "", "EnabledOther",
+ GURL("http://other.config-service/"),
},
{
- "Command line precedence",
- "http://commandline.config-service/",
- "Enabled",
- GURL("http://commandline.config-service/"),
+ "Command line precedence", "http://commandline.config-service/",
+ "Enabled", GURL("http://commandline.config-service/"),
},
};
« no previous file with comments | « components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698