Move Mojo channel initialization closer to IPC::Channel setup

content/browser/mojo/mojo_application_host.h

+// Copyright 2014 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 CONTENT_BROWSER_MOJO_MOJO_APPLICATION_HOST_H_
+#define CONTENT_BROWSER_MOJO_MOJO_APPLICATION_HOST_H_
+
+#include "base/process/process_handle.h"
+#include "content/common/mojo/mojo_channel_init.h"
+#include "mojo/embedder/scoped_platform_handle.h"
+#include "mojo/public/cpp/bindings/remote_ptr.h"
+#include "mojo/public/interfaces/shell/shell.mojom.h"
+
+namespace IPC {
+class Sender;
+}
+
+namespace content {
+
+class MojoApplicationHost {

[sky, 2014/04/16 17:18:10]

Add description.

[darin (slow to review), 2014/04/16 23:26:04]

Will do.

+ public:
+  MojoApplicationHost();
+  ~MojoApplicationHost();
+
+  // Two-phase initialization:
+  //  1- Init makes the shell_client() available synchronously.
+  //  2- Activate establishes the actual connection to the peer process.
+  bool Init();
+  bool Activate(IPC::Sender* sender, base::ProcessHandle process_handle);
+
+  mojo::ShellClient* shell_client() { return shell_client_.get(); }
+
+ private:
+  MojoChannelInit channel_init_;
+  mojo::embedder::ScopedPlatformHandle client_handle_;
+  mojo::RemotePtr<mojo::ShellClient> shell_client_;
+};

[sky, 2014/04/16 17:18:10]

DISALLOW...

[darin (slow to review), 2014/04/16 23:26:04]

Will do.

+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_MOJO_MOJO_APPLICATION_HOST_H_