Adding commandline switch and user pref for autofill server url.

chrome/browser/autofill/autofill_download_url_unittest.cc

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/autofill/autofill_download_url.h"
+#include "chrome/test/base/testing_pref_service.h"
+#include "googleurl/src/gurl.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using testing::HasSubstr;
+using testing::Not;
+
+class AutofillDownloadUrlTest : public testing::Test {
+ public:
+  AutofillDownloadUrlTest() {}
+
+ protected:
+  virtual void SetUp() {
+    pref_service_.reset(new TestingPrefService());
+  }
+
+  scoped_ptr<TestingPrefService> pref_service_;
+};
+
+TEST_F(AutofillDownloadUrlTest, CheckDefaultUrls) {
+  std::string request_url =
+      AutofillDownloadUrl(pref_service_.get()).
+      GetAutofillRequestUrl().spec();
+  EXPECT_THAT(request_url, HasSubstr("clients1.google.com"));
+  EXPECT_THAT(request_url, HasSubstr("/tbproxy/"));
+  EXPECT_THAT(request_url, HasSubstr("/af/"));
+  EXPECT_THAT(request_url, HasSubstr("/query"));
+  EXPECT_THAT(request_url, Not(HasSubstr("/upload")));
+  EXPECT_THAT(request_url, HasSubstr("?client="));

[Ilya Sherman, 2012/10/25 22:52:02]

nit: This does look tidy, but why not just check that request_url contains the
prefix "https://clients1.google.com/tbproxy/af/query?client="?  That's a
slightly stronger test, as it doesn't allow the URL fragments to be re-arranged
arbitrarily w.r.t. one another.

[ahutter, 2012/10/25 23:56:03]

Done.

+
+

[Ilya Sherman, 2012/10/25 22:52:02]

nit: Spurious newline.

[ahutter, 2012/10/25 23:56:03]

Done.

+  std::string upload_url =
+      AutofillDownloadUrl(pref_service_.get()).
+      GetAutofillUploadUrl().spec();
+  EXPECT_THAT(upload_url, HasSubstr("clients1.google.com"));
+  EXPECT_THAT(upload_url, HasSubstr("/tbproxy/"));
+  EXPECT_THAT(upload_url, HasSubstr("/af/"));
+  EXPECT_THAT(upload_url, HasSubstr("/upload"));
+  EXPECT_THAT(upload_url, Not(HasSubstr("/query")));
+  EXPECT_THAT(upload_url, HasSubstr("?client="));
+}
+