Chromium Code Reviews| Index: content/browser/mojo/mojo_application_host.h |
| diff --git a/content/browser/mojo/mojo_application_host.h b/content/browser/mojo/mojo_application_host.h |
| index 4e3e3e1625faf8823e3cb8986ad9fa366fd8baed..283e9682d41fe9681a7ece832e178e3e579efae6 100644 |
| --- a/content/browser/mojo/mojo_application_host.h |
| +++ b/content/browser/mojo/mojo_application_host.h |
| @@ -27,39 +27,17 @@ class MojoApplicationHost { |
| virtual ~MojoApplicationHost(); |
| // Two-phase initialization: |
| - // 1- Init makes the shell_client() available synchronously. |
| + // 1- Init makes |shell_handle| available synchronously. |
| // 2- Activate establishes the actual connection to the peer process. |
| - bool Init(); |
| + bool Init(mojo::ScopedMessagePipeHandle* shell_handle); |
|
darin (slow to review)
2014/06/12 05:27:12
nit: |shell_handle| doesn't seem like the right na
Sam McNally
2014/06/12 08:56:40
Done.
|
| bool Activate(IPC::Sender* sender, base::ProcessHandle process_handle); |
| bool did_activate() const { return did_activate_; } |
| - mojo::ServiceProvider* service_provider() { |
| - DCHECK(child_service_provider_.get()); |
| - return child_service_provider_->client(); |
| - } |
| - |
| private: |
| - class ServiceProviderImpl |
| - : public mojo::InterfaceImpl<mojo::ServiceProvider> { |
| - public: |
| - virtual void OnConnectionError() OVERRIDE { |
| - // TODO(darin): How should we handle this error? |
| - } |
| - |
| - // mojo::ServiceProvider methods: |
| - virtual void ConnectToService( |
| - const mojo::String& service_url, |
| - const mojo::String& service_name, |
| - mojo::ScopedMessagePipeHandle handle, |
| - const mojo::String& requestor_url) OVERRIDE; |
| - }; |
| - |
| mojo::common::ChannelInit channel_init_; |
| mojo::embedder::ScopedPlatformHandle client_handle_; |
| - scoped_ptr<ServiceProviderImpl> child_service_provider_; |
| - |
| bool did_activate_; |
| DISALLOW_COPY_AND_ASSIGN(MojoApplicationHost); |