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

Unified Diff: ipc/ipc_sync_message_filter.cc

Issue 343593006: Revert 276939 "Decouple IPC::MessageFilter from IPC::Channel" (Closed) Base URL: svn://svn.chromium.org/chrome/branches/2057/src/
Patch Set: 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
« no previous file with comments | « ipc/ipc_sync_message_filter.h ('k') | ipc/message_filter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_sync_message_filter.cc
===================================================================
--- ipc/ipc_sync_message_filter.cc (revision 278168)
+++ ipc/ipc_sync_message_filter.cc (working copy)
@@ -17,7 +17,7 @@
namespace IPC {
SyncMessageFilter::SyncMessageFilter(base::WaitableEvent* shutdown_event)
- : sender_(NULL),
+ : channel_(NULL),
listener_loop_(MessageLoopProxy::current()),
shutdown_event_(shutdown_event) {
}
@@ -67,19 +67,19 @@
return pending_message.send_result;
}
-void SyncMessageFilter::OnFilterAdded(Sender* sender) {
- sender_ = sender;
+void SyncMessageFilter::OnFilterAdded(Channel* channel) {
+ channel_ = channel;
base::AutoLock auto_lock(lock_);
io_loop_ = MessageLoopProxy::current();
}
void SyncMessageFilter::OnChannelError() {
- sender_ = NULL;
+ channel_ = NULL;
SignalAllEvents();
}
void SyncMessageFilter::OnChannelClosing() {
- sender_ = NULL;
+ channel_ = NULL;
SignalAllEvents();
}
@@ -104,8 +104,8 @@
}
void SyncMessageFilter::SendOnIOThread(Message* message) {
- if (sender_) {
- sender_->Send(message);
+ if (channel_) {
+ channel_->Send(message);
return;
}
« no previous file with comments | « ipc/ipc_sync_message_filter.h ('k') | ipc/message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698