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

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

Issue 133273016: Fix extension install verification for v1 packaged apps (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/extensions/install_verifier.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index eb2342a4cf785e9dff4c644d8d50359ef09f101a..cc2a8d7487730b7c02401fb477f8341d38f66ae4 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -575,8 +575,7 @@ void ExtensionService::VerifyAllExtensions() {
i != all_extensions->end(); ++i) {
const Extension& extension = **i;
- if (extensions::ManifestURL::UpdatesFromGallery(&extension) &&
- extension.is_extension())
+ if (InstallVerifier::NeedsVerification(extension))
to_add.insert(extension.id());
}
extensions::ExtensionSystem::Get(profile_)->install_verifier()->AddMany(
@@ -2192,7 +2191,7 @@ void ExtensionService::AddNewOrUpdatedExtension(
blacklisted_for_malware,
page_ordinal);
delayed_installs_.Remove(extension->id());
- if (extensions::ManifestURL::UpdatesFromGallery(extension)) {
+ if (InstallVerifier::NeedsVerification(*extension)) {
extensions::ExtensionSystem::Get(profile_)->install_verifier()->Add(
extension->id(), InstallVerifier::AddResultCallback());
}
« no previous file with comments | « no previous file | chrome/browser/extensions/install_verifier.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698