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

Unified Diff: chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc

Issue 2416763002: Replace FOR_EACH_OBSERVER in c/b/chromeos with range-based for (Closed)
Patch Set: 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: chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc
diff --git a/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc b/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc
index 34437eb8a1d1b963d3e3f1059b8766b32c26f364..5523f1b937643c2f25ce16dde96246ef8b69279d 100644
--- a/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc
+++ b/chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc
@@ -336,7 +336,7 @@ class WallpaperManager::PendingWallpaper :
// The only known case for this check to fail is global destruction during
// wallpaper load. It should never happen.
if (!BrowserThread::CurrentlyOn(BrowserThread::UI))
- return; // We are in a process of global destruction.
+ return; // We are in a process of global destruction.
timer.Stop(); // Erase reference to self.
@@ -881,8 +881,10 @@ void WallpaperManager::RemovePendingWallpaperFromList(
}
}
- if (loading_.empty())
- FOR_EACH_OBSERVER(Observer, observers_, OnPendingListEmptyForTesting());
+ if (loading_.empty()) {
+ for (auto& observer : observers_)
+ observer.OnPendingListEmptyForTesting();
+ }
}
void WallpaperManager::SetPolicyControlledWallpaper(

Powered by Google App Engine
This is Rietveld 408576698