[cros] Initial user image loading and repeated profile image loading fixes.

chrome/browser/chromeos/login/default_user_images.cc

Index: chrome/browser/chromeos/login/default_user_images.cc
diff --git a/chrome/browser/chromeos/login/default_user_images.cc b/chrome/browser/chromeos/login/default_user_images.cc
index 3757767222940a555dbf6224b6514f98ceb48337..fc0950956f63d3b9c9c51a43e17b84e0d396d717 100644
--- a/chrome/browser/chromeos/login/default_user_images.cc
+++ b/chrome/browser/chromeos/login/default_user_images.cc
@@ -10,6 +10,7 @@
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "grit/theme_resources.h"
+#include "ui/base/resource/resource_bundle.h"
 

[whywhat, 2011/11/11 15:40:42]

include SkBitmap.h?

[Ivan Korotkov, 2011/11/11 16:08:00]

Done.

 namespace chromeos {
 
@@ -93,6 +94,12 @@ bool IsDefaultImageUrl(const std::string url, int* image_id) {
   return IsDefaultImageString(url, kDefaultUrlPrefix, image_id);
 }
 
+const SkBitmap& GetDefaultImage(int index) {

[whywhat, 2011/11/11 15:40:42]

God bless us if we keep this reference in UserManager singleton and
ResourceBundle dies before it...
AFAIK, SkBitmap is okay to copy as it shares pixels data 

[Ivan Korotkov, 2011/11/11 16:08:00]

Blessed are we.

+  DCHECK(index >= 0 && index < kDefaultImagesCount);
+  return *ResourceBundle::GetSharedInstance().
+      GetBitmapNamed(kDefaultImageResources[index]);
+}
+
 // Resource IDs of default user images.
 const int kDefaultImageResources[] = {
   IDR_LOGIN_DEFAULT_USER,