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

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

Issue 10777003: Refactor APIResourceController to ProfileKeyedService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge/antony/fix build. Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/extension_service.h ('k') | chrome/browser/extensions/extension_system.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 2eff617ac029c5c9343cdf57911c671d6bf4e4e7..2b4ada6f159035c738572a22a4653cf37f11ee46 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -29,7 +29,6 @@
#include "chrome/browser/bookmarks/bookmark_extension_api.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/chrome_plugin_service_filter.h"
-#include "chrome/browser/extensions/api/api_resource_controller.h"
#include "chrome/browser/extensions/api/cookies/cookies_api.h"
#include "chrome/browser/extensions/api/declarative/rules_registry_service.h"
#include "chrome/browser/extensions/api/managed_mode/managed_mode_api.h"
@@ -340,7 +339,6 @@ ExtensionService::ExtensionService(Profile* profile,
app_sync_bundle_(ALLOW_THIS_IN_INITIALIZER_LIST(this)),
extension_sync_bundle_(ALLOW_THIS_IN_INITIALIZER_LIST(this)),
extension_warnings_(profile),
- api_resource_controller_(NULL),
app_shortcut_manager_(profile) {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ -2545,18 +2543,3 @@ ExtensionService::NaClModuleInfoList::iterator
}
return nacl_module_list_.end();
}
-
-extensions::APIResourceController*
-ExtensionService::api_resource_controller() {
- // TODO(miket): Find a better place for this thing to live. Like every other
- // piece of baggage on ExtensionService, it's scoped along with a Profile.
-
- // To coexist with certain unit tests that don't have a work-thread message
- // loop available at ExtensionService shutdown, we lazy-initialize this
- // object so that those cases neither create nor destroy an
- // APIResourceController.
- if (!api_resource_controller_.get()) {
- api_resource_controller_.reset(new extensions::APIResourceController());
- }
- return api_resource_controller_.get();
-}
« no previous file with comments | « chrome/browser/extensions/extension_service.h ('k') | chrome/browser/extensions/extension_system.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698