Refactor UITestBase/ProxyLauncher.

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 0b5556fa5a50176bbbda9bc76e420547564001c3..a2df81a5bc7a074c278303a624c4a947eb429491 100644
--- a/chrome/test/automation/automation_proxy_uitest.h
+++ b/chrome/test/automation/automation_proxy_uitest.h
@@ -107,7 +107,8 @@ class ExternalTabUITestMockClient : public AutomationProxy {
 // Base your external tab UI tests on this.
 class ExternalTabUITest : public UITest {
  public:
-  ExternalTabUITest() : UITest(MessageLoop::TYPE_UI) {}
+  ExternalTabUITest() : UITest(MessageLoop::TYPE_UI) {

[John Grabowski, 2011/01/07 03:36:11]

In a header, empty methods are usually {}, not {\n}
I don't think that's in the style guide as such but is a clear convention.
So remove this change.

[dtu, 2011/01/08 01:54:57]

Done.

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