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

Side by Side Diff: chrome/browser/chromeos/login/users/avatar/user_image_manager_impl.cc

Issue 1747843002: Rename raw_image() to image_bytes() in UserImage (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/login/users/avatar/user_image_manager_impl.h" 5 #include "chrome/browser/chromeos/login/users/avatar/user_image_manager_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 case user_manager::User::USER_IMAGE_PROFILE: 158 case user_manager::User::USER_IMAGE_PROFILE:
159 return default_user_image::kHistogramImageFromProfile; 159 return default_user_image::kHistogramImageFromProfile;
160 default: 160 default:
161 return image_index; 161 return image_index;
162 } 162 }
163 } 163 }
164 164
165 bool SaveImage(const user_manager::UserImage& user_image, 165 bool SaveImage(const user_manager::UserImage& user_image,
166 const base::FilePath& image_path) { 166 const base::FilePath& image_path) {
167 user_manager::UserImage safe_image; 167 user_manager::UserImage safe_image;
168 const user_manager::UserImage::RawImage* encoded_image = NULL; 168 const user_manager::UserImage::Bytes* encoded_image = NULL;
169 if (!user_image.is_safe_format()) { 169 if (!user_image.is_safe_format()) {
170 safe_image = user_manager::UserImage::CreateAndEncode(user_image.image()); 170 safe_image = user_manager::UserImage::CreateAndEncode(user_image.image());
171 encoded_image = &safe_image.raw_image(); 171 encoded_image = &safe_image.image_bytes();
172 UMA_HISTOGRAM_MEMORY_KB("UserImage.RecodedJpegSize", encoded_image->size()); 172 UMA_HISTOGRAM_MEMORY_KB("UserImage.RecodedJpegSize", encoded_image->size());
173 } else if (user_image.has_raw_image()) { 173 } else if (user_image.has_image_bytes()) {
174 encoded_image = &user_image.raw_image(); 174 encoded_image = &user_image.image_bytes();
175 } else { 175 } else {
176 NOTREACHED() << "Raw image missing."; 176 NOTREACHED() << "image data bytes missing.";
177 return false; 177 return false;
178 } 178 }
179 179
180 if (!encoded_image->size() || 180 if (!encoded_image->size() ||
181 base::WriteFile(image_path, 181 base::WriteFile(image_path,
182 reinterpret_cast<const char*>(&(*encoded_image)[0]), 182 reinterpret_cast<const char*>(&(*encoded_image)[0]),
183 encoded_image->size()) == -1) { 183 encoded_image->size()) == -1) {
184 LOG(ERROR) << "Failed to save image to file."; 184 LOG(ERROR) << "Failed to save image to file.";
185 return false; 185 return false;
186 } 186 }
(...skipping 838 matching lines...) Expand 10 before | Expand all | Expand 10 after
1025 } 1025 }
1026 1026
1027 bool UserImageManagerImpl::IsUserLoggedInAndHasGaiaAccount() const { 1027 bool UserImageManagerImpl::IsUserLoggedInAndHasGaiaAccount() const {
1028 const user_manager::User* user = GetUser(); 1028 const user_manager::User* user = GetUser();
1029 if (!user) 1029 if (!user)
1030 return false; 1030 return false;
1031 return user->is_logged_in() && user->HasGaiaAccount(); 1031 return user->is_logged_in() && user->HasGaiaAccount();
1032 } 1032 }
1033 1033
1034 } // namespace chromeos 1034 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/login/users/avatar/user_image_loader.cc ('k') | chrome/browser/ui/webui/chromeos/image_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698