Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Unified Diff: ipc/ipc_channel_proxy.cc

Issue 324143002: Decouple IPC::MessageFilter from IPC::Channel (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing compilation errors Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ipc/ipc_channel_proxy.cc
diff --git a/ipc/ipc_channel_proxy.cc b/ipc/ipc_channel_proxy.cc
index 7441c6543b6caaab7aa34ef18c8a439aedd233a1..8ba91b01d4db196f33cc1c8d0b74b13304a9f08e 100644
--- a/ipc/ipc_channel_proxy.cc
+++ b/ipc/ipc_channel_proxy.cc
@@ -111,7 +111,7 @@ void ChannelProxy::Context::OnChannelConnected(int32 peer_pid) {
// Called on the IPC::Channel thread
void ChannelProxy::Context::OnChannelError() {
for (size_t i = 0; i < filters_.size(); ++i)
- filters_[i]->OnChannelError();
+ filters_[i]->OnSenderError();
// See above comment about using listener_task_runner_ here.
listener_task_runner_->PostTask(
@@ -143,7 +143,7 @@ void ChannelProxy::Context::OnChannelClosed() {
return;
for (size_t i = 0; i < filters_.size(); ++i) {
- filters_[i]->OnChannelClosing();
+ filters_[i]->OnSenderClosing();
filters_[i]->OnFilterRemoved();
}
@@ -199,7 +199,7 @@ void ChannelProxy::Context::OnAddFilter() {
// The channel has already been created and connected, so we need to
// inform the filters right now.
new_filters[i]->OnFilterAdded(channel_.get());
- new_filters[i]->OnChannelConnected(peer_pid_);
+ new_filters[i]->OnSenderConnected();
}
}

Powered by Google App Engine
This is Rietveld 408576698