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

Side by Side Diff: chromecast/metrics/cast_metrics_service_client.cc

Issue 602393004: Chromecast: adds stability metrics provider to track crashes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: break common->metrics dependency Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chromecast/metrics/cast_metrics_service_client.h" 5 #include "chromecast/metrics/cast_metrics_service_client.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/i18n/rtl.h" 8 #include "base/i18n/rtl.h"
9 #include "chromecast/common/chromecast_config.h" 9 #include "chromecast/common/chromecast_config.h"
10 #include "chromecast/common/chromecast_switches.h" 10 #include "chromecast/common/chromecast_switches.h"
11 #include "chromecast/metrics/cast_stability_metrics_provider.h"
11 #include "chromecast/metrics/platform_metrics_providers.h" 12 #include "chromecast/metrics/platform_metrics_providers.h"
12 #include "components/metrics/client_info.h" 13 #include "components/metrics/client_info.h"
13 #include "components/metrics/gpu/gpu_metrics_provider.h" 14 #include "components/metrics/gpu/gpu_metrics_provider.h"
14 #include "components/metrics/metrics_provider.h" 15 #include "components/metrics/metrics_provider.h"
15 #include "components/metrics/metrics_service.h" 16 #include "components/metrics/metrics_service.h"
16 #include "components/metrics/metrics_state_manager.h" 17 #include "components/metrics/metrics_state_manager.h"
17 #include "components/metrics/net/net_metrics_log_uploader.h" 18 #include "components/metrics/net/net_metrics_log_uploader.h"
18 #include "components/metrics/net/network_metrics_provider.h" 19 #include "components/metrics/net/network_metrics_provider.h"
19 #include "components/metrics/profiler/profiler_metrics_provider.h" 20 #include "components/metrics/profiler/profiler_metrics_provider.h"
20 21
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 // (indirectly) included in feedback, and can be queried during setup. 132 // (indirectly) included in feedback, and can be queried during setup.
132 // For UMA and crash reporting, associated opt-in settings will control 133 // For UMA and crash reporting, associated opt-in settings will control
133 // sending reports as directed by the user. 134 // sending reports as directed by the user.
134 // For Setup (which also communicates the user's opt-in preferences), 135 // For Setup (which also communicates the user's opt-in preferences),
135 // report the client-id and expect that setup will handle the current opt-in 136 // report the client-id and expect that setup will handle the current opt-in
136 // value. 137 // value.
137 metrics_state_manager_->ForceClientIdCreation(); 138 metrics_state_manager_->ForceClientIdCreation();
138 139
139 metrics_service_->RegisterMetricsProvider( 140 metrics_service_->RegisterMetricsProvider(
140 scoped_ptr< ::metrics::MetricsProvider>( 141 scoped_ptr< ::metrics::MetricsProvider>(
142 new CastStabilityMetricsProvider));
143 metrics_service_->RegisterMetricsProvider(
144 scoped_ptr< ::metrics::MetricsProvider>(
141 new ::metrics::GPUMetricsProvider)); 145 new ::metrics::GPUMetricsProvider));
142 metrics_service_->RegisterMetricsProvider( 146 metrics_service_->RegisterMetricsProvider(
143 scoped_ptr< ::metrics::MetricsProvider>( 147 scoped_ptr< ::metrics::MetricsProvider>(
144 new NetworkMetricsProvider(io_task_runner))); 148 new NetworkMetricsProvider(io_task_runner)));
145 metrics_service_->RegisterMetricsProvider( 149 metrics_service_->RegisterMetricsProvider(
146 scoped_ptr< ::metrics::MetricsProvider>( 150 scoped_ptr< ::metrics::MetricsProvider>(
147 new ::metrics::ProfilerMetricsProvider)); 151 new ::metrics::ProfilerMetricsProvider));
148 RegisterPlatformMetricsProviders(metrics_service_.get()); 152 RegisterPlatformMetricsProviders(metrics_service_.get());
149 153
150 metrics_service_->InitializeMetricsRecordingState(); 154 metrics_service_->InitializeMetricsRecordingState();
151 155
152 if (IsReportingEnabled()) 156 if (IsReportingEnabled())
153 metrics_service_->Start(); 157 metrics_service_->Start();
154 } 158 }
155 159
156 CastMetricsServiceClient::~CastMetricsServiceClient() { 160 CastMetricsServiceClient::~CastMetricsServiceClient() {
157 } 161 }
158 162
159 bool CastMetricsServiceClient::IsReportingEnabled() { 163 bool CastMetricsServiceClient::IsReportingEnabled() {
160 return PlatformIsReportingEnabled(); 164 return PlatformIsReportingEnabled();
161 } 165 }
162 166
163 } // namespace metrics 167 } // namespace metrics
164 } // namespace chromecast 168 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698