Some cleanup of WallpaperManager.

chrome/browser/chromeos/login/user.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USER_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_USER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 89 matching lines @@
      OAUTH2_TOKEN_STATUS_VALID   = 4,
   } OAuthTokenStatus;
 
   // Returned as |image_index| when user-selected file or photo is used as
   // user image.
   static const int kExternalImageIndex = -1;
   // Returned as |image_index| when user profile image is used as user image.
   static const int kProfileImageIndex = -2;
   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
   };
 
   // Returns the user type.
   virtual UserType GetType() const = 0;
 
   // The email the user used to log in.
   const std::string& email() const { return email_; }
 
   // Returns the human name to display for this user.
@@ skipping 193 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(User);
 };
 
 // List of known users.
 typedef std::vector<User*> UserList;
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_USER_H_