extensions: Introduce a fake ExtensionsClient implementation.

extensions/test/extensions_unittests_main.cc

 // Copyright 2014 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 "base/bind.h"
 #include "base/macros.h"
 #include "base/test/launcher/unit_test_launcher.h"
 #include "base/test/test_suite.h"
+#include "extensions/test/test_extensions_client.h"
 
 namespace {
 
 class ExtensionsTestSuite : public base::TestSuite {
  public:
   ExtensionsTestSuite(int argc, char** argv);
 
- protected:
+ private:
   // base::TestSuite:
   virtual void Initialize() OVERRIDE;
   virtual void Shutdown() OVERRIDE;
 
- private:
+  scoped_ptr<extensions::TestExtensionsClient> client_;
+
   DISALLOW_COPY_AND_ASSIGN(ExtensionsTestSuite);
 };
 
 ExtensionsTestSuite::ExtensionsTestSuite(int argc, char** argv)
     : base::TestSuite(argc, argv) {}
 
 void ExtensionsTestSuite::Initialize() {
   base::TestSuite::Initialize();
+
+  client_.reset(new extensions::TestExtensionsClient());
+  extensions::ExtensionsClient::Set(client_.get());
 }
 
 void ExtensionsTestSuite::Shutdown() {
   base::TestSuite::Shutdown();

[James Cook, 2014/04/11 16:26:49]

Question: Do you want to explicitly client_.reset() and Set(NULL) above this
line?  I'm not sure when Shutdown() gets called relative to this object being
destroyed and that would make sure that the client can do cleanup before the
test framework starts tearing itself down.

[Yoyo Zhou, 2014/04/11 21:10:29]

I'm not sure it's necessary - for example, ChromeUnitTestSuite doesn't do
especial cleanup.

 }
 
 }  // namespace
 
 int main(int argc, char** argv) {
   ExtensionsTestSuite test_suite(argc, argv);
 
   return base::LaunchUnitTests(argc,
                                argv,
                                base::Bind(&ExtensionsTestSuite::Run,
                                           base::Unretained(&test_suite)));
 }