Remove FOR_EACH_OBSERVER macro usage in chrome/browser/extensions

chrome/browser/extensions/extension_error_reporter.cc

 // Copyright (c) 2012 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 "chrome/browser/extensions/extension_error_reporter.h"
 
 #include "build/build_config.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 42 matching lines @@
       content::Source<Profile>(Profile::FromBrowserContext(browser_context)),
       content::Details<const std::string>(&error));
 
   std::string path_str = base::UTF16ToUTF8(extension_path.LossyDisplayName());
   base::string16 message = base::UTF8ToUTF16(base::StringPrintf(
       "%s %s. %s",
       l10n_util::GetStringUTF8(IDS_EXTENSIONS_LOAD_ERROR_MESSAGE).c_str(),
       path_str.c_str(),
       error.c_str()));
   ReportError(message, be_noisy);
-  FOR_EACH_OBSERVER(Observer,
-                    observers_,
-                    OnLoadFailure(browser_context, extension_path, error));
+  for (auto& observer : observers_)
+    observer.OnLoadFailure(browser_context, extension_path, error);
 }
 
 void ExtensionErrorReporter::ReportError(const base::string16& message,
                                          bool be_noisy) {
   // NOTE: There won't be a |ui_task_runner_| in the unit test environment.
   CHECK(!ui_task_runner_ || ui_task_runner_->BelongsToCurrentThread())
       << "ReportError can only be called from the UI thread.";
 
   errors_.push_back(message);
 
@@ skipping 16 matching lines @@
   errors_.clear();
 }
 
 void ExtensionErrorReporter::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ExtensionErrorReporter::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }