IPC::ChannelMojo: Introduce IPC::MojoBootstrap for Windows

ipc/mojo/ipc_mojo_bootstrap_delegate.h

Index: ipc/mojo/ipc_mojo_bootstrap_delegate.h
diff --git a/ipc/mojo/ipc_mojo_bootstrap_delegate.h b/ipc/mojo/ipc_mojo_bootstrap_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..10ebe8dfb526931007760cc6a7f0a98b562e7146
--- /dev/null
+++ b/ipc/mojo/ipc_mojo_bootstrap_delegate.h
@@ -0,0 +1,19 @@
+// 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 IPC_MOJO_IPC_MOJO_BOOTSTRAP_DELEGATE_H_
+#define IPC_MOJO_IPC_MOJO_BOOTSTRAP_DELEGATE_H_
+
+#include "mojo/embedder/scoped_platform_handle.h"
+
+namespace IPC {
+
+class MojoBootstrapDelegate {

[viettrungluu, 2014/09/15 20:19:44]

Normally, I'm a fan of standalone delegate classes, but in this case is it
really worth having a separate header file (and separating it from
MojoBootstrap)?

(Unless there's a compelling reason, I'd just have this as an inner class of
MojoBootstrap.)

[Hajime Morrita, 2014/09/15 22:01:37]

Made it an inner class.
I hoped it help to hide some dependency from ipc_channel_mojo.h in
ipc_mojo_bootstrap.h but it didn't work well :-/

+ public:
+  virtual void OnPipeAvailable(mojo::embedder::ScopedPlatformHandle handle) = 0;
+  virtual void OnBootstrapError() = 0;
+};
+}
+
+#endif  // namaespace IPC_MOJO_IPC_MOJO_BOOTSTRAP_DELEGATE_H_