| Index: content/browser/child_process_launcher.cc
|
| diff --git a/content/browser/child_process_launcher.cc b/content/browser/child_process_launcher.cc
|
| index 830d5b9345a536d9207d0f5f075b8b4147ec5372..e54cada9326d13645a4890028f87a29d0ea846f4 100644
|
| --- a/content/browser/child_process_launcher.cc
|
| +++ b/content/browser/child_process_launcher.cc
|
| @@ -48,17 +48,6 @@
|
| #include "gin/v8_initializer.h"
|
| #endif
|
|
|
| -#if defined(MOJO_SHELL_CLIENT)
|
| -#include "base/thread_task_runner_handle.h"
|
| -#include "content/public/common/mojo_shell_connection.h"
|
| -#include "mojo/application/public/cpp/application_impl.h"
|
| -#include "mojo/converters/network/network_type_converters.h"
|
| -#include "mojo/shell/application_manager.mojom.h"
|
| -#include "third_party/mojo/src/mojo/edk/embedder/embedder.h"
|
| -#include "third_party/mojo/src/mojo/edk/embedder/platform_channel_pair.h"
|
| -#include "third_party/mojo/src/mojo/edk/embedder/scoped_platform_handle.h"
|
| -#endif
|
| -
|
| namespace content {
|
|
|
| namespace {
|
| @@ -333,10 +322,6 @@ void SetProcessBackgroundedOnLauncherThread(base::Process process,
|
| #endif
|
| }
|
|
|
| -#if defined(MOJO_SHELL_CLIENT)
|
| -void DidCreateChannel(mojo::embedder::ChannelInfo* info) {}
|
| -#endif
|
| -
|
| } // namespace
|
|
|
| ChildProcessLauncher::ChildProcessLauncher(
|
| @@ -380,10 +365,6 @@ void ChildProcessLauncher::Launch(
|
| int child_process_id) {
|
| DCHECK(CalledOnValidThread());
|
|
|
| -#if defined(MOJO_SHELL_CLIENT)
|
| - CreateMojoShellChannel(cmd_line, child_process_id);
|
| -#endif
|
| -
|
| #if defined(OS_ANDROID)
|
| // Android only supports renderer, sandboxed utility and gpu.
|
| std::string process_type =
|
| @@ -523,46 +504,6 @@ void ChildProcessLauncher::Notify(
|
| }
|
| }
|
|
|
| -#if defined(MOJO_SHELL_CLIENT)
|
| -void ChildProcessLauncher::CreateMojoShellChannel(
|
| - base::CommandLine* command_line,
|
| - int child_process_id) {
|
| - // Some process types get created before the main message loop.
|
| - if (!MojoShellConnection::Get())
|
| - return;
|
| -
|
| - // Create the channel to be shared with the target process.
|
| - mojo::embedder::HandlePassingInformation handle_passing_info;
|
| - mojo::embedder::PlatformChannelPair platform_channel_pair;
|
| -
|
| - // Give one end to the shell so that it can create an instance.
|
| - mojo::embedder::ScopedPlatformHandle platform_channel =
|
| - platform_channel_pair.PassServerHandle();
|
| - mojo::ScopedMessagePipeHandle handle(mojo::embedder::CreateChannel(
|
| - platform_channel.Pass(), base::Bind(&DidCreateChannel),
|
| - base::ThreadTaskRunnerHandle::Get()));
|
| - mojo::shell::mojom::ApplicationManagerPtr application_manager;
|
| - MojoShellConnection::Get()->GetApplication()->ConnectToService(
|
| - mojo::URLRequest::From(std::string("mojo:shell")),
|
| - &application_manager);
|
| - // The content of the URL/qualifier we pass is actually meaningless, it's only
|
| - // important that they're unique per process.
|
| - // TODO(beng): We need to specify a restrictive CapabilityFilter here that
|
| - // matches the needs of the target process. Figure out where that
|
| - // specification is best determined (not here, this is a common
|
| - // chokepoint for all process types) and how to wire it through.
|
| - // http://crbug.com/555393
|
| - application_manager->CreateInstanceForHandle(
|
| - mojo::ScopedHandle(mojo::Handle(handle.release().value())),
|
| - "exe:chrome_renderer", // See above about how this string is meaningless.
|
| - base::IntToString(child_process_id));
|
| -
|
| - // Put the other end on the command line used to launch the target.
|
| - platform_channel_pair.PrepareToPassClientHandleToChildProcess(
|
| - command_line, &handle_passing_info);
|
| -}
|
| -#endif // defined(MOJO_SHELL_CLIENT)
|
| -
|
| bool ChildProcessLauncher::IsStarting() {
|
| // TODO(crbug.com/469248): This fails in some tests.
|
| // DCHECK(CalledOnValidThread());
|
|
|