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

Side by Side Diff: chrome/browser/extensions/shared_module_service.cc

Issue 398083002: Add "UninstallReason" parameter to ExtensionRegistryObserver::OnExtensionUninstalled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix header file. Created 6 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/shared_module_service.h" 5 #include "chrome/browser/extensions/shared_module_service.h"
6 6
7 #include <set> 7 #include <set>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/version.h" 10 #include "base/version.h"
11 #include "chrome/browser/extensions/extension_service.h" 11 #include "chrome/browser/extensions/extension_service.h"
12 #include "chrome/browser/extensions/pending_extension_manager.h" 12 #include "chrome/browser/extensions/pending_extension_manager.h"
13 #include "chrome/common/extensions/extension_constants.h" 13 #include "chrome/common/extensions/extension_constants.h"
14 #include "extensions/browser/extension_registry.h" 14 #include "extensions/browser/extension_registry.h"
15 #include "extensions/browser/extension_system.h" 15 #include "extensions/browser/extension_system.h"
16 #include "extensions/browser/uninstall_reason.h"
16 #include "extensions/common/extension.h" 17 #include "extensions/common/extension.h"
17 18
18 namespace extensions { 19 namespace extensions {
19 20
20 namespace { 21 namespace {
21 22
22 typedef std::vector<SharedModuleInfo::ImportInfo> ImportInfoVector; 23 typedef std::vector<SharedModuleInfo::ImportInfo> ImportInfoVector;
23 typedef std::list<SharedModuleInfo::ImportInfo> ImportInfoList; 24 typedef std::list<SharedModuleInfo::ImportInfo> ImportInfoList;
24 25
25 } // namespace 26 } // namespace
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 } 160 }
160 161
161 std::vector<std::string>::const_iterator shared_modules_iter; 162 std::vector<std::string>::const_iterator shared_modules_iter;
162 for (shared_modules_iter = shared_modules.begin(); 163 for (shared_modules_iter = shared_modules.begin();
163 shared_modules_iter != shared_modules.end(); 164 shared_modules_iter != shared_modules.end();
164 shared_modules_iter++) { 165 shared_modules_iter++) {
165 if (used_shared_modules.count(*shared_modules_iter)) 166 if (used_shared_modules.count(*shared_modules_iter))
166 continue; 167 continue;
167 service->UninstallExtension( 168 service->UninstallExtension(
168 *shared_modules_iter, 169 *shared_modules_iter,
169 ExtensionService::UNINSTALL_REASON_ORPHANED_SHARED_MODULE, 170 extensions::UNINSTALL_REASON_ORPHANED_SHARED_MODULE,
170 NULL); // Ignore error. 171 NULL); // Ignore error.
171 } 172 }
172 } 173 }
173 174
174 void SharedModuleService::OnExtensionInstalled( 175 void SharedModuleService::OnExtensionInstalled(
175 content::BrowserContext* browser_context, 176 content::BrowserContext* browser_context,
176 const Extension* extension, 177 const Extension* extension,
177 bool is_update) { 178 bool is_update) {
178 if (is_update) 179 if (is_update)
179 PruneSharedModules(); 180 PruneSharedModules();
180 } 181 }
181 182
182 void SharedModuleService::OnExtensionUninstalled( 183 void SharedModuleService::OnExtensionUninstalled(
183 content::BrowserContext* browser_context, 184 content::BrowserContext* browser_context,
184 const Extension* extension) { 185 const Extension* extension,
186 extensions::UninstallReason reason) {
185 PruneSharedModules(); 187 PruneSharedModules();
186 } 188 }
187 189
188 } // namespace extensions 190 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/shared_module_service.h ('k') | chrome/browser/extensions/shared_module_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698