[cros] UMA for profile image download times.

chrome/browser/chromeos/login/user_manager.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_manager.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
@@ skipping 80 matching lines @@
 enum ProfileDownloadResult {
   kDownloadSuccessChanged,
   kDownloadSuccess,
   kDownloadFailure,
   kDownloadDefault,
 
   // Must be the last, convenient count.
   kDownloadResultsCount
 };
 
+// Time histogram name for the default profile image download.
+const char kProfileDownloadDefaultTime[] =
+    "UserImage.ProfileDownloadDefaultTime";
+// Time histogram name for a failed profile image download.
+const char kProfileDownloadFailureTime[] =
+    "UserImage.ProfileDownloadFailureTime";
+// Time histogram name for a successful profile image download.
+const char ProfileDownloadSuccessTime[] =
+    "UserImage.ProfileDownloadSuccessTime";
+
 // Used to handle the asynchronous response of deleting a cryptohome directory.
 class RemoveAttempt : public CryptohomeLibrary::Delegate {
  public:
   // Creates new remove attempt for the given user. Note, |delegate| can
   // be NULL.
   RemoveAttempt(const std::string& user_email,
                 chromeos::RemoveUserDelegate* delegate)
       : user_email_(user_email),
         delegate_(delegate),
         weak_factory_(this) {
@@ skipping 355 matching lines @@
   if (profile_image_downloader_.get()) {
     // Another download is already in progress
     return;
   }
 
   if (logged_in_user().email().empty()) {
     // This is a guest login so there's no profile image to download.
     return;
   }
 
+  profile_image_load_start_time_ = base::Time::Now();
   profile_image_downloader_.reset(new ProfileDownloader(this));
   profile_image_downloader_->Start();
-  profile_image_load_start_time_ = base::Time::Now();
 }
 
 void UserManager::Observe(int type,
                           const content::NotificationSource& source,
                           const content::NotificationDetails& details) {
   if (type == chrome::NOTIFICATION_OWNER_KEY_FETCH_ATTEMPT_SUCCEEDED) {
     BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE,
                             base::Bind(&UserManager::CheckOwnership,
                                        base::Unretained(this)));
   }
@@ skipping 323 matching lines @@
   return login::kUserImageSize;
 }
 
 Profile* UserManager::GetBrowserProfile() {
   return ProfileManager::GetDefaultProfile();
 }
 
 void UserManager::OnDownloadComplete(ProfileDownloader* downloader,
                                      bool success) {
   ProfileDownloadResult result;
-  if (!success)
+  std::string time_histogram_name;
+  if (!success) {
     result = kDownloadFailure;
-  else if (downloader->GetProfilePicture().isNull())
+    time_histogram_name = kProfileDownloadFailureTime;
+  } else if (downloader->GetProfilePicture().isNull()) {
     result = kDownloadDefault;
-  else
+    time_histogram_name = kProfileDownloadDefaultTime;
+  } else {
     result = kDownloadSuccess;
+    time_histogram_name = ProfileDownloadSuccessTime;
+  }
+
   UMA_HISTOGRAM_ENUMERATION("UserImage.ProfileDownloadResult",
       result, kDownloadResultsCount);
 
+  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(time_histogram_name, delta);
+
   if (result == kDownloadSuccess) {
     // Check if this image is not the same as already downloaded.
     std::string new_image_data_url =
         web_ui_util::GetImageDataUrl(downloader->GetProfilePicture());
     if (!downloaded_profile_image_data_url_.empty() &&
         new_image_data_url == downloaded_profile_image_data_url_)
       return;
 
     downloaded_profile_image_data_url_ = new_image_data_url;
     downloaded_profile_image_ = downloader->GetProfilePicture();
@@ skipping 31 matching lines @@
 
 User* UserManager::CreateUser(const std::string& email) const {
   User* user = new User(email);
   user->set_oauth_token_status(GetUserOAuthStatus(email));
   // Used to determine whether user's display name is unique.
   ++display_name_count_[user->GetDisplayName()];
   return user;
 }
 
 }  // namespace chromeos