Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(941)

Unified Diff: chrome/browser/chromeos/login/user_image.cc

Issue 10830111: [cros] Encode user images when set from WebUI (regression fix). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review fixes Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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) {
+ 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

Powered by Google App Engine
This is Rietveld 408576698