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

Unified Diff: content/browser/dom_storage/dom_storage_context_impl.cc

Issue 2422923002: Reduce usage of FOR_EACH_OBSERVER macro in content/browser (Closed)
Patch Set: skip cases in 2418143004 and 2418373002 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/dom_storage/dom_storage_context_impl.cc
diff --git a/content/browser/dom_storage/dom_storage_context_impl.cc b/content/browser/dom_storage/dom_storage_context_impl.cc
index cd2cdbd49181b9fce9b888f912c708c41fc2096e..2479f8f8d50adaa0fc904ae0846fc2ca4cf1e234 100644
--- a/content/browser/dom_storage/dom_storage_context_impl.cc
+++ b/content/browser/dom_storage/dom_storage_context_impl.cc
@@ -301,9 +301,8 @@ void DOMStorageContextImpl::NotifyItemSet(
const base::string16& new_value,
const base::NullableString16& old_value,
const GURL& page_url) {
- FOR_EACH_OBSERVER(
- EventObserver, event_observers_,
- OnDOMStorageItemSet(area, key, new_value, old_value, page_url));
+ for (auto& observer : event_observers_)
+ observer.OnDOMStorageItemSet(area, key, new_value, old_value, page_url);
}
void DOMStorageContextImpl::NotifyItemRemoved(
@@ -311,17 +310,15 @@ void DOMStorageContextImpl::NotifyItemRemoved(
const base::string16& key,
const base::string16& old_value,
const GURL& page_url) {
- FOR_EACH_OBSERVER(
- EventObserver, event_observers_,
- OnDOMStorageItemRemoved(area, key, old_value, page_url));
+ for (auto& observer : event_observers_)
+ observer.OnDOMStorageItemRemoved(area, key, old_value, page_url);
}
void DOMStorageContextImpl::NotifyAreaCleared(
const DOMStorageArea* area,
const GURL& page_url) {
- FOR_EACH_OBSERVER(
- EventObserver, event_observers_,
- OnDOMStorageAreaCleared(area, page_url));
+ for (auto& observer : event_observers_)
+ observer.OnDOMStorageAreaCleared(area, page_url);
}
int64_t DOMStorageContextImpl::AllocateSessionId() {
« no previous file with comments | « content/browser/devtools/service_worker_devtools_manager.cc ('k') | content/browser/download/download_item_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698