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

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

Issue 317993003: Send an onInstalled event for shared module update (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 6 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/shared_module_service.cc
diff --git a/chrome/browser/extensions/shared_module_service.cc b/chrome/browser/extensions/shared_module_service.cc
index 22c9e25117e781595423f0f681d031c80e6ae2b0..05d28046ac345f41e3abad10e5834c5015d81887 100644
--- a/chrome/browser/extensions/shared_module_service.cc
+++ b/chrome/browser/extensions/shared_module_service.cc
@@ -104,7 +104,7 @@ SharedModuleService::ImportStatus SharedModuleService::SatisfyImports(
return status;
}
-scoped_ptr<const ExtensionSet> SharedModuleService::GetDependentExtensions(
+scoped_ptr<ExtensionSet> SharedModuleService::GetDependentExtensions(
const Extension* extension) {
scoped_ptr<ExtensionSet> dependents(new ExtensionSet());
@@ -127,7 +127,7 @@ scoped_ptr<const ExtensionSet> SharedModuleService::GetDependentExtensions(
}
}
}
- return dependents.PassAs<const ExtensionSet>();
+ return dependents.PassAs<ExtensionSet>();
}
void SharedModuleService::OnExtensionUninstalled(
@@ -149,7 +149,7 @@ void SharedModuleService::OnExtensionUninstalled(
registry->GetExtensionById(iter->extension_id,
ExtensionRegistry::EVERYTHING);
if (imported_module && imported_module->from_webstore()) {
- scoped_ptr<const ExtensionSet> dependents =
+ scoped_ptr<ExtensionSet> dependents =
GetDependentExtensions(imported_module);
if (dependents->is_empty()) {
service->UninstallExtension(iter->extension_id,
« no previous file with comments | « chrome/browser/extensions/shared_module_service.h ('k') | chrome/browser/extensions/test_extension_system.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698