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

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

Issue 2776853002: Make UMA_HISTOGRAM_ENUMERATION work with scoped enums. (Closed)
Patch Set: rebase Created 3 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 side-by-side diff with in-line comments
Download patch
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 d1796b121325d7b2f7d146b0c00553fa424c065e..e5661d299db19e356a83355987f2894083ce1303 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
@@ -352,16 +352,16 @@ void ChangePictureOptionsHandler::HandleSelectImage(
user_manager::UserImage::CreateAndEncode(
previous_image_, user_manager::UserImage::FORMAT_JPEG));
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
- default_user_image::kHistogramImageOld,
- default_user_image::kHistogramImagesCount);
+ UMA_HISTOGRAM_EXACT_LINEAR("UserImage.ChangeChoice",
+ default_user_image::kHistogramImageOld,
+ default_user_image::kHistogramImagesCount);
VLOG(1) << "Selected old user image";
} else if (image_type == "default" &&
default_user_image::IsDefaultImageUrl(image_url, &image_index)) {
// One of the default user images.
user_image_manager->SaveUserDefaultImageIndex(image_index);
- UMA_HISTOGRAM_ENUMERATION(
+ UMA_HISTOGRAM_EXACT_LINEAR(
"UserImage.ChangeChoice",
default_user_image::GetDefaultImageHistogramValue(image_index),
default_user_image::kHistogramImagesCount);
@@ -379,14 +379,14 @@ void ChangePictureOptionsHandler::HandleSelectImage(
user_image_manager->SaveUserImageFromProfileImage();
if (previous_image_index_ == user_manager::User::USER_IMAGE_PROFILE) {
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
- default_user_image::kHistogramImageOld,
- default_user_image::kHistogramImagesCount);
+ UMA_HISTOGRAM_EXACT_LINEAR("UserImage.ChangeChoice",
+ default_user_image::kHistogramImageOld,
+ default_user_image::kHistogramImagesCount);
VLOG(1) << "Selected old (profile) user image";
} else {
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
- default_user_image::kHistogramImageFromProfile,
- default_user_image::kHistogramImagesCount);
+ UMA_HISTOGRAM_EXACT_LINEAR("UserImage.ChangeChoice",
+ default_user_image::kHistogramImageFromProfile,
+ default_user_image::kHistogramImagesCount);
VLOG(1) << "Selected profile image";
}
} else {
@@ -404,9 +404,9 @@ void ChangePictureOptionsHandler::FileSelected(const base::FilePath& path,
ChromeUserManager::Get()
->GetUserImageManager(GetUser()->GetAccountId())
->SaveUserImageFromFile(path);
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
- default_user_image::kHistogramImageFromFile,
- default_user_image::kHistogramImagesCount);
+ UMA_HISTOGRAM_EXACT_LINEAR("UserImage.ChangeChoice",
+ default_user_image::kHistogramImageFromFile,
+ default_user_image::kHistogramImagesCount);
VLOG(1) << "Selected image from file";
}
@@ -416,9 +416,9 @@ void ChangePictureOptionsHandler::SetImageFromCamera(
->GetUserImageManager(GetUser()->GetAccountId())
->SaveUserImage(user_manager::UserImage::CreateAndEncode(
photo, user_manager::UserImage::FORMAT_JPEG));
- UMA_HISTOGRAM_ENUMERATION("UserImage.ChangeChoice",
- default_user_image::kHistogramImageFromCamera,
- default_user_image::kHistogramImagesCount);
+ UMA_HISTOGRAM_EXACT_LINEAR("UserImage.ChangeChoice",
+ default_user_image::kHistogramImageFromCamera,
+ default_user_image::kHistogramImagesCount);
VLOG(1) << "Selected camera photo";
}

Powered by Google App Engine
This is Rietveld 408576698