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

Unified Diff: ash/desktop_background/desktop_background_controller.h

Issue 110463004: ash: Fix DesktopBackgroundController getting stuck. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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: ash/desktop_background/desktop_background_controller.h
diff --git a/ash/desktop_background/desktop_background_controller.h b/ash/desktop_background/desktop_background_controller.h
index 4cfbc8ea5dfad68de57c45febbe3a28f060ca8d7..447e5e3e00deeff4a78596324d151681ddbbb90a 100644
--- a/ash/desktop_background/desktop_background_controller.h
+++ b/ash/desktop_background/desktop_background_controller.h
@@ -109,7 +109,7 @@ class ASH_EXPORT DesktopBackgroundController
void SetCustomWallpaper(const gfx::ImageSkia& image, WallpaperLayout layout);
// Cancels the current wallpaper loading operation.
- void CancelPendingWallpaperOperation();
+ void CancelDefaultWallpaperLoader();
DaveMoore 2013/12/26 21:00:35 Nit: Is the comment still correct?
Daniel Erat 2013/12/26 21:01:52 Yes, but I'll update it to be a bit clearer. (Only
// Creates an empty wallpaper. Some tests require a wallpaper widget is ready
// when running. However, the wallpaper widgets are now created asynchronously
@@ -211,7 +211,8 @@ class ASH_EXPORT DesktopBackgroundController
gfx::Size current_max_display_size_;
- scoped_refptr<WallpaperLoader> wallpaper_loader_;
+ // Loads default wallpaper from disk.
+ scoped_refptr<WallpaperLoader> default_wallpaper_loader_;
base::WeakPtrFactory<DesktopBackgroundController> weak_ptr_factory_;

Powered by Google App Engine
This is Rietveld 408576698