Move ExtensionsProcessManager to src/extensions, part 2

extensions/browser/extensions_browser_client.cc

 // Copyright 2013 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 "extensions/browser/extensions_browser_client.h"
 
 #include "base/basictypes.h"
 
 namespace extensions {
 
 namespace {
 
 ExtensionsBrowserClient* g_client = NULL;
 
 }  // namespace
 
 ExtensionsBrowserClient* ExtensionsBrowserClient::Get() {
   return g_client;
 }
 
 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;
 }
 
 }  // namespace extensions