Move sockets APIs out of src/chrome

apps/shell/browser/shell_extensions_browser_client.cc

Index: apps/shell/browser/shell_extensions_browser_client.cc
diff --git a/apps/shell/browser/shell_extensions_browser_client.cc b/apps/shell/browser/shell_extensions_browser_client.cc
index 8089b5e8cb63d11ac8e0275caa8c0f318c85956d..d6f2e5ec331721028848c9cfc618726dbc43085c 100644
--- a/apps/shell/browser/shell_extensions_browser_client.cc
+++ b/apps/shell/browser/shell_extensions_browser_client.cc
@@ -14,6 +14,7 @@
 #include "components/user_prefs/user_prefs.h"
 #include "extensions/browser/app_sorting.h"
 #include "extensions/browser/extension_prefs.h"
+#include "extensions/common/api/generated_api.h"
 
 using content::BrowserContext;
 
@@ -157,8 +158,9 @@ ShellExtensionsBrowserClient::GetExtensionSystemFactory() {
 
 void ShellExtensionsBrowserClient::RegisterExtensionFunctions(
     ExtensionFunctionRegistry* registry) const {
-  // TODO(rockot): Remove this once we have enough APIs moved out of src/chrome
-  // See http://crbug.com/349042
+  extensions::core_api::GeneratedFunctionRegistry::RegisterAll(registry);

[Yoyo Zhou, 2014/03/06 23:50:08]

I agree, it's not ideally named right now. But I don't expect renaming
extensions::api to be a fun chore.

extensions::api and chrome_api, I think, would be my preferred final names (for
extensions::core_api and extensions::api). Then each GeneratedFunctionRegistry
belongs to a particular namespace.

[Ken Rockot(use gerrit already), 2014/03/07 00:06:36]

Great, I think we have a quorum on this. Bug filed http://crbug.com/350169

+  // TODO(rockot): Remove dependency on src/chrome once we have some core APIs
+  // moved out. See http://crbug.com/349042.
   extensions::api::GeneratedFunctionRegistry::RegisterAll(registry);
 }