Chromium Code Reviews| Index: chrome/browser/android/bookmarks/partner_bookmarks_shim.cc |
| diff --git a/chrome/browser/android/bookmarks/partner_bookmarks_shim.cc b/chrome/browser/android/bookmarks/partner_bookmarks_shim.cc |
| index 7c095e45bd0595242347155bbf82fe527b65f392..6ef0c2db08084e8ab4c5802f3d4ec087d45b0cb8 100644 |
| --- a/chrome/browser/android/bookmarks/partner_bookmarks_shim.cc |
| +++ b/chrome/browser/android/bookmarks/partner_bookmarks_shim.cc |
| @@ -123,8 +123,8 @@ void PartnerBookmarksShim::RenameBookmark(const BookmarkNode* node, |
| const NodeRenamingMapKey key(node->url(), node->GetTitle()); |
| node_rename_remove_map_[key] = title; |
| SaveNodeMapping(); |
| - FOR_EACH_OBSERVER(PartnerBookmarksShim::Observer, observers_, |
| - PartnerShimChanged(this)); |
| + for (PartnerBookmarksShim::Observer& observer : observers_) |
|
Bernhard Bauer
2016/10/19 11:42:51
I think you could just use Observer (you're inside
|
| + observer.PartnerShimChanged(this); |
| } |
| void PartnerBookmarksShim::AddObserver( |
| @@ -180,8 +180,8 @@ void PartnerBookmarksShim::SetPartnerBookmarksRoot( |
| DCHECK_CURRENTLY_ON(BrowserThread::UI); |
| g_partner_model_keeper.Get().partner_bookmarks_root = std::move(root_node); |
| g_partner_model_keeper.Get().loaded = true; |
| - FOR_EACH_OBSERVER(PartnerBookmarksShim::Observer, observers_, |
| - PartnerShimLoaded(this)); |
| + for (PartnerBookmarksShim::Observer& observer : observers_) |
| + observer.PartnerShimLoaded(this); |
| } |
| PartnerBookmarksShim::NodeRenamingMapKey::NodeRenamingMapKey( |
| @@ -221,8 +221,8 @@ PartnerBookmarksShim::PartnerBookmarksShim(PrefService* prefs) |
| } |
| PartnerBookmarksShim::~PartnerBookmarksShim() { |
| - FOR_EACH_OBSERVER(PartnerBookmarksShim::Observer, observers_, |
| - ShimBeingDeleted(this)); |
| + for (PartnerBookmarksShim::Observer& observer : observers_) |
| + observer.ShimBeingDeleted(this); |
| } |
| const BookmarkNode* PartnerBookmarksShim::GetNodeByID( |