diff options
author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-26 11:43:07 +0000 |
---|---|---|
committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-26 11:43:07 +0000 |
commit | bf6bdef431d0f5ae1af407e68988f8697e213052 (patch) | |
tree | 88ca8aa9ff1ba8a68824a2bb2e9eff02e0b05e55 /ppapi/proxy/broker_dispatcher.cc | |
parent | a1d076eef3be6ab74325d3d5ab30a267a1c9cc9a (diff) | |
download | chromium_src-bf6bdef431d0f5ae1af407e68988f8697e213052.zip chromium_src-bf6bdef431d0f5ae1af407e68988f8697e213052.tar.gz chromium_src-bf6bdef431d0f5ae1af407e68988f8697e213052.tar.bz2 |
Pepper: Fix channel init in ProxyChannel.
Previously, there was no way to savely add filters to the underlying
SyncChannel in a ProxyChannel safely.
This change splits channel initialization into two methods so there's a clear
step where filters may be added safely.
BUG=343768
Review URL: https://codereview.chromium.org/179743005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253408 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/broker_dispatcher.cc')
-rw-r--r-- | ppapi/proxy/broker_dispatcher.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ppapi/proxy/broker_dispatcher.cc b/ppapi/proxy/broker_dispatcher.cc index 7187852..6095679 100644 --- a/ppapi/proxy/broker_dispatcher.cc +++ b/ppapi/proxy/broker_dispatcher.cc @@ -24,8 +24,9 @@ bool BrokerDispatcher::InitBrokerWithChannel( base::ProcessId peer_pid, const IPC::ChannelHandle& channel_handle, bool is_client) { - return ProxyChannel::InitWithChannel(delegate, peer_pid, channel_handle, - is_client); + InitWithChannel(delegate, peer_pid); + ConnectChannel(channel_handle, is_client); + return true; } bool BrokerDispatcher::OnMessageReceived(const IPC::Message& msg) { |