Do not spawn a thread in browser/interactive ui tests before spawning sandbox host process

chromeos/dbus/dbus_thread_manager.h

Index: chromeos/dbus/dbus_thread_manager.h
diff --git a/chromeos/dbus/dbus_thread_manager.h b/chromeos/dbus/dbus_thread_manager.h
index c980b27f960898f2d19f412e52234a1aee7e399e..5c48f9176d309658659cddcac030e0681a02415d 100644
--- a/chromeos/dbus/dbus_thread_manager.h
+++ b/chromeos/dbus/dbus_thread_manager.h
@@ -83,10 +83,18 @@ class CHROMEOS_EXPORT DBusThreadManager {
   // making it a Singleton, to ensure clean startup and shutdown.
   static void Initialize();
 
-  // Similar to Initialize(), but can inject an alternative
-  // DBusThreadManager such as MockDBusThreadManager for testing.
-  // The injected object will be owned by the internal pointer and deleted
-  // by Shutdown().
+  // Sets an alternative DBusThreadManager such as MockDBusThreadManager
+  // to be used in |Initialize()| for testing. Tests that calls calls

[satorux1, 2013/11/27 03:27:17]

Tests that calls calls -> Tests that call

[oshima, 2013/11/27 07:41:54]

Done.

+  // DBusThreadManager::Initialize() (such as browser_tests and
+  // interactive_ui_tests) should use this instead of calling
+  // |InitiailzeForTesting|.  The injected object will be owned by the
+  // internal pointer and deleted by Shutdown().
+  static void SetInstanceForTesting(DBusThreadManager* dbus_thread_manager);
+
+  // Similar to Initialize(), but injects an alternative
+  // DBusThreadManager using SetInstanceForTest first.  The injected
+  // object will be owned by the internal pointer and deleted by
+  // Shutdown().
   static void InitializeForTesting(DBusThreadManager* dbus_thread_manager);
 
   // Initialize with stub implementations for tests based on stubs.