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

Side by Side Diff: chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Fix Win GN build. Created 5 years, 1 month 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/ui/webui/options/chromeos/change_picture_options_handle r.h" 5 #include "chrome/browser/ui/webui/options/chromeos/change_picture_options_handle r.h"
6 6
7 #include "ash/audio/sounds.h" 7 #include "ash/audio/sounds.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/bind_helpers.h" 9 #include "base/bind_helpers.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
298 void ChangePictureOptionsHandler::SendProfileImage(const gfx::ImageSkia& image, 298 void ChangePictureOptionsHandler::SendProfileImage(const gfx::ImageSkia& image,
299 bool should_select) { 299 bool should_select) {
300 base::StringValue data_url(webui::GetBitmapDataUrl(*image.bitmap())); 300 base::StringValue data_url(webui::GetBitmapDataUrl(*image.bitmap()));
301 base::FundamentalValue select(should_select); 301 base::FundamentalValue select(should_select);
302 web_ui()->CallJavascriptFunction("ChangePictureOptions.setProfileImage", 302 web_ui()->CallJavascriptFunction("ChangePictureOptions.setProfileImage",
303 data_url, select); 303 data_url, select);
304 } 304 }
305 305
306 void ChangePictureOptionsHandler::UpdateProfileImage() { 306 void ChangePictureOptionsHandler::UpdateProfileImage() {
307 UserImageManager* user_image_manager = 307 UserImageManager* user_image_manager =
308 ChromeUserManager::Get()->GetUserImageManager(GetUser()->email()); 308 ChromeUserManager::Get()->GetUserImageManager(GetUser()->GetAccountId());
309 // If we have a downloaded profile image and haven't sent it in 309 // If we have a downloaded profile image and haven't sent it in
310 // |SendSelectedImage|, send it now (without selecting). 310 // |SendSelectedImage|, send it now (without selecting).
311 if (previous_image_index_ != user_manager::User::USER_IMAGE_PROFILE && 311 if (previous_image_index_ != user_manager::User::USER_IMAGE_PROFILE &&
312 !user_image_manager->DownloadedProfileImage().isNull()) 312 !user_image_manager->DownloadedProfileImage().isNull())
313 SendProfileImage(user_image_manager->DownloadedProfileImage(), false); 313 SendProfileImage(user_image_manager->DownloadedProfileImage(), false);
314 314
315 user_image_manager->DownloadProfileImage(kProfileDownloadReason); 315 user_image_manager->DownloadProfileImage(kProfileDownloadReason);
316 } 316 }
317 317
318 void ChangePictureOptionsHandler::SendOldImage(const std::string& image_url) { 318 void ChangePictureOptionsHandler::SendOldImage(const std::string& image_url) {
(...skipping 10 matching lines...) Expand all
329 args->GetSize() != 2 || 329 args->GetSize() != 2 ||
330 !args->GetString(0, &image_url) || 330 !args->GetString(0, &image_url) ||
331 !args->GetString(1, &image_type)) { 331 !args->GetString(1, &image_type)) {
332 NOTREACHED(); 332 NOTREACHED();
333 return; 333 return;
334 } 334 }
335 DCHECK(!image_url.empty()); 335 DCHECK(!image_url.empty());
336 DCHECK(!image_type.empty()); 336 DCHECK(!image_type.empty());
337 337
338 UserImageManager* user_image_manager = 338 UserImageManager* user_image_manager =
339 ChromeUserManager::Get()->GetUserImageManager(GetUser()->email()); 339 ChromeUserManager::Get()->GetUserImageManager(GetUser()->GetAccountId());
340 int image_index = user_manager::User::USER_IMAGE_INVALID; 340 int image_index = user_manager::User::USER_IMAGE_INVALID;
341 bool waiting_for_camera_photo = false; 341 bool waiting_for_camera_photo = false;
342 342
343 if (image_type == "old") { 343 if (image_type == "old") {
344 // Previous image (from camera or manually uploaded) re-selected. 344 // Previous image (from camera or manually uploaded) re-selected.
345 DCHECK(!previous_image_.isNull()); 345 DCHECK(!previous_image_.isNull());
346 user_image_manager->SaveUserImage( 346 user_image_manager->SaveUserImage(
347 user_manager::UserImage::CreateAndEncode(previous_image_)); 347 user_manager::UserImage::CreateAndEncode(previous_image_));
348 348
349 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", 349 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 389
390 // Ignore the result of the previous decoding if it's no longer needed. 390 // Ignore the result of the previous decoding if it's no longer needed.
391 if (!waiting_for_camera_photo) 391 if (!waiting_for_camera_photo)
392 ImageDecoder::Cancel(this); 392 ImageDecoder::Cancel(this);
393 } 393 }
394 394
395 void ChangePictureOptionsHandler::FileSelected(const base::FilePath& path, 395 void ChangePictureOptionsHandler::FileSelected(const base::FilePath& path,
396 int index, 396 int index,
397 void* params) { 397 void* params) {
398 ChromeUserManager::Get() 398 ChromeUserManager::Get()
399 ->GetUserImageManager(GetUser()->email()) 399 ->GetUserImageManager(GetUser()->GetAccountId())
400 ->SaveUserImageFromFile(path); 400 ->SaveUserImageFromFile(path);
401 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", 401 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
402 user_manager::kHistogramImageFromFile, 402 user_manager::kHistogramImageFromFile,
403 user_manager::kHistogramImagesCount); 403 user_manager::kHistogramImagesCount);
404 VLOG(1) << "Selected image from file"; 404 VLOG(1) << "Selected image from file";
405 } 405 }
406 406
407 void ChangePictureOptionsHandler::SetImageFromCamera( 407 void ChangePictureOptionsHandler::SetImageFromCamera(
408 const gfx::ImageSkia& photo) { 408 const gfx::ImageSkia& photo) {
409 ChromeUserManager::Get() 409 ChromeUserManager::Get()
410 ->GetUserImageManager(GetUser()->email()) 410 ->GetUserImageManager(GetUser()->GetAccountId())
411 ->SaveUserImage(user_manager::UserImage::CreateAndEncode(photo)); 411 ->SaveUserImage(user_manager::UserImage::CreateAndEncode(photo));
412 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", 412 UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
413 user_manager::kHistogramImageFromCamera, 413 user_manager::kHistogramImageFromCamera,
414 user_manager::kHistogramImagesCount); 414 user_manager::kHistogramImagesCount);
415 VLOG(1) << "Selected camera photo"; 415 VLOG(1) << "Selected camera photo";
416 } 416 }
417 417
418 void ChangePictureOptionsHandler::SetCameraPresent(bool present) { 418 void ChangePictureOptionsHandler::SetCameraPresent(bool present) {
419 base::FundamentalValue present_value(present); 419 base::FundamentalValue present_value(present);
420 420
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
463 Profile* profile = Profile::FromWebUI(web_ui()); 463 Profile* profile = Profile::FromWebUI(web_ui());
464 const user_manager::User* user = 464 const user_manager::User* user =
465 ProfileHelper::Get()->GetUserByProfile(profile); 465 ProfileHelper::Get()->GetUserByProfile(profile);
466 if (!user) 466 if (!user)
467 return user_manager::UserManager::Get()->GetActiveUser(); 467 return user_manager::UserManager::Get()->GetActiveUser();
468 return user; 468 return user;
469 } 469 }
470 470
471 } // namespace options 471 } // namespace options
472 } // namespace chromeos 472 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698