Move launcher_.reset() from UITestBase constructor to SetUp().

chrome/test/automation/automation_proxy_uitest.h

Index: chrome/test/automation/automation_proxy_uitest.h
diff --git a/chrome/test/automation/automation_proxy_uitest.h b/chrome/test/automation/automation_proxy_uitest.h
index 2ce19d46a3429013df297ecea4026868d7ab1ddf..c08aa91d25f7f90c9cab7785159f8804c1f1de3f 100644
--- a/chrome/test/automation/automation_proxy_uitest.h
+++ b/chrome/test/automation/automation_proxy_uitest.h
@@ -109,7 +109,7 @@ class ExternalTabUITestMockClient : public AutomationProxy {
 // Base your external tab UI tests on this.
 class ExternalTabUITest : public UITest {
  public:
-  ExternalTabUITest();
+  ExternalTabUITest() : UITest(MessageLoop::TYPE_UI) {}

[Paweł Hajdan Jr., 2011/01/12 21:34:29]

nit: Please keep the constructor's body in the .cc file, just remove the call to
CreateProxyLauncher. Decluttering headers helps with binary size and build time,
ask Elliot Glaysher (erg/eglaysher) for more info.

   // Override UITest's CreateProxyLauncher to provide the unit test
   // with our special implementation of AutomationProxy.
   // This function is called from within UITest::SetUp().