Automatic Win8 default browser registration for the ash unittests.

ash/test/test_metro_viewer_process_host.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_TEST_TEST_METRO_VIEWER_PROCESS_HOST_H_
 #define ASH_TEST_TEST_METRO_VIEWER_PROCESS_HOST_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 10 matching lines @@
 namespace ash {
 namespace test {
 
 class TestMetroViewerProcessHost : public IPC::Listener,
                                    public IPC::Sender,
                                    public base::NonThreadSafe {
  public:
   explicit TestMetroViewerProcessHost(const std::string& ipc_channel_name);
   virtual ~TestMetroViewerProcessHost();
 
-  // Launches the Chrome viewer process and blocks until that viewer process
-  // connects or until a timeout is reached. Returns true if the viewer process
-  // connects before the timeout is reached.
+  // Launches the viewer process associated with the given |app_user_model_id|
+  // and blocks until that viewer process connects or until a timeout is
+  // reached. Returns true if the viewer process connects before the timeout is
+  // reached.
   // TODO(robertshield): This creates a run-time dependency on chrome.exe as the

[grt (UTC plus 2), 2013/02/12 15:37:43]

The dependency is created in the registration rather than here, correct? If so,
please move this part of the comment elsewhere.

[robertshield, 2013/02/12 16:57:08]

Done (moved to test_registrar.cc).

-  // viewer process and, indirectly, setup.exe as the only thing that can
-  // correctly register a program as the default browser on metro. Investigate
-  // extracting the registration code and the metro init code and building them
-  // into a standalone viewer process.
-  bool LaunchImmersiveChromeAndWaitForConnection();
+  // viewer process. Investigate extracting the registration code and the metro
+  // init code and building them into a standalone viewer process.
+  bool LaunchImmersiveChromeAndWaitForConnection(

[grt (UTC plus 2), 2013/02/12 15:37:43]

Chrome -> Viewer?

[robertshield, 2013/02/12 16:57:08]

Done.

+      const string16& app_user_model_id);
 
   // IPC::Sender implementation:
   virtual bool Send(IPC::Message* msg) OVERRIDE;
 
   // IPC::Listener implementation:
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
   virtual void OnChannelError() OVERRIDE;
 
   bool closed_unexpectedly() { return closed_unexpectedly_; }
 
@@ skipping 29 matching lines @@
   bool closed_unexpectedly_;
 
   DISALLOW_COPY_AND_ASSIGN(TestMetroViewerProcessHost);
 };
 
 
 }  // namespace test
 }  // namespace ash
 
 #endif  // ASH_TEST_TEST_METRO_VIEWER_PROCESS_HOST_H_