Index: chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc |
diff --git a/chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc b/chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc |
index 61f420c14805999e5fab3be688b221cfb57916fb..53e91c280ed28ea94fe3846014d07108b1a587d0 100644 |
--- a/chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc |
+++ b/chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc |
@@ -15,7 +15,6 @@ |
#include "base/values.h" |
#include "chrome/browser/chrome_notification_types.h" |
#include "chrome/browser/chromeos/camera_presence_notifier.h" |
-#include "chrome/browser/chromeos/login/users/avatar/default_user_images.h" |
#include "chrome/browser/chromeos/login/users/avatar/user_image_manager.h" |
#include "chrome/browser/chromeos/login/users/user_manager.h" |
#include "chrome/browser/chromeos/profiles/profile_helper.h" |
@@ -26,6 +25,7 @@ |
#include "chrome/common/chrome_switches.h" |
#include "chrome/common/url_constants.h" |
#include "chromeos/audio/chromeos_sounds.h" |
+#include "components/user_manager/user_image/default_user_images.h" |
#include "components/user_manager/user_image/user_image.h" |
#include "content/public/browser/browser_thread.h" |
#include "content/public/browser/notification_service.h" |
@@ -166,14 +166,18 @@ void ChangePictureOptionsHandler::RegisterMessages() { |
void ChangePictureOptionsHandler::SendDefaultImages() { |
base::ListValue image_urls; |
- for (int i = kFirstDefaultImageIndex; i < kDefaultImagesCount; ++i) { |
+ for (int i = user_manager::kFirstDefaultImageIndex; |
+ i < user_manager::kDefaultImagesCount; |
+ ++i) { |
scoped_ptr<base::DictionaryValue> image_data(new base::DictionaryValue); |
- image_data->SetString("url", GetDefaultImageUrl(i)); |
+ image_data->SetString("url", user_manager::GetDefaultImageUrl(i)); |
image_data->SetString( |
- "author", l10n_util::GetStringUTF16(kDefaultImageAuthorIDs[i])); |
+ "author", |
+ l10n_util::GetStringUTF16(user_manager::kDefaultImageAuthorIDs[i])); |
image_data->SetString( |
- "website", l10n_util::GetStringUTF16(kDefaultImageWebsiteIDs[i])); |
- image_data->SetString("title", GetDefaultImageDescription(i)); |
+ "website", |
+ l10n_util::GetStringUTF16(user_manager::kDefaultImageWebsiteIDs[i])); |
+ image_data->SetString("title", user_manager::GetDefaultImageDescription(i)); |
image_urls.Append(image_data.release()); |
} |
web_ui()->CallJavascriptFunction("ChangePictureOptions.setDefaultImages", |
@@ -281,16 +285,17 @@ void ChangePictureOptionsHandler::SendSelectedImage() { |
} |
default: { |
DCHECK(previous_image_index_ >= 0 && |
- previous_image_index_ < kDefaultImagesCount); |
- if (previous_image_index_ >= kFirstDefaultImageIndex) { |
+ previous_image_index_ < user_manager::kDefaultImagesCount); |
+ if (previous_image_index_ >= user_manager::kFirstDefaultImageIndex) { |
// User has image from the current set of default images. |
- base::StringValue image_url(GetDefaultImageUrl(previous_image_index_)); |
+ base::StringValue image_url( |
+ user_manager::GetDefaultImageUrl(previous_image_index_)); |
web_ui()->CallJavascriptFunction( |
"ChangePictureOptions.setSelectedImage", image_url); |
} else { |
// User has an old default image, so present it in the same manner as a |
// previous image from file. |
- SendOldImage(GetDefaultImageUrl(previous_image_index_)); |
+ SendOldImage(user_manager::GetDefaultImageUrl(previous_image_index_)); |
} |
} |
} |
@@ -348,17 +353,18 @@ void ChangePictureOptionsHandler::HandleSelectImage( |
user_manager::UserImage::CreateAndEncode(previous_image_)); |
UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
- kHistogramImageOld, |
- kHistogramImagesCount); |
+ user_manager::kHistogramImageOld, |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected old user image"; |
} else if (image_type == "default" && |
- IsDefaultImageUrl(image_url, &image_index)) { |
+ user_manager::IsDefaultImageUrl(image_url, &image_index)) { |
// One of the default user images. |
user_image_manager->SaveUserDefaultImageIndex(image_index); |
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
- GetDefaultImageHistogramValue(image_index), |
- kHistogramImagesCount); |
+ UMA_HISTOGRAM_ENUMERATION( |
+ "UserImage.ChangeChoice", |
+ user_manager::GetDefaultImageHistogramValue(image_index), |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected default user image: " << image_index; |
} else if (image_type == "camera") { |
// Camera image is selected. |
@@ -375,13 +381,13 @@ void ChangePictureOptionsHandler::HandleSelectImage( |
if (previous_image_index_ == User::kProfileImageIndex) { |
UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
- kHistogramImageOld, |
- kHistogramImagesCount); |
+ user_manager::kHistogramImageOld, |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected old (profile) user image"; |
} else { |
UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
- kHistogramImageFromProfile, |
- kHistogramImagesCount); |
+ user_manager::kHistogramImageFromProfile, |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected profile image"; |
} |
} else { |
@@ -399,8 +405,9 @@ void ChangePictureOptionsHandler::FileSelected(const base::FilePath& path, |
UserManager* user_manager = UserManager::Get(); |
user_manager->GetUserImageManager(GetUser()->email())-> |
SaveUserImageFromFile(path); |
- UMA_HISTOGRAM_ENUMERATION( |
- "UserImage.ChangeChoice", kHistogramImageFromFile, kHistogramImagesCount); |
+ UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
+ user_manager::kHistogramImageFromFile, |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected image from file"; |
} |
@@ -410,8 +417,8 @@ void ChangePictureOptionsHandler::SetImageFromCamera( |
user_manager->GetUserImageManager(GetUser()->email()) |
->SaveUserImage(user_manager::UserImage::CreateAndEncode(photo)); |
UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice", |
- kHistogramImageFromCamera, |
- kHistogramImagesCount); |
+ user_manager::kHistogramImageFromCamera, |
+ user_manager::kHistogramImagesCount); |
VLOG(1) << "Selected camera photo"; |
} |