Reduce usage of FOR_EACH_OBSERVER macro in components/

components/offline_pages/downloads/download_ui_adapter.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/offline_pages/downloads/download_ui_adapter.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
@@ skipping 78 matching lines @@
 
 void DownloadUIAdapter::OfflinePageDeleted(
   int64_t offline_id, const ClientId& client_id) {
   if (!IsVisibleInUI(client_id))
     return;
   std::string guid = client_id.id;
   DownloadUIItems::const_iterator it = items_.find(guid);
   if (it == items_.end())
     return;
   items_.erase(it);
-  FOR_EACH_OBSERVER(Observer, observers_, ItemDeleted(guid));
+  for (Observer& observer : observers_)
+    observer.ItemDeleted(guid);
 }
 
 std::vector<const DownloadUIItem*> DownloadUIAdapter::GetAllItems() const {
   std::vector<const DownloadUIItem*> result;
   for (const auto& item : items_)
     result.push_back(item.second->ui_item.get());
   return result;
 }
 
 const DownloadUIItem*
@@ skipping 58 matching lines @@
     return;
   for (const auto& page : pages) {
     if (IsVisibleInUI(page.client_id)) {
       std::string guid = page.client_id.id;
       DCHECK(items_.find(guid) == items_.end());
       items_[guid] = base::MakeUnique<ItemInfo>(page);
     }
   }
   model_->AddObserver(this);
   state_ = State::LOADED;
-  FOR_EACH_OBSERVER(Observer, observers_, ItemsLoaded());
+  for (Observer& observer : observers_)
+    observer.ItemsLoaded();
 }
 
 void DownloadUIAdapter::NotifyItemsLoaded(Observer* observer) {
   if (observer && observers_.HasObserver(observer))
     observer->ItemsLoaded();
 }
 
 // This method is only called by OPM when a single item added.
 // TODO(dimich): change OPM to have real OnPageAdded/OnPageUpdated and
 // simplify this code.
 void DownloadUIAdapter::OnOfflinePagesChanged(
     const MultipleOfflinePageItemResult& pages) {
   std::vector<std::string> added_guids;
   for (const auto& page : pages) {
     if (!IsVisibleInUI(page.client_id))  // Item should be filtered out.
       continue;
     const std::string& guid = page.client_id.id;
     if (items_.find(guid) != items_.end())  // Item already exists.
       continue;
     items_[guid] = base::MakeUnique<ItemInfo>(page);
     added_guids.push_back(guid);
   }
   for (auto& guid : added_guids) {
     const DownloadUIItem& item = *(items_.find(guid)->second->ui_item.get());
-    FOR_EACH_OBSERVER(Observer, observers_, ItemAdded(item));
+    for (Observer& observer : observers_)
+      observer.ItemAdded(item);
   }
 }
 
 void DownloadUIAdapter::OnDeletePagesDone(DeletePageResult result) {
   // TODO(dimich): Consider adding UMA to record user actions.
 }
 
 bool DownloadUIAdapter::IsVisibleInUI(const ClientId& client_id) {
   const std::string& name_space = client_id.name_space;
   return model_->GetPolicyController()->IsSupportedByDownload(name_space) &&
          base::IsValidGUID(client_id.id);
 }
 
 }  // namespace offline_pages