Using random wallpaper until user select one

chrome/browser/chromeos/login/user_manager_impl.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_MANAGER_IMPL_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_USER_MANAGER_IMPL_H_
 #pragma once
 
 #include <string>
 
@@ skipping 211 matching lines @@
   // Cached flag of whether the currently logged-in user existed before this
   // login.
   bool is_current_user_new_;
 
   // Cached flag of whether the currently logged-in user is ephemeral. Storage
   // of persistent information is avoided for such users by not adding them to
   // the user list in local state, not downloading their custom user images and
   // mounting their cryptohomes using tmpfs.
   bool is_current_user_ephemeral_;
 
+  // Cache ephemeral user selected index in memory instead of local state.
+  int ephemeral_user_wallpaper_index_;

[Nikita (slow), 2012/03/26 15:31:58]

Do you think this may be moved somewhere to ash/* instead?

[bshe, 2012/03/26 16:12:10]

I can probably put it in ash/desktop_background/desktop_background_controller
with some plumbing work. I just thought it might be a good idea to put
everything user related to user_manager_impl. I didnt see similar cache pattern
used in this file though. I am wondering is there any particular reason why it
is not preferred? 

On 2012/03/26 15:31:58, Nikita Kostylev wrote:

[Nikita (slow), 2012/03/26 16:50:07]

I've thought that ash should have something like current wallpaper index but it
seems that it only has current wallpaper bitmap. And it doesn't make sense to
cache index in ash as it always is fetched from LocalState if needed. If we'd
have more such cases when we'll need to cache local_state data, we'll have to
provide some in memory storage.

+
   // Cached flag indicating whether ephemeral users are enabled. Defaults to
   // |false| if the value has not been read from trusted device policy yet.
   bool ephemeral_users_enabled_;
 
   // Cached name of device owner. Defaults to empty string if the value has not
   // been read from trusted device policy yet.
   std::string owner_email_;
 
   content::NotificationRegistrar registrar_;
 
@@ skipping 23 matching lines @@
 
   // Data URL for |downloaded_profile_image_|.
   std::string downloaded_profile_image_data_url_;
 
   DISALLOW_COPY_AND_ASSIGN(UserManagerImpl);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_USER_MANAGER_IMPL_H_