Reland: Link the name of the extension on the management page to the...

chrome/browser/extensions/extension_context_menu_model.cc

 // Copyright (c) 2010 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/extensions/extension_context_menu_model.h"
 
 #include "app/l10n_util.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_tabs_module.h"
@@ skipping 56 matching lines @@
 }
 
 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_->update_url().DomainIs("google.com");
+    return extension_->GalleryUrl().is_valid();
   } else if (command_id == INSPECT_POPUP) {
     TabContents* contents = browser_->GetSelectedTabContents();
     if (!contents)
       return false;
 
     return extension_action_->HasPopup(ExtensionTabUtil::GetTabId(contents));
   }
   return true;
 }
 
 bool ExtensionContextMenuModel::GetAcceleratorForCommandId(
     int command_id, menus::Accelerator* accelerator) {
   return false;
 }
 
 void ExtensionContextMenuModel::ExecuteCommand(int command_id) {
   switch (command_id) {
     case NAME: {
-      GURL url(std::string(extension_urls::kGalleryBrowsePrefix) +
-               std::string("/detail/") + extension_->id());
-      browser_->OpenURL(url, GURL(), NEW_FOREGROUND_TAB, PageTransition::LINK);
+      browser_->OpenURL(extension_->GalleryUrl(), GURL(),
+                        NEW_FOREGROUND_TAB, PageTransition::LINK);
       break;
     }
     case CONFIGURE:
       DCHECK(!extension_->options_url().is_empty());
       profile_->GetExtensionProcessManager()->OpenOptionsPage(extension_,
                                                               browser_);
       break;
     case DISABLE: {
       ExtensionsService* extension_service = profile_->GetExtensionsService();
       extension_service->DisableExtension(extension_->id());
@@ skipping 25 matching lines @@
 
   std::string id = extension_->id();
   profile_->GetExtensionsService()->UninstallExtension(id, false);
 
   Release();
 }
 
 void ExtensionContextMenuModel::InstallUIAbort() {
   Release();
 }