Base for Ash browser_tests on Win8.

chrome/browser/browser_process_platform_part_aurawin.cc

Index: chrome/browser/browser_process_platform_part_aurawin.cc
diff --git a/chrome/browser/browser_process_platform_part_aurawin.cc b/chrome/browser/browser_process_platform_part_aurawin.cc
index e6be06dd24c062da2c72692ac30e9c84372a7d02..ae4cde1129c911dfe7b5d77d6f0eb14f445e312d 100644
--- a/chrome/browser/browser_process_platform_part_aurawin.cc
+++ b/chrome/browser/browser_process_platform_part_aurawin.cc
@@ -11,6 +11,7 @@
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/metro_viewer/chrome_metro_viewer_process_host_aurawin.h"
 #include "chrome/common/chrome_switches.h"
+#include "win8/test/test_registrar_constants.h"
 
 BrowserProcessPlatformPart::BrowserProcessPlatformPart() {
 }
@@ -24,13 +25,22 @@ void BrowserProcessPlatformPart::OnMetroViewerProcessTerminated() {
 
 void BrowserProcessPlatformPart::PlatformSpecificCommandLineProcessing(
     const CommandLine& command_line) {
-  if (base::win::GetVersion() >= base::win::VERSION_WIN8 &&
-      command_line.HasSwitch(switches::kViewerConnection) &&
-      !metro_viewer_process_host_.get()) {
-    // Tell the metro viewer process host to connect to the given IPC channel.
-    metro_viewer_process_host_.reset(
-        new ChromeMetroViewerProcessHost(
-            command_line.GetSwitchValueASCII(switches::kViewerConnection)));
+  // Check for Windows 8 specific commandlines requesting that this process
+  // either connect to an existing viewer or launch a new viewer and
+  // synchronously wait for it to connect.
+  if (base::win::GetVersion() >= base::win::VERSION_WIN8) {
+    if (command_line.HasSwitch(switches::kViewerConnection) &&
+        !metro_viewer_process_host_.get()) {
+      // Tell the metro viewer process host to connect to the given IPC channel.
+      metro_viewer_process_host_.reset(
+          new ChromeMetroViewerProcessHost(
+              command_line.GetSwitchValueASCII(switches::kViewerConnection)));
+    } else if (command_line.HasSwitch(switches::kAshBrowserTests)) {

[grt (UTC plus 2), 2013/05/29 02:22:51]

i'm a little uncomfortable with this test-only code being in the product. is
there precedent for this? i guess it's a bit similar to kTestingChannelID. hmm.
please have one of the reviewers in chrome/OWNERS weigh in.

[gab, 2013/05/29 14:52:20]

Ya, I agree that this is ugly, especially that I also have to make this target
depend on test_registrar_constants (incidentally this week's edition of TotT
talks about avoiding exactly this!)...

How about I add a generic switch to the chrome switches (e.g.
kViewerLaunchViaAppId) which would only be set by browser tests for now when
they find the kAshBrowserTests switch (which could then be a test-only switch
instead of being in chrome_switches)?

A bit more involved, but it feels cleaner, WDYT?

[grt (UTC plus 2), 2013/05/29 15:51:03]

If I understand, yeah, this seems better. For what it's worth, I just became
aware of kTestCompositor, which lives along with the regular content switches,
so I see that there is precedent for putting handling of test-only switches in
the product. This makes me feel better.

[gab, 2013/05/29 23:09:42]

Okay, did that in patch set 6, let me know if you like it.

I don't like checking success with a CHECK() now that this is no longer
"test-only" code though... any better idea?

At least chrome code no longer depends on a test target...

Still having trouble linking though since weirdly enough
test/base/test_switches.cc is linked into 'browser' not 'test_support_common'
where test/base/in_process_browser_test.cc is linked into...

+      metro_viewer_process_host_.reset(
+          new ChromeMetroViewerProcessHost("viewer"));
+      CHECK(metro_viewer_process_host_->LaunchViewerAndWaitForConnection(
+          win8::test::kDefaultTestAppUserModelId));
+    }
   }
 }