Adding commandline switch and user pref for autofill server url.

chrome/browser/autofill/autofill_download_url_unittest.cc

Index: chrome/browser/autofill/autofill_download_url_unittest.cc
diff --git a/chrome/browser/autofill/autofill_download_url_unittest.cc b/chrome/browser/autofill/autofill_download_url_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..305bfdaa2b4fef7406565488d43e5be5b68d8e58
--- /dev/null
+++ b/chrome/browser/autofill/autofill_download_url_unittest.cc
@@ -0,0 +1,30 @@
+// 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 "googleurl/src/gurl.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using testing::StartsWith;
+
+class AutofillDownloadUrlTest : public testing::Test {
+ public:
+  AutofillDownloadUrlTest() {}
+};

[Ilya Sherman, 2012/10/26 00:10:49]

nit: I believe that you can omit this class, and just use TEST rather than
TEST_F below.

[ahutter, 2012/10/29 15:38:13]

Done.

+
+TEST_F(AutofillDownloadUrlTest, CheckDefaultUrls) {
+  std::string query_url =
+      AutofillDownloadUrl().
+      GetAutofillQueryUrl().spec();
+  EXPECT_THAT(query_url,
+              StartsWith("https://clients1.google.com/tbproxy/af/query?client="));

[Ilya Sherman, 2012/10/26 00:10:49]

nit: 80-col

[ahutter, 2012/10/29 15:38:13]

Just wondering how you catch this?  Is there a presubmit script I'm not running?

+
+  std::string upload_url =
+      AutofillDownloadUrl().
+      GetAutofillUploadUrl().spec();
+  EXPECT_THAT(upload_url,
+              StartsWith("https://clients1.google.com/tbproxy/af/upload?client="));

[Ilya Sherman, 2012/10/26 00:10:49]

nit: 80-col

+}
+