[cros] OOBE: add histograms for Profile image fetching.

chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.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/ui/webui/chromeos/login/user_image_screen_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
+#include "base/metrics/histogram.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/login/camera_detector.h"
 #include "chrome/browser/chromeos/login/default_user_images.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/webui_login_display.h"
 #include "chrome/browser/chromeos/options/take_photo_dialog.h"
 #include "chrome/browser/ui/views/window.h"
 #include "chrome/browser/ui/webui/web_ui_util.h"
 #include "chrome/common/url_constants.h"
 #include "grit/generated_resources.h"
@@ skipping 56 matching lines @@
 void UserImageScreenHandler::SetDelegate(
     UserImageScreenActor::Delegate* screen) {
   screen_ = screen;
 }
 
 void UserImageScreenHandler::Show() {
   if (!page_is_ready()) {
     show_on_init_ = true;
     return;
   }
+  screen_show_time_ = base::Time::Now();
   ShowScreen(kUserImageScreen, NULL);
   // When shown, query camera presence again (first-time query is done by
   // OobeUI::OnLoginPromptVisible).
   CheckCameraPresence();
 }
 
 void UserImageScreenHandler::Hide() {
 }
 
 void UserImageScreenHandler::PrepareToShow() {
@@ skipping 41 matching lines @@
 void UserImageScreenHandler::RegisterMessages() {
   web_ui_->RegisterMessageCallback("takePhoto",
       base::Bind(&UserImageScreenHandler::HandleTakePhoto,
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("selectImage",
       base::Bind(&UserImageScreenHandler::HandleSelectImage,
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("onUserImageAccepted",
       base::Bind(&UserImageScreenHandler::HandleImageAccepted,
                  base::Unretained(this)));
+  web_ui_->RegisterMessageCallback("onUserImageScreenLoaded",
+      base::Bind(&UserImageScreenHandler::HandleScreenLoaded,
+                 base::Unretained(this)));
 }
 
 void UserImageScreenHandler::OnPhotoAccepted(const SkBitmap& photo) {
   user_photo_ = photo;
   selected_image_ = UserManager::User::kExternalImageIndex;
   base::StringValue data_url(web_ui_util::GetImageDataUrl(user_photo_));
   web_ui_->CallJavascriptFunction("oobe.UserImageScreen.setUserPhoto",
                                   data_url);
 }
 
@@ skipping 32 matching lines @@
   if (selected_image_ == UserManager::User::kExternalImageIndex) {
     screen_->OnPhotoTaken(user_photo_);
   } else if (selected_image_ == UserManager::User::kProfileImageIndex) {
     screen_->OnProfileImageSelected(profile_picture_);
   } else {
     DCHECK(selected_image_ >= 0);
     screen_->OnDefaultImageSelected(selected_image_);
   }
 }
 
+void UserImageScreenHandler::HandleScreenLoaded(const base::ListValue* args) {
+  double loaded_time;
+  if (!args ||
+      args->GetSize() != 1 ||
+      !args->GetDouble(0, &loaded_time)) {
+    NOTREACHED();
+    return;
+  }
+
+  DCHECK(!screen_show_time_.is_null());
+  base::TimeDelta delta =
+      base::Time::FromDoubleT(loaded_time) - screen_show_time_;
+  LOG(ERROR) << "Screen load time: " << delta.InSecondsF();
+  UMA_HISTOGRAM_TIMES("UserImage.LoadTime", delta);

[whywhat, 2011/10/20 16:14:50]

I'd call it ScreenIsShownTime, since LoadTime may refer to anything :)

[Ivan Korotkov, 2011/10/20 17:13:16]

Done.

+}
+
 void UserImageScreenHandler::OnCameraPresenceCheckDone() {
   base::FundamentalValue present_value(
       CameraDetector::camera_presence() == CameraDetector::kCameraPresent);
   web_ui_->CallJavascriptFunction("oobe.UserImageScreen.setCameraPresent",
                                   present_value);
 }
 
 }  // namespace chromeos