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

Unified Diff: chrome/browser/resources/chromeos/login/oobe_screen_user_image.js

Issue 8360003: [cros] OOBE: add histograms for Profile image fetching. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Renamed HandleScreenShown. 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
Index: chrome/browser/resources/chromeos/login/oobe_screen_user_image.js
diff --git a/chrome/browser/resources/chromeos/login/oobe_screen_user_image.js b/chrome/browser/resources/chromeos/login/oobe_screen_user_image.js
index b3b39e0047e2a01eeaa9916bb25884b2a4317373..479b9aa06c656f3043e82b4c1675f7f11ce2d173 100644
--- a/chrome/browser/resources/chromeos/login/oobe_screen_user_image.js
+++ b/chrome/browser/resources/chromeos/login/oobe_screen_user_image.js
@@ -211,6 +211,7 @@ cr.define('oobe', function() {
onBeforeShow: function(data) {
Oobe.getInstance().headerHidden = true;
$('user-image-grid').updateAndFocus();
+ chrome.send('onUserImageScreenShown', [(new Date()).getTime()/1000]);
zel 2011/10/20 21:57:44 you should not use time from JS side here, you sho
Ivan Korotkov 2011/10/21 07:52:00 Why? I'm interested in the delay between Show() is
Ivan Korotkov 2011/10/21 12:29:52 Hm, ok, as discussed offline it's better to use a
},
/**

Powered by Google App Engine
This is Rietveld 408576698