Remove ShellConnection::WaitForInitialize

content/browser/browser_main_loop.cc

Index: content/browser/browser_main_loop.cc
diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
index 162f0269445b157301337215fa80f7ae86decccc..3d166e1afbc29fbaac611852ed46e84419b9de53 100644
--- a/content/browser/browser_main_loop.cc
+++ b/content/browser/browser_main_loop.cc
@@ -77,6 +77,7 @@
 #include "media/base/media.h"
 #include "media/base/user_input_monitor.h"
 #include "media/midi/midi_manager.h"
+#include "mojo/edk/embedder/embedder.h"
 #include "mojo/shell/public/cpp/shell.h"
 #include "net/base/network_change_notifier.h"
 #include "net/socket/client_socket_factory.h"
@@ -913,11 +914,14 @@ int BrowserMainLoop::CreateThreads() {
 
 int BrowserMainLoop::PreMainMessageLoopRun() {
   if (IsRunningInMojoShell()) {
+    mojo::edk::SetParentPipeHandleFromCommandLine();

[Ken Rockot(use gerrit already), 2016/03/13 16:19:10]

Also note: I added this API for convenience since sharing a common init path
through RunnerConnection meant ripping this magic out from it. We don't always
use the command line handle for content children.

     MojoShellConnectionImpl::Create();
-    MojoShellConnectionImpl::Get()->BindToCommandLinePlatformChannel();
+    MojoShellConnectionImpl::Get()->BindToRequestFromCommandLine();
 #if defined(MOJO_SHELL_CLIENT) && defined(USE_AURA)
-    views::WindowManagerConnection::Create(
-        MojoShellConnection::Get()->GetConnector());
+    if (MojoShellConnection::Get()) {
+      views::WindowManagerConnection::Create(
+          MojoShellConnection::Get()->GetConnector());
+    }
 #endif
   }