Move ExtensionsProcessManager to src/extensions, part 2

extensions/browser/extensions_browser_client.cc

Index: extensions/browser/extensions_browser_client.cc
diff --git a/extensions/browser/extensions_browser_client.cc b/extensions/browser/extensions_browser_client.cc
index 2724d476235a46d6e1236a341110a8a9d047e592..ce89b9c9452f9783d98b057191acc9a10e0edde5 100644
--- a/extensions/browser/extensions_browser_client.cc
+++ b/extensions/browser/extensions_browser_client.cc
@@ -19,9 +19,6 @@ ExtensionsBrowserClient* ExtensionsBrowserClient::Get() {
 }
 
 void ExtensionsBrowserClient::Set(ExtensionsBrowserClient* client) {
-  // This can happen in tests.

[Yoyo Zhou, 2013/11/01 22:54:06]

I assume this was just copied over from ExtensionsClient?

In ExtensionsClient there are some adverse effects when this gets set twice.

[James Cook, 2013/11/04 19:06:19]

This should be OK here - the case in ExtensionsClient has side effects each time
it is called.  Things are simpler here - I've got complete control of the
lifetime of this object and just need to set it up and tear it down for each
test invocation.

-  if (g_client)
-    return;
   g_client = client;
 }