VideoCaptureImpl & relatives small cleanup.

content/renderer/media/video_capture_message_filter.cc

Index: content/renderer/media/video_capture_message_filter.cc
diff --git a/content/renderer/media/video_capture_message_filter.cc b/content/renderer/media/video_capture_message_filter.cc
index 7157407fff2cbb9ddbc8671085d2f233c0eff035..1d509c620c9cae96753bd91348ab811e04cb6f78 100644
--- a/content/renderer/media/video_capture_message_filter.cc
+++ b/content/renderer/media/video_capture_message_filter.cc
@@ -77,10 +77,9 @@ void VideoCaptureMessageFilter::OnFilterAdded(IPC::Sender* sender) {
   DVLOG(1) << "VideoCaptureMessageFilter::OnFilterAdded()";
   sender_ = sender;
 
-  for (Delegates::iterator it = pending_delegates_.begin();
-       it != pending_delegates_.end(); it++) {
-    it->second->OnDelegateAdded(it->first);
-    delegates_[it->first] = it->second;
+  for (const auto& it : pending_delegates_) {
+    it.second->OnDelegateAdded(it.first);
+    delegates_[it.first] = it.second;

[emircan, 2015/03/02 19:29:07]

If map::insert can be used if the incoming pending_delegates_ keys are going to
be unique.

[mcasas, 2015/03/02 22:57:58]

map::insert() never replaces while operator[] might do.
I'm not sure of the semantics but the key should be
unique or get the it.second replaced... I propose we leave
it as is in this cleanup-only CL.

   }
   pending_delegates_.clear();
 }