Only do aggressive metro viewer termination in tests

ash/test/test_metro_viewer_process_host.cc

Index: ash/test/test_metro_viewer_process_host.cc
diff --git a/ash/test/test_metro_viewer_process_host.cc b/ash/test/test_metro_viewer_process_host.cc
index 85df3e21cbccfb6f64faad4fb6aaa9ea2dbdf6f5..d3e863411fd34f70a4604bc0666e22317b6094cb 100644
--- a/ash/test/test_metro_viewer_process_host.cc
+++ b/ash/test/test_metro_viewer_process_host.cc
@@ -22,6 +22,21 @@ TestMetroViewerProcessHost::TestMetroViewerProcessHost(
 TestMetroViewerProcessHost::~TestMetroViewerProcessHost() {
 }
 
+void TestMetroViewerProcessHost::TerminateViewer() {
+  base::ProcessId viewer_process_id = GetViewerProcessId();
+  if (viewer_process_id != base::kNullProcessId) {
+    base::ProcessHandle viewer_process = NULL;
+    base::OpenProcessHandleWithAccess(
+        viewer_process_id,
+        PROCESS_QUERY_INFORMATION | SYNCHRONIZE | PROCESS_TERMINATE,
+        &viewer_process);
+    if (viewer_process) {
+      ::TerminateProcess(viewer_process, 0);
+      ::CloseHandle(viewer_process);

[ananta, 2014/09/20 00:32:54]

Please add a WaitForSingleObject here to ensure that the process has terminated.
As per msdn the TerminateProcess is async.

[scottmg, 2014/09/20 02:12:50]

Thanks, done.

+    }
+  }
+}
+
 void TestMetroViewerProcessHost::OnChannelError() {
   closed_unexpectedly_ = true;
   aura::RemoteWindowTreeHostWin::Instance()->Disconnected();