Add optional homepage_url to manifest and use it in place of Gallery URL in UI.

BUG=28907
TEST=Added homepage_url to manifest, check that it replaces gallery URL link.

Review URL: http://codereview.chromium.org/3767013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63749 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 8 insertions, 10 deletions

diff --git a/chrome/browser/extensions/extension_context_menu_model.cc b/chrome/browser/extensions/extension_context_menu_model.cc
index 736b387..6432f9d 100644
--- a/chrome/browser/extensions/extension_context_menu_model.cc
+++ b/chrome/browser/extensions/extension_context_menu_model.cc
@@ -80,11 +80,9 @@ bool ExtensionContextMenuModel::IsCommandIdEnabled(int command_id) const {
   if (command_id == CONFIGURE) {
     return extension->options_url().spec().length() > 0;
   } else if (command_id == NAME) {
-    // The NAME links to the gallery page, which only makes sense if Google is
-    // hosting the extension. For other 3rd party extensions we don't have a
-    // homepage url, so we just disable this menu item on those cases, at least
-    // for now.
-    return extension->GalleryUrl().is_valid();
+    // The NAME links to the Homepage URL. If the extension doesn't have a
+    // homepage, we just disable this menu item.
+    return extension->GetHomepageURL().is_valid();
   } else if (command_id == INSPECT_POPUP) {
     TabContents* contents = browser_->GetSelectedTabContents();
     if (!contents)
@@ -107,7 +105,7 @@ void ExtensionContextMenuModel::ExecuteCommand(int command_id) {
 
   switch (command_id) {
     case NAME: {
-      browser_->OpenURL(extension->GalleryUrl(), GURL(),
+      browser_->OpenURL(extension->GetHomepageURL(), GURL(),
                         NEW_FOREGROUND_TAB, PageTransition::LINK);
       break;
     }
diff --git a/chrome/browser/extensions/extensions_ui.cc b/chrome/browser/extensions/extensions_ui.cc
index 586ec6f..45b1cb6 100644
--- a/chrome/browser/extensions/extensions_ui.cc
+++ b/chrome/browser/extensions/extensions_ui.cc
@@ -840,7 +840,7 @@ DictionaryValue* ExtensionsDOMHandler::CreateExtensionDetailValue(
   extension_data->Set("views", views);
   extension_data->SetBoolean("hasPopupAction",
       extension->browser_action() || extension->page_action());
-  extension_data->SetString("galleryUrl", extension->GalleryUrl().spec());
+  extension_data->SetString("homepageUrl", extension->GetHomepageURL().spec());
 
   return extension_data;
 }
diff --git a/chrome/browser/resources/extensions_ui.html b/chrome/browser/resources/extensions_ui.html
index 4a7c035..cf469bb 100644
--- a/chrome/browser/resources/extensions_ui.html
+++ b/chrome/browser/resources/extensions_ui.html
@@ -838,12 +838,12 @@ document.addEventListener('DOMContentLoaded', requestExtensionsData);
           </td>
           <td valign="top">
             <div>
-              <a jsdisplay="galleryUrl.length > 0"
-                 jsvalues=".href:galleryUrl">
+              <a jsdisplay="homepageUrl.length > 0"
+                 jsvalues=".href:homepageUrl">
               <span class="extension-name"
                     jscontent="name">EXTENSION NAME</span></a>
               <span class="extension-name"
-                    jsdisplay="galleryUrl.length == 0"
+                    jsdisplay="homepageUrl.length == 0"
                     jscontent="name">EXTENSION NAME</span>
               - <span i18n-content="extensionVersion">VERSION</span>
               <span jscontent="version">x.x.x.x</span>