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

Unified Diff: ipc/ipc_channel_proxy.cc

Issue 7670006: ipc: rename a variable to make code clearer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 9 years, 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_channel_proxy.cc
diff --git a/ipc/ipc_channel_proxy.cc b/ipc/ipc_channel_proxy.cc
index da3da24ab0444f932cbbc9eef8bca022d8be9476..dc990c26184ab3c6d8d9aaaee9a6c9ef0acffeba 100644
--- a/ipc/ipc_channel_proxy.cc
+++ b/ipc/ipc_channel_proxy.cc
@@ -194,22 +194,22 @@ void ChannelProxy::Context::OnSendMessage(Message* message) {
// Called on the IPC::Channel thread
void ChannelProxy::Context::OnAddFilter() {
- std::vector<scoped_refptr<MessageFilter> > filters;
+ std::vector<scoped_refptr<MessageFilter> > new_filters;
{
base::AutoLock auto_lock(pending_filters_lock_);
- filters.swap(pending_filters_);
+ new_filters.swap(pending_filters_);
}
- for (size_t i = 0; i < filters.size(); ++i) {
- filters_.push_back(filters[i]);
+ for (size_t i = 0; i < new_filters.size(); ++i) {
+ filters_.push_back(new_filters[i]);
// If the channel has already been created, then we need to send this
// message so that the filter gets access to the Channel.
if (channel_.get())
- filters[i]->OnFilterAdded(channel_.get());
- // Ditto for the peer process id.
+ new_filters[i]->OnFilterAdded(channel_.get());
+ // Ditto for if the channel has been connected.
if (peer_pid_)
- filters[i]->OnChannelConnected(peer_pid_);
+ new_filters[i]->OnChannelConnected(peer_pid_);
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698