Move extension/webstore-related URL getters to the extension_urls namespace.

chrome/common/extensions/extension_constants.h

Index: chrome/common/extensions/extension_constants.h
diff --git a/chrome/common/extensions/extension_constants.h b/chrome/common/extensions/extension_constants.h
index 976e4c181e5d9ef7688cd47235a256079b8cc026..17aa0d9c4ede39c67958e599d88426ede477e7c7 100644
--- a/chrome/common/extensions/extension_constants.h
+++ b/chrome/common/extensions/extension_constants.h
@@ -9,6 +9,7 @@
 #include <string>
 
 #include "base/basictypes.h"
+#include "googleurl/src/gurl.h"
 
 // Keys used in JSON representation of extensions.
 namespace extension_manifest_keys {
@@ -268,6 +269,26 @@ namespace extension_manifest_errors {
 }  // namespace extension_manifest_errors
 
 namespace extension_urls {
+  // Returns the URL prefix for the extension/apps gallery. Can be set via the
+  // --apps-gallery-url switch. The URL returned will not contain a trailing
+  // slash. Do not use this as a prefix/extent for the store.  Instead see
+  // ExtensionService::GetWebStoreApp or
+  // ExtensionService::IsDownloadFromGallery
+  std::string GetWebstoreLaunchURL();
+
+  // Returns the URL prefix for an item in the extension/app gallery. This URL
+  // will contain a trailing slash and should be concatenated with an item ID
+  // to get the item detail URL.
+  std::string GetWebstoreItemDetailURLPrefix();
+
+  // Return the update URL used by gallery/webstore extensions/apps.

[asargent_no_longer_on_chrome, 2011/08/25 00:05:09]

Can you add a note to this comment that |secure| might be ignored if
--apps-gallery-update-url is passed on the command line?

[Mihai Parparita -not on Chrome, 2011/08/25 00:46:29]

Done.

+  GURL GetWebstoreUpdateUrl(bool secure);
+
+  // Return the URL for an extension/app's .crx file that is hosted by the
+  // webstore.
+  GURL GetWebstoreInstallUrl(const std::string& extension_id,
+                             const std::string& locale);
+
   // The greatest common prefixes of the main extensions gallery's browse and
   // download URLs.
   extern const char* kGalleryBrowsePrefix;
@@ -420,18 +441,6 @@ namespace extension_misc {
     UNLOAD_REASON_UPDATE,     // Extension is being updated to a newer version.
     UNLOAD_REASON_UNINSTALL,  // Extension is being uninstalled.
   };
-
-  // Returns the URL prefix for the extension/apps gallery. Can be set via the
-  // --apps-gallery-url switch. The URL returned will not contain a trailing
-  // slash. Do not use this as a prefix/extent for the store.  Instead see
-  // ExtensionService::GetWebStoreApp or
-  // ExtensionService::IsDownloadFromGallery
-  std::string GetWebstoreLaunchURL();
-
-  // Returns the URL prefix for an item in the extension/app gallery. This URL
-  // will contain a trailing slash and should be concatenated with an item ID
-  // to get the item detail URL.
-  std::string GetWebstoreItemDetailURLPrefix();
-}  // extension_misc
+} // extension_misc
 
 #endif  // CHROME_COMMON_EXTENSIONS_EXTENSION_CONSTANTS_H_