Adding commandline switch and user pref for autofill server url.

chrome/browser/autofill/autofill_download_url.h

Index: chrome/browser/autofill/autofill_download_url.h
diff --git a/chrome/browser/autofill/autofill_download_url.h b/chrome/browser/autofill/autofill_download_url.h
new file mode 100644
index 0000000000000000000000000000000000000000..f1a481340aed8e782ca27c318f3a1b3375b3fd07
--- /dev/null
+++ b/chrome/browser/autofill/autofill_download_url.h
@@ -0,0 +1,31 @@
+// 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.
+
+#ifndef CHROME_BROWSER_AUTOFILL_AUTOFILL_DOWNLOAD_URL_H_
+#define CHROME_BROWSER_AUTOFILL_AUTOFILL_DOWNLOAD_URL_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+
+class GURL;
+class PrefServiceBase;
+
+class AutofillDownloadUrl {
+ public:
+  explicit AutofillDownloadUrl(PrefServiceBase* pref_service)
+      : pref_service_(pref_service) {}
+
+  GURL GetAutofillRequestUrl();

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

nit: s/Request/Query

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

Done.

+  GURL GetAutofillUploadUrl();

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

nit: Please leave a blank line after this one.

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

Done.

+ private:
+  void RegisterPreferences();

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

nit: If we keep this, the common style is for this to be a static method with
the signature "void RegisterPrefs(PrefService* prefs)" (or, in some cases,
local_state rather than prefs).  We shouldn't diverge from that style unless we
have good reason to do so -- do we?

+  std::string GetBaseAutofillUrl();
+
+  PrefServiceBase* pref_service_;

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

nit: If we keep this, "PrefServiceBase* const pref_service_;"

+  DISALLOW_COPY_AND_ASSIGN(AutofillDownloadUrl);
+};
+
+#endif  // CHROME_BROWSER_AUTOFILL_AUTOFILL_DOWNLOAD_URL_H_
+