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

Unified Diff: third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js

Issue 2617553002: DevTools: Split Timeline started UMA metric into two parts. (Closed)
Patch Set: update the test. Created 3 years, 11 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: third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js b/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
index 81484a65bd0b9c8b9338af1f57e7905cdc8598b3..72855212f1d71a9ac7b5358f4c9242149afd3b89 100644
--- a/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
@@ -553,7 +553,7 @@ Timeline.TimelinePanel = class extends UI.Panel {
for (var i = 0; i < this._overviewControls.length; ++i)
this._overviewControls[i].timelineStarted();
- Host.userMetrics.actionTaken(Host.UserMetrics.Action.TimelineStarted);
+ Host.userMetrics.actionTaken(userInitiated ? Host.UserMetrics.Action.TimelineStarted : Host.UserMetrics.Action.TimelinePageReloadStarted);
this._setUIControlsEnabled(false);
this._hideLandingPage();
}
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/host/UserMetrics.js ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698