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

Side by Side Diff: chrome/browser/media/router/mojo/media_router_mojo_metrics.cc

Issue 2894623002: [Media Router] Add UMA metrics for media route controller creation (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #include "chrome/browser/media/router/mojo/media_router_mojo_metrics.h" 5 #include "chrome/browser/media/router/mojo/media_router_mojo_metrics.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/metrics/histogram_macros.h" 8 #include "base/metrics/histogram_macros.h"
9 #include "base/version.h" 9 #include "base/version.h"
10 #include "components/version_info/version_info.h" 10 #include "components/version_info/version_info.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 // static 70 // static
71 void MediaRouterMojoMetrics::RecordMediaRouteProviderTerminateRoute( 71 void MediaRouterMojoMetrics::RecordMediaRouteProviderTerminateRoute(
72 RouteRequestResult::ResultCode result_code) { 72 RouteRequestResult::ResultCode result_code) {
73 DCHECK_LT(result_code, RouteRequestResult::ResultCode::TOTAL_COUNT); 73 DCHECK_LT(result_code, RouteRequestResult::ResultCode::TOTAL_COUNT);
74 UMA_HISTOGRAM_ENUMERATION( 74 UMA_HISTOGRAM_ENUMERATION(
75 "MediaRouter.Provider.TerminateRoute.Result", result_code, 75 "MediaRouter.Provider.TerminateRoute.Result", result_code,
76 RouteRequestResult::ResultCode::TOTAL_COUNT); 76 RouteRequestResult::ResultCode::TOTAL_COUNT);
77 } 77 }
78 78
79 // static 79 // static
80 void MediaRouterMojoMetrics::RecordMediaRouteControllerCreationResult(
81 bool success) {
82 UMA_HISTOGRAM_BOOLEAN("MediaRouter.Provider.RouteControllerCreationOutcome",
83 success);
84 }
85
86 // static
80 MediaRouteProviderVersion MediaRouterMojoMetrics::GetMediaRouteProviderVersion( 87 MediaRouteProviderVersion MediaRouterMojoMetrics::GetMediaRouteProviderVersion(
81 const base::Version& extension_version, 88 const base::Version& extension_version,
82 const base::Version& browser_version) { 89 const base::Version& browser_version) {
83 if (!extension_version.IsValid() || extension_version.components().empty() || 90 if (!extension_version.IsValid() || extension_version.components().empty() ||
84 !browser_version.IsValid() || browser_version.components().empty()) { 91 !browser_version.IsValid() || browser_version.components().empty()) {
85 return MediaRouteProviderVersion::UNKNOWN; 92 return MediaRouteProviderVersion::UNKNOWN;
86 } 93 }
87 94
88 uint32_t extension_major = extension_version.components()[0]; 95 uint32_t extension_major = extension_version.components()[0];
89 uint32_t browser_major = browser_version.components()[0]; 96 uint32_t browser_major = browser_version.components()[0];
90 // Sanity check. 97 // Sanity check.
91 if (extension_major == 0 || browser_major == 0) { 98 if (extension_major == 0 || browser_major == 0) {
92 return MediaRouteProviderVersion::UNKNOWN; 99 return MediaRouteProviderVersion::UNKNOWN;
93 } else if (extension_major >= browser_major) { 100 } else if (extension_major >= browser_major) {
94 return MediaRouteProviderVersion::SAME_VERSION_AS_CHROME; 101 return MediaRouteProviderVersion::SAME_VERSION_AS_CHROME;
95 } else if (browser_major - extension_major == 1) { 102 } else if (browser_major - extension_major == 1) {
96 return MediaRouteProviderVersion::ONE_VERSION_BEHIND_CHROME; 103 return MediaRouteProviderVersion::ONE_VERSION_BEHIND_CHROME;
97 } else { 104 } else {
98 return MediaRouteProviderVersion::MULTIPLE_VERSIONS_BEHIND_CHROME; 105 return MediaRouteProviderVersion::MULTIPLE_VERSIONS_BEHIND_CHROME;
99 } 106 }
100 } 107 }
101 108
102 } // namespace media_router 109 } // namespace media_router
OLDNEW
« no previous file with comments | « chrome/browser/media/router/mojo/media_router_mojo_metrics.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698