Fix unit test to allow repeated successful runs by avoiding a function static variable in tests.

chrome/browser/component_updater/test/component_installers_unittest.cc

Index: chrome/browser/component_updater/test/component_installers_unittest.cc
diff --git a/chrome/browser/component_updater/component_installers_unittest.cc b/chrome/browser/component_updater/test/component_installers_unittest.cc
similarity index 91%
rename from chrome/browser/component_updater/component_installers_unittest.cc
rename to chrome/browser/component_updater/test/component_installers_unittest.cc
index 1b11497865540f0fd4603635c9212113e1adc02e..59c4610137697ad4c480e604ebdebfa2e66f240a 100644
--- a/chrome/browser/component_updater/component_installers_unittest.cc
+++ b/chrome/browser/component_updater/test/component_installers_unittest.cc
@@ -14,6 +14,7 @@
 #include "build/build_config.h"
 #include "chrome/common/chrome_paths.h"
 #include "content/public/test/test_browser_thread.h"
+#include "ppapi/shared_impl/test_globals.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ -39,6 +40,10 @@ const FilePath::CharType kDataPath[] =
 
 // TODO(viettrungluu): Separate out into two separate tests; use a test fixture.
 TEST(ComponentInstallerTest, PepperFlashCheck) {
+  scoped_ptr<ppapi::TestGlobals> for_test(

[brettw, 2012/07/13 16:36:22]

I think you can just make this on the stack rather than bothering with the
scoped_refptr/new junk.

+      new ppapi::TestGlobals(ppapi::PpapiGlobals::ForTest()));
+  ppapi::PpapiGlobals::SetPpapiGlobalsOnThreadForTest(for_test.get());
+
   MessageLoop message_loop;
   content::TestBrowserThread ui_thread(BrowserThread::UI, &message_loop);