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

Unified Diff: content/browser/service_worker/embedded_worker_instance.cc

Issue 2410283004: Remove usage of base::ObserverList<T>::Iter::GetNext() in //content. (Closed)
Patch Set: Created 4 years, 2 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: content/browser/service_worker/embedded_worker_instance.cc
diff --git a/content/browser/service_worker/embedded_worker_instance.cc b/content/browser/service_worker/embedded_worker_instance.cc
index 842a57a357553eb9d6d591088973fd0bfc0b1306..8e81ee915edfcf7a3dc0c8a01fe186ef4f6338aa 100644
--- a/content/browser/service_worker/embedded_worker_instance.cc
+++ b/content/browser/service_worker/embedded_worker_instance.cc
@@ -775,9 +775,8 @@ base::WeakPtr<EmbeddedWorkerInstance> EmbeddedWorkerInstance::AsWeakPtr() {
}
bool EmbeddedWorkerInstance::OnMessageReceived(const IPC::Message& message) {
- ListenerList::Iterator it(&listener_list_);
- while (Listener* listener = it.GetNext()) {
- if (listener->OnMessageReceived(message))
+ for (auto& listener : listener_list_) {
+ if (listener.OnMessageReceived(message))
return true;
}
return false;

Powered by Google App Engine
This is Rietveld 408576698