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

Unified Diff: chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc

Issue 8360003: [cros] OOBE: add histograms for Profile image fetching. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 years, 2 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
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
index acc32e7326ac9b5607fd08b4b95bf3c42e319a06..59658a1bb4d6d7406f5751fe9cabcb737d830a95 100644
--- a/chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
@@ -7,6 +7,7 @@
#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"
@@ -83,6 +84,7 @@ void UserImageScreenHandler::Show() {
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).
@@ -144,6 +146,9 @@ void UserImageScreenHandler::RegisterMessages() {
web_ui_->RegisterMessageCallback("onUserImageAccepted",
base::Bind(&UserImageScreenHandler::HandleImageAccepted,
base::Unretained(this)));
+ web_ui_->RegisterMessageCallback("onUserImageScreenShown",
+ base::Bind(&UserImageScreenHandler::HandleScreenShown,
+ base::Unretained(this)));
}
void UserImageScreenHandler::OnPhotoAccepted(const SkBitmap& photo) {
@@ -196,6 +201,15 @@ void UserImageScreenHandler::HandleImageAccepted(const base::ListValue* args) {
}
}
+void UserImageScreenHandler::HandleScreenShown(const base::ListValue* args) {
+ DCHECK(args && args->empty());
+ DCHECK(!screen_show_time_.is_null());
+
+ base::TimeDelta delta = base::Time::Now() - screen_show_time_;
+ VLOG(1) << "Screen load time: " << delta.InSecondsF();
+ UMA_HISTOGRAM_TIMES("UserImage.ScreenIsShownTime", delta);
+}
+
void UserImageScreenHandler::OnCameraPresenceCheckDone() {
base::FundamentalValue present_value(
CameraDetector::camera_presence() == CameraDetector::kCameraPresent);
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698