Check for *.googlezip.net proxies when migrating proxy pref for DRP.

chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc

Index: chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc
diff --git a/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc b/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc
index 9d0d21343b5b0c9d7d3614cefc3819e480a0ee5c..469babb9e1d0e4bf776e9337cdb87a986cea1858 100644
--- a/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc
+++ b/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings_unittest.cc
@@ -74,35 +74,78 @@ TEST_F(DataReductionProxyChromeSettingsTest, MigrateSystemProxy) {
 }
 
 TEST_F(DataReductionProxyChromeSettingsTest, MigrateDataReductionProxy) {
-  dict_->SetString("mode", "fixed_servers");
-  dict_->SetString("server", "http=https://proxy.googlezip.net");
-  test_context_->pref_service()->Set(prefs::kProxy, *dict_.get());
-  EXPECT_CALL(*config_, ContainsDataReductionProxy(_)).Times(1)
-    .WillOnce(Return(true));
-
-  drp_chrome_settings_->MigrateDataReductionProxyOffProxyPrefs(
-      test_context_->pref_service());
+  const std::string kTestServers[] = {"http=http://proxy.googlezip.net",
+                                      "http=https://my-drp.org",
+                                      "https=https://tunneldrp.com"};
+
+  for (const std::string& test_server : kTestServers) {
+    dict_.reset(new base::DictionaryValue());
+    dict_->SetString("mode", "fixed_servers");
+    dict_->SetString("server", test_server);
+    test_context_->pref_service()->Set(prefs::kProxy, *dict_.get());
+    EXPECT_CALL(*config_, ContainsDataReductionProxy(_))
+        .Times(1)
+        .WillOnce(Return(true));
+
+    drp_chrome_settings_->MigrateDataReductionProxyOffProxyPrefs(
+        test_context_->pref_service());
+
+    EXPECT_EQ(NULL, test_context_->pref_service()->GetUserPref(prefs::kProxy));
+  }
+}
 
-  EXPECT_EQ(NULL, test_context_->pref_service()->GetUserPref(prefs::kProxy));
+TEST_F(DataReductionProxyChromeSettingsTest,
+       MigrateGooglezipDataReductionProxy) {
+  const std::string kTestServers[] = {
+      "http=http://proxy-dev.googlezip.net",
+      "http=https://arbitraryprefix.googlezip.net",
+      "https=https://tunnel.googlezip.net"};
+
+  for (const std::string& test_server : kTestServers) {
+    dict_.reset(new base::DictionaryValue());
+    // The proxy pref is set to a Data Reduction Proxy that doesn't match the
+    // currently configured DRP, but the pref should still be cleared.
+    dict_->SetString("mode", "fixed_servers");
+    dict_->SetString("server", test_server);
+    test_context_->pref_service()->Set(prefs::kProxy, *dict_.get());
+    EXPECT_CALL(*config_, ContainsDataReductionProxy(_))
+        .Times(1)
+        .WillOnce(Return(false));
+
+    drp_chrome_settings_->MigrateDataReductionProxyOffProxyPrefs(
+        test_context_->pref_service());
+
+    EXPECT_EQ(NULL, test_context_->pref_service()->GetUserPref(prefs::kProxy));
+  }
 }
 
 TEST_F(DataReductionProxyChromeSettingsTest, MigrateIgnoreOtherProxy) {
-  dict_->SetString("mode", "fixed_servers");
-  dict_->SetString("server", "http=https://youtube.com");
-  test_context_->pref_service()->Set(prefs::kProxy, *dict_.get());
-  EXPECT_CALL(*config_, ContainsDataReductionProxy(_)).Times(1)
-    .WillOnce(Return(false));
-
-  drp_chrome_settings_->MigrateDataReductionProxyOffProxyPrefs(
-      test_context_->pref_service());
-
-  base::DictionaryValue* value =
-      (base::DictionaryValue*)test_context_->pref_service()->GetUserPref(
-          prefs::kProxy);
-  std::string mode;
-  EXPECT_TRUE(value->GetString("mode", &mode));
-  EXPECT_EQ("fixed_servers", mode);
-  std::string server;
-  EXPECT_TRUE(value->GetString("server", &server));
-  EXPECT_EQ("http=https://youtube.com", server);
+  const std::string kTestServers[] = {
+      "http=https://youtube.com",
+      "http=http://googlezip.net",
+      "http=http://thisismyproxynotgooglezip.net",
+      "https=http://arbitraryprefixgooglezip.net"};
+
+  for (const std::string& test_server : kTestServers) {
+    dict_.reset(new base::DictionaryValue());
+    dict_->SetString("mode", "fixed_servers");
+    dict_->SetString("server", test_server);
+    test_context_->pref_service()->Set(prefs::kProxy, *dict_.get());
+    EXPECT_CALL(*config_, ContainsDataReductionProxy(_))
+        .Times(1)
+        .WillOnce(Return(false));
+
+    drp_chrome_settings_->MigrateDataReductionProxyOffProxyPrefs(
+        test_context_->pref_service());
+
+    base::DictionaryValue* value =
+        (base::DictionaryValue*)test_context_->pref_service()->GetUserPref(
+            prefs::kProxy);
+    std::string mode;
+    EXPECT_TRUE(value->GetString("mode", &mode));
+    EXPECT_EQ("fixed_servers", mode);
+    std::string server;
+    EXPECT_TRUE(value->GetString("server", &server));
+    EXPECT_EQ(test_server, server);
+  }
 }