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

Side by Side Diff: chrome/browser/media/router/media_router_metrics.h

Issue 1510603003: [Media Router] Add UMA metrics for Media Router Dialog WebUI performance. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes per mfoltz@'s comments. Removed logging. Created 5 years 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 unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/media/router/media_router_metrics.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_ 5 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_
6 #define CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_ 6 #define CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_
7 7
8 #include "base/time/time.h"
9
8 namespace media_router { 10 namespace media_router {
9 11
10 // NOTE: Do not renumber enums as that would confuse interpretation of 12 // NOTE: Do not renumber enums as that would confuse interpretation of
11 // previously logged data. When making changes, also update the enum list 13 // previously logged data. When making changes, also update the enum list
12 // in tools/metrics/histograms/histograms.xml to keep it in sync. 14 // in tools/metrics/histograms/histograms.xml to keep it in sync.
13 15
14 // Where the user clicked to open the Media Router dialog. 16 // Where the user clicked to open the Media Router dialog.
15 enum class MediaRouterDialogOpenOrigin { 17 enum class MediaRouterDialogOpenOrigin {
16 TOOLBAR = 0, 18 TOOLBAR = 0,
17 OVERFLOW_MENU = 1, 19 OVERFLOW_MENU = 1,
(...skipping 27 matching lines...) Expand all
45 47
46 class MediaRouterMetrics { 48 class MediaRouterMetrics {
47 public: 49 public:
48 // Records where the user clicked to open the Media Router dialog. 50 // Records where the user clicked to open the Media Router dialog.
49 static void RecordMediaRouterDialogOrigin( 51 static void RecordMediaRouterDialogOrigin(
50 MediaRouterDialogOpenOrigin origin); 52 MediaRouterDialogOpenOrigin origin);
51 53
52 // Records why the media route provider extension was woken up. 54 // Records why the media route provider extension was woken up.
53 static void RecordMediaRouteProviderWakeReason( 55 static void RecordMediaRouteProviderWakeReason(
54 MediaRouteProviderWakeReason reason); 56 MediaRouteProviderWakeReason reason);
57
58 // Records the duration it takes for the Media Router dialog to open and
59 // finish painting after a user clicks to open the dialog.
60 static void RecordMediaRouterDialogPaint(
61 const base::TimeDelta& delta);
Ilya Sherman 2015/12/10 04:48:55 nit: I think that TimeDelta is typically passed by
apacible 2015/12/10 17:22:18 Done.
62
63 // Records the duration it takes for the Media Router dialog to load its
64 // initial data after a user clicks to open the dialog.
65 static void RecordMediaRouterDialogLoaded(
66 const base::TimeDelta& delta);
55 }; 67 };
56 68
57 } // namespace media_router 69 } // namespace media_router
58 70
59 #endif // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_ 71 #endif // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_METRICS_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/media/router/media_router_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698