Delete UDPClientSocket on same thread as creation. Also refactor.

chrome/browser/extensions/extension_service.h

Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index 67b0f4dcc6a9efbc8c16dec0d7746f0297a31b37..305b2688dd20edeed72663764703475ad3de7dc5 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -75,6 +75,7 @@ class ExtensionInputMethodEventRouter;
 namespace extensions {
 class ComponentLoader;
 class SettingsFrontend;
+class SocketController;
 }
 
 // This is an interface class to encapsulate the dependencies that
@@ -569,6 +570,10 @@ class ExtensionService
     return &extension_warnings_;
   }
 
+  extensions::SocketController* socket_controller() {
+    return socket_controller_;
+  }
+
  private:
   // Bundle of type (app or extension)-specific sync stuff.
   struct SyncBundle {
@@ -685,6 +690,9 @@ class ExtensionService
 
   NaClModuleInfoList::iterator FindNaClModule(const GURL& url);
 
+  static void DeleteSocketController(
+      extensions::SocketController* socket_controller);
+
   // The profile this ExtensionService is part of.
   Profile* profile_;
 
@@ -819,6 +827,8 @@ class ExtensionService
   // Contains an entry for each warning that shall be currently shown.
   ExtensionWarningSet extension_warnings_;
 
+  extensions::SocketController* socket_controller_;

[Aaron Boodman, 2011/12/07 23:16:22]

Add a comment explaining why this is a bare pointer.

+
   extensions::ProcessMap process_map_;
 
   FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,