Implement serial API for Windows.

chrome/browser/extensions/extension_service.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/extension_service.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 420 matching lines @@
 ExtensionService::~ExtensionService() {
   // No need to unload extensions here because they are profile-scoped, and the
   // profile is in the process of being deleted.
 
   ProviderCollection::const_iterator i;
   for (i = external_extension_providers_.begin();
        i != external_extension_providers_.end(); ++i) {
     ExternalExtensionProviderInterface* provider = i->get();
     provider->ServiceShutdown();
   }
-
-  if (api_resource_controller_) {
-    // If this check failed, then a unit test was using an APIResource but
-    // didn't provide the IO thread message loop needed for those resources to
-    // do their job (including destroying themselves at shutdown).
-    DCHECK(BrowserThread::IsMessageLoopValid(BrowserThread::IO));
-    BrowserThread::DeleteSoon(BrowserThread::IO, FROM_HERE,
-                              api_resource_controller_);
-  }
 }
 
 void ExtensionService::InitEventRoutersAfterImport() {
   RegisterForImportFinished();
 }
 
 void ExtensionService::RegisterForImportFinished() {
   if (!registrar_.IsRegistered(this, chrome::NOTIFICATION_IMPORT_FINISHED,
                                content::Source<Profile>(profile_))) {
     registrar_.Add(this, chrome::NOTIFICATION_IMPORT_FINISHED,
@@ skipping 2084 matching lines @@
   for (NaClModuleInfoList::iterator iter = nacl_module_list_.begin();
        iter != nacl_module_list_.end(); ++iter) {
     if (iter->url == url)
       return iter;
   }
   return nacl_module_list_.end();
 }
 
 extensions::APIResourceController*
 ExtensionService::api_resource_controller() {
-  // TODO(miket): Find a better place for this thing to live. It needs to be
-  // scoped such that it can be created and destroyed on the IO thread.
-  //
-  // To coexist with certain unit tests that don't have an IO thread message
+  // 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.
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-  if (!api_resource_controller_) {
-    api_resource_controller_ = new extensions::APIResourceController();
+  if (!api_resource_controller_.get()) {
+    api_resource_controller_.reset(new extensions::APIResourceController());
   }
-  return api_resource_controller_;
+  return api_resource_controller_.get();
 }