Chromium Code Reviews| Index: components/nacl/loader/nacl_trusted_listener.h |
| diff --git a/components/nacl/loader/nacl_trusted_listener.h b/components/nacl/loader/nacl_trusted_listener.h |
| index a368ab5b3331ca68ee1827f2807f2a122ec365a4..f6166ca7dda108ede02a20c93f74c0434e8d7640 100644 |
| --- a/components/nacl/loader/nacl_trusted_listener.h |
| +++ b/components/nacl/loader/nacl_trusted_listener.h |
| @@ -6,32 +6,22 @@ |
| #define COMPONENTS_NACL_LOADER_NACL_TRUSTED_LISTENER_H_ |
| #include "base/macros.h" |
| -#include "base/memory/ref_counted.h" |
| -#include "ipc/ipc_channel_handle.h" |
| -#include "ipc/ipc_listener.h" |
| -#include "ipc/ipc_sync_channel.h" |
| +#include "components/nacl/common/nacl.mojom.h" |
| namespace base { |
| class SingleThreadTaskRunner; |
| } |
| -class NaClTrustedListener : public base::RefCounted<NaClTrustedListener>, |
| - public IPC::Listener { |
| +class NaClTrustedListener { |
| public: |
| - NaClTrustedListener(const IPC::ChannelHandle& handle, |
| - base::SingleThreadTaskRunner* ipc_task_runner, |
| - base::WaitableEvent* shutdown_event); |
| + NaClTrustedListener(nacl::mojom::NaClRendererHostPtr renderer_host, |
| + base::SingleThreadTaskRunner* io_task_runner); |
| + ~NaClTrustedListener(); |
| - // Listener implementation. |
| - bool OnMessageReceived(const IPC::Message& message) override; |
| - |
| - bool Send(IPC::Message* msg); |
| + nacl::mojom::NaClRendererHost& renderer_host() { return *renderer_host_; } |
|
Mark Seaborn
2016/12/19 19:00:36
Is there a reason why this is a reference? I thou
Sam McNally
2016/12/19 23:12:12
They're only banned as function parameters. Change
|
| private: |
| - friend class base::RefCounted<NaClTrustedListener>; |
| - ~NaClTrustedListener() override; |
| - IPC::ChannelHandle channel_handle_; |
| - std::unique_ptr<IPC::SyncChannel> channel_; |
| + nacl::mojom::NaClRendererHostPtr renderer_host_; |
| DISALLOW_COPY_AND_ASSIGN(NaClTrustedListener); |
| }; |