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

Unified Diff: chrome/browser/resources/settings/settings_ui/settings_ui.js

Issue 2715933004: MD Settings/Options: add equivalent logs for "time until interactive" (Closed)
Patch Set: Created 3 years, 10 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/resources/options/options.js ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/settings_ui/settings_ui.js
diff --git a/chrome/browser/resources/settings/settings_ui/settings_ui.js b/chrome/browser/resources/settings/settings_ui/settings_ui.js
index 91a143e06f78dd2340faad009165f9e03a40620a..ea70eceb194e7b1c1132a6751677f2a0c0d6a622 100644
--- a/chrome/browser/resources/settings/settings_ui/settings_ui.js
+++ b/chrome/browser/resources/settings/settings_ui/settings_ui.js
@@ -158,6 +158,11 @@ Polymer({
/** @override */
attached: function() {
+ setTimeout(function() {
+ chrome.send(
+ 'metricsHandler:recordTime',
+ ['Settings.TimeUntilInteractive', window.performance.now()]);
dpapad 2017/02/25 01:04:08 How do we differentiate between old VS new timings
Dan Beam 2017/03/04 02:36:05 i really don't care, to be honest. in the case of
dpapad 2017/03/04 02:54:05 SGTM. I did not know we can filter by URL.
+ });
// Preload bold Roboto so it doesn't load and flicker the first time used.
document.fonts.load('bold 12px Roboto');
settings.setGlobalScrollTarget(this.$.headerPanel.scroller);
« no previous file with comments | « chrome/browser/resources/options/options.js ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698