Reduce usage of FOR_EACH_OBSERVER macro in components/

components/offline_pages/offline_page_model_impl.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/offline_page_model_impl.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
@@ skipping 772 matching lines @@
   }
   InformSavePageDone(callback, result, offline_page.client_id,
                      offline_page.offline_id);
   if (result == SavePageResult::SUCCESS) {
     DeleteExistingPagesWithSameURL(offline_page);
   } else {
     PostClearStorageIfNeededTask();
   }
 
   DeletePendingArchiver(archiver);
-  FOR_EACH_OBSERVER(Observer, observers_, OfflinePageModelChanged(this));
+  for (Observer& observer : observers_)
+    observer.OfflinePageModelChanged(this);
 }
 
 void OfflinePageModelImpl::OnMarkPageAccesseDone(
     const OfflinePageItem& offline_page_item,
     std::unique_ptr<OfflinePagesUpdateResult> result) {
   // Update the item in the cache only upon success.
   if (result->updated_items.size() > 0)
     offline_pages_[offline_page_item.offline_id] = offline_page_item;
 
   // No need to fire OfflinePageModelChanged event since updating access info
@@ skipping 27 matching lines @@
 
   // Create Storage Manager.
   storage_manager_.reset(new OfflinePageStorageManager(
       this, GetPolicyController(), archive_manager_.get()));
 
   // Run all the delayed tasks.
   for (const auto& delayed_task : delayed_tasks_)
     delayed_task.Run();
   delayed_tasks_.clear();
 
-  FOR_EACH_OBSERVER(Observer, observers_, OfflinePageModelLoaded(this));
+  for (Observer& observer : observers_)
+    observer.OfflinePageModelLoaded(this);
 
   CheckMetadataConsistency();
 
   base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
       FROM_HERE, base::Bind(&OfflinePageModelImpl::ClearStorageIfNeeded,
                             weak_ptr_factory_.GetWeakPtr(),
                             base::Bind(&OfflinePageModelImpl::OnStorageCleared,
                                        weak_ptr_factory_.GetWeakPtr())),
       kStorageManagerStartingDelay);
 }
@@ skipping 88 matching lines @@
   for (const auto& page : result->updated_items) {
     int64_t offline_id = page.offline_id;
     offline_event_logger_.RecordPageDeleted(std::to_string(offline_id));
     auto iter = offline_pages_.find(offline_id);
     if (iter == offline_pages_.end())
       continue;
     offline_pages_.erase(iter);
   }
 
   for (const auto& page : result->updated_items) {
-    FOR_EACH_OBSERVER(Observer, observers_,
-                      OfflinePageDeleted(page.offline_id, page.client_id));
+    for (Observer& observer : observers_)
+      observer.OfflinePageDeleted(page.offline_id, page.client_id);
   }
 
   // TODO(fgorski): React the FAILED_INITIALIZATION, FAILED_RESET here.
   // TODO(fgorski): We need a better callback interface for the Remove action on
   // the this class. Currently removing an item that does not exist is
   // considered a success, but not called out as such to the caller.
   DeletePageResult delete_result;
   if (result->store_state == StoreState::LOADED)
     delete_result = DeletePageResult::SUCCESS;
   else
@@ skipping 118 matching lines @@
   }
 
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, task);
 }
 
 base::Time OfflinePageModelImpl::GetCurrentTime() const {
   return testing_clock_ ? testing_clock_->Now() : base::Time::Now();
 }
 
 }  // namespace offline_pages