Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1415)

Unified Diff: chrome/browser/extensions/extension_service.cc

Issue 8404007: Delete code for and references to mini-gallery and theme install bubbles. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index f3e4832487c8252475ded1095cf00e7775527e52..7385dc4071eea6f8009ca94ed0f578cfb50efc68 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -456,14 +456,6 @@ static const char* kAllowedDownloadURLPatterns[] = {
bool ExtensionService::IsDownloadFromGallery(const GURL& download_url,
const GURL& referrer_url) {
- // Special-case the themes mini-gallery.
- // TODO(erikkay) When that gallery goes away, remove this code.
- if (IsDownloadFromMiniGallery(download_url) &&
- StartsWithASCII(referrer_url.spec(),
- extension_urls::kMiniGalleryBrowsePrefix, false)) {
- return true;
- }
-
const Extension* download_extension = GetExtensionByWebExtent(download_url);
const Extension* referrer_extension = GetExtensionByWebExtent(referrer_url);
const Extension* webstore_app = GetWebStoreApp();
@@ -515,12 +507,6 @@ bool ExtensionService::IsDownloadFromGallery(const GURL& download_url,
return (referrer_valid && download_valid);
}
-bool ExtensionService::IsDownloadFromMiniGallery(const GURL& download_url) {
- return StartsWithASCII(download_url.spec(),
- extension_urls::kMiniGalleryDownloadPrefix,
- false); // case_sensitive
-}
-
const Extension* ExtensionService::GetInstalledApp(const GURL& url) {
// Check for hosted app.
const Extension* app = GetExtensionByWebExtent(url);

Powered by Google App Engine
This is Rietveld 408576698