Do not return a homepage URL for shared modules

chrome/common/extensions/manifest_url_handler.cc

 // 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/common/extensions/manifest_url_handler.h"
 
 #include "base/files/file_util.h"
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/generated_resources.h"
 #include "extensions/common/error_utils.h"
 #include "extensions/common/file_util.h"
 #include "extensions/common/manifest.h"
 #include "extensions/common/manifest_constants.h"
 #include "extensions/common/manifest_handlers/permissions_parser.h"
+#include "extensions/common/manifest_handlers/shared_module_info.h"
 #include "extensions/common/permissions/api_permission.h"
 #include "extensions/common/permissions/api_permission_set.h"
 #include "ui/base/l10n/l10n_util.h"
 
 #if defined(USE_AURA)
 #include "ui/keyboard/keyboard_constants.h"
 #endif
 
 namespace extensions {
 
@@ skipping 16 matching lines @@
 // static
 const GURL& ManifestURL::GetDevToolsPage(const Extension* extension) {
   return GetManifestURL(extension, keys::kDevToolsPage);
 }
 
 // static
 const GURL ManifestURL::GetHomepageURL(const Extension* extension) {
   const GURL& homepage_url = GetManifestURL(extension, keys::kHomepageURL);
   if (homepage_url.is_valid())
     return homepage_url;
-  return UpdatesFromGallery(extension) ?
-      GURL(extension_urls::GetWebstoreItemDetailURLPrefix() + extension->id()) :
-      GURL::EmptyGURL();
+  bool use_webstore_url = UpdatesFromGallery(extension) &&
+                          !SharedModuleInfo::IsSharedModule(extension);
+  return use_webstore_url
+             ? GURL(extension_urls::GetWebstoreItemDetailURLPrefix() +
+                    extension->id())
+             : GURL::EmptyGURL();
 }
 
 // static
 const GURL& ManifestURL::GetUpdateURL(const Extension* extension) {
   return GetManifestURL(extension, keys::kUpdateURL);
 }
 
 // static
 bool ManifestURL::UpdatesFromGallery(const Extension* extension) {
   return extension_urls::IsWebstoreUpdateUrl(GetUpdateURL(extension));
@@ skipping 308 matching lines @@
   extension->SetManifestData(keys::kChromeURLOverrides,
                              url_overrides.release());
   return true;
 }
 
 const std::vector<std::string> URLOverridesHandler::Keys() const {
   return SingleKey(keys::kChromeURLOverrides);
 }
 
 }  // namespace extensions