[cros] Encode user images when set from WebUI (regression fix).

chrome/browser/chromeos/login/user_image.cc

Index: chrome/browser/chromeos/login/user_image.cc
diff --git a/chrome/browser/chromeos/login/user_image.cc b/chrome/browser/chromeos/login/user_image.cc
index 088e2ff260fd95a06d5aa6633edf3f8055561981..fd00c1ecf4e39451d558e72c5b87a32e7b17970a 100644
--- a/chrome/browser/chromeos/login/user_image.cc
+++ b/chrome/browser/chromeos/login/user_image.cc
@@ -23,6 +23,13 @@ bool IsAnimatedImage(const UserImage::RawImage& data) {
 
 }  // namespace
 
+// static
+UserImage UserImage::CreateAndEncode(const gfx::ImageSkia& image) {

[Nikita (slow), 2012/08/01 03:56:18]

const UserImage&

[Ivan Korotkov, 2012/08/01 21:51:28]

Uh, you can't return a const reference to a stack-allocated object. This one is
subject to return value optimization, though. Also, I'm going to make RawImage
be RefCountedMemory, anyway.

+  RawImage raw_image;
+  return gfx::PNGCodec::EncodeBGRASkBitmap(image, false, &raw_image) ?
+      UserImage(image, raw_image) : UserImage(image);
+}
+
 UserImage::UserImage()
     : has_raw_image_(false),
       has_animated_image_(false) {
@@ -52,4 +59,8 @@ UserImage::UserImage(const gfx::ImageSkia& image,
 
 UserImage::~UserImage() {}
 
+void UserImage::DiscardRawImage() {
+  RawImage().swap(raw_image_);  // Clear |raw_image_|.
+}
+
 }  // namespace chromeos