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

Unified Diff: chrome/browser/chromeos/login/user_image_screen.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_screen.cc
diff --git a/chrome/browser/chromeos/login/user_image_screen.cc b/chrome/browser/chromeos/login/user_image_screen.cc
index dd23b949e53c5c9f431205da10eebbfb59a561df..6122e5ec0a5ae7918403b0116251188b916b6e38 100644
--- a/chrome/browser/chromeos/login/user_image_screen.cc
+++ b/chrome/browser/chromeos/login/user_image_screen.cc
@@ -116,8 +116,10 @@ void UserImageScreen::StopCamera() {
void UserImageScreen::OnPhotoTaken(const gfx::ImageSkia& image) {
UserManager* user_manager = UserManager::Get();
+ // TODO(ivankr): once old camera UI is gone, there's raw data in image
+ // decoder, pass UserImage and user it instead.
user_manager->SaveUserImage(user_manager->GetLoggedInUser().email(),
- UserImage(image));
+ UserImage::CreateAndEncode(image));
get_screen_observer()->OnExit(ScreenObserver::USER_IMAGE_SELECTED);

Powered by Google App Engine
This is Rietveld 408576698