Some cleanup of WallpaperManager.

chrome/browser/chromeos/login/user.h

Index: chrome/browser/chromeos/login/user.h
diff --git a/chrome/browser/chromeos/login/user.h b/chrome/browser/chromeos/login/user.h
index 0a4f28a3b835e94ff5661306045d072c7a7d22b4..0bc589762cafd3697a52377b8990dd5c41fd2776 100644
--- a/chrome/browser/chromeos/login/user.h
+++ b/chrome/browser/chromeos/login/user.h
@@ -108,12 +108,11 @@ class User {
   static const int kInvalidImageIndex = -3;
 
   enum WallpaperType {

[bshe, 2014/04/02 15:11:10]

Could you document the removed enum please? This enum is used for UMA stats and
the value shouldn't be reused in the future. A document could reduce the risk of
these enum values being reused.

[Thiemo Nagel, 2014/04/02 15:39:56]

Done.

-    DAILY = 0,
-    CUSTOMIZED = 1,
-    DEFAULT = 2,
+    CUSTOMIZED = 1,  // Selected by user.

[bshe, 2014/04/02 15:11:10]

nit: Selected by user or Google pushed wallpaper.

[Thiemo Nagel, 2014/04/02 15:39:56]

How can Google push a wallpaper?  Are you referring to the "surprise me"
checkbox in the wallpaper picker?

+    DEFAULT = 2,     // Default.
     UNKNOWN = 3,
-    ONLINE = 4,
-    POLICY = 5,  // Controlled by policy, can't be changed by the user.
+    ONLINE = 4,      // WallpaperInfo.file denotes an URL.
+    POLICY = 5,      // Controlled by policy, can't be changed by the user.
     WALLPAPER_TYPE_COUNT = 6
   };