Merge 107584 - [cros] Don't succeed if user has default profile picture. Merge 108860 - [cros] Fi...

chrome/browser/chromeos/login/user_image_screen.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/login/user_image_screen.h"
 
 #include "base/compiler_specific.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/chromeos/login/default_user_images.h"
 #include "chrome/browser/chromeos/login/login_utils.h"
@@ skipping 169 matching lines @@
   else if (actor_ && actor_->IsCapturing())
     StartCamera();
 }
 
 void UserImageScreen::OnDownloadSuccess(const SkBitmap& image) {
   DCHECK(profile_image_load_start_time_.is_null());
 
   base::TimeDelta delta = base::Time::Now() - profile_image_load_start_time_;
   VLOG(1) << "Profile image download time: " << delta.InSecondsF();
   UMA_HISTOGRAM_TIMES("UserImage.FirstTimeProfileImageDownload", delta);
+  UMA_HISTOGRAM_ENUMERATION("UserImageDownloadResult.NewUser",
+                            ProfileImageDownloader::kDownloadSuccess,
+                            ProfileImageDownloader::kDownloadResultsCount);
 
-  // TODO(avayvod): Check for the default image.
   if (actor_)
     actor_->AddProfileImage(image);
 }
 
+void UserImageScreen::OnDownloadFailure() {
+  VLOG(1) << "Download of profile image for new user failed.";
+  UMA_HISTOGRAM_ENUMERATION("UserImageDownloadResult.NewUser",
+                            ProfileImageDownloader::kDownloadFailure,
+                            ProfileImageDownloader::kDownloadResultsCount);
+}
+
+void UserImageScreen::OnDownloadDefaultImage() {
+  VLOG(1) << "New user has the default profile image.";
+  UMA_HISTOGRAM_ENUMERATION("UserImageDownloadResult.NewUser",
+                            ProfileImageDownloader::kDownloadDefault,
+                            ProfileImageDownloader::kDownloadResultsCount);
+}
+
 }  // namespace chromeos