Refactor APIResourceController to ProfileKeyedService.

chrome/browser/extensions/extension_system.cc

Index: chrome/browser/extensions/extension_system.cc
diff --git a/chrome/browser/extensions/extension_system.cc b/chrome/browser/extensions/extension_system.cc
index bb398542e6b63a287f532ab3cd1a98b7b6443ba2..900eab4d5a0185939dbba4a0aa8ad497370547dc 100644
--- a/chrome/browser/extensions/extension_system.cc
+++ b/chrome/browser/extensions/extension_system.cc
@@ -9,10 +9,13 @@
 #include "base/file_path.h"
 #include "base/string_tokenizer.h"
 #include "chrome/browser/browser_process.h"
-#include "chrome/browser/extensions/component_loader.h"
 #include "chrome/browser/content_settings/cookie_settings.h"
 #include "chrome/browser/extensions/api/alarms/alarm_manager.h"
 #include "chrome/browser/extensions/api/declarative/rules_registry_service.h"
+#include "chrome/browser/extensions/api/serial/serial_connection.h"
+#include "chrome/browser/extensions/api/socket/socket.h"
+#include "chrome/browser/extensions/api/usb/usb_device_resource.h"
+#include "chrome/browser/extensions/component_loader.h"
 #include "chrome/browser/extensions/extension_devtools_manager.h"
 #include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/extensions/extension_event_router.h"
@@ -278,8 +281,17 @@ void ExtensionSystemImpl::Init(bool extensions_enabled) {
   shared_->InitInfoMap();
 
   extension_process_manager_.reset(ExtensionProcessManager::Create(profile_));
-  alarm_manager_.reset(new AlarmManager(profile_,
-                                        &base::Time::Now));
+  alarm_manager_.reset(new AlarmManager(profile_, &base::Time::Now));
+
+  serial_connection_manager_.reset(new ApiResourceManager<SerialConnection>());
+  serial_connection_manager_->set_thread_id(BrowserThread::FILE);

[asargent_no_longer_on_chrome, 2012/07/16 18:08:25]

Had you considered making the thread id either an argument to the constructor or
one of the template arguments? Then when creating these guys you'd do something
like: 


serial_connection_manager.reset(
    new ApiResourceManager<SerialConnection>(
                           BrowserThread::FILE))

or

serial_connection_manager.reset(
    new ApiResourceManager<SerialConnection,
                           BrowserThread::FILE>())

It seems like ApiResourceManager is designed with the assumption that the thread
id should not be changed once set, and this would help you to enforce that.

[miket_OOO, 2012/07/16 19:46:02]

That's a good idea. I had something in my head about not everyone wanting to
specify the thread, but that's a YAGNI.

+
+  socket_manager_.reset(new ApiResourceManager<Socket>());
+  socket_manager_->set_thread_id(BrowserThread::IO);
+
+  usb_device_resource_manager_.reset(
+      new ApiResourceManager<UsbDeviceResource>());
+  usb_device_resource_manager_->set_thread_id(BrowserThread::IO);
 
   shared_->Init(extensions_enabled);
 }
@@ -334,6 +346,20 @@ RulesRegistryService* ExtensionSystemImpl::rules_registry_service() {
   return shared_->rules_registry_service();
 }
 
+ApiResourceManager<SerialConnection>*
+ExtensionSystemImpl::serial_connection_manager() {
+  return serial_connection_manager_.get();
+}
+
+ApiResourceManager<Socket>*ExtensionSystemImpl::socket_manager() {
+  return socket_manager_.get();
+}
+
+ApiResourceManager<UsbDeviceResource>*
+ExtensionSystemImpl::usb_device_resource_manager() {
+  return usb_device_resource_manager_.get();
+}
+
 void ExtensionSystemImpl::RegisterExtensionWithRequestContexts(
     const Extension* extension) {
   base::Time install_time;