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

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: DEPS changes not caught by git cl presubmit 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 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 // (indirectly) included in feedback, and can be queried during setup. 147 // (indirectly) included in feedback, and can be queried during setup.
147 // For UMA and crash reporting, associated opt-in settings will control 148 // For UMA and crash reporting, associated opt-in settings will control
148 // sending reports as directed by the user. 149 // sending reports as directed by the user.
149 // For Setup (which also communicates the user's opt-in preferences), 150 // For Setup (which also communicates the user's opt-in preferences),
150 // report the client-id and expect that setup will handle the current opt-in 151 // report the client-id and expect that setup will handle the current opt-in
151 // value. 152 // value.
152 metrics_state_manager_->ForceClientIdCreation(); 153 metrics_state_manager_->ForceClientIdCreation();
153 154
154 metrics_service_->RegisterMetricsProvider( 155 metrics_service_->RegisterMetricsProvider(
155 scoped_ptr< ::metrics::MetricsProvider>( 156 scoped_ptr< ::metrics::MetricsProvider>(
157 new CastStabilityMetricsProvider));
158 metrics_service_->RegisterMetricsProvider(
159 scoped_ptr< ::metrics::MetricsProvider>(
156 new ::metrics::GPUMetricsProvider)); 160 new ::metrics::GPUMetricsProvider));
157 metrics_service_->RegisterMetricsProvider( 161 metrics_service_->RegisterMetricsProvider(
158 scoped_ptr< ::metrics::MetricsProvider>( 162 scoped_ptr< ::metrics::MetricsProvider>(
159 new NetworkMetricsProvider(io_task_runner))); 163 new NetworkMetricsProvider(io_task_runner)));
160 metrics_service_->RegisterMetricsProvider( 164 metrics_service_->RegisterMetricsProvider(
161 scoped_ptr< ::metrics::MetricsProvider>( 165 scoped_ptr< ::metrics::MetricsProvider>(
162 new ::metrics::ProfilerMetricsProvider)); 166 new ::metrics::ProfilerMetricsProvider));
163 RegisterPlatformMetricsProviders(metrics_service_.get()); 167 RegisterPlatformMetricsProviders(metrics_service_.get());
164 168
165 metrics_service_->InitializeMetricsRecordingState(); 169 metrics_service_->InitializeMetricsRecordingState();
166 170
167 if (IsReportingEnabled()) 171 if (IsReportingEnabled())
168 metrics_service_->Start(); 172 metrics_service_->Start();
169 } 173 }
170 174
171 CastMetricsServiceClient::~CastMetricsServiceClient() { 175 CastMetricsServiceClient::~CastMetricsServiceClient() {
172 } 176 }
173 177
174 bool CastMetricsServiceClient::IsReportingEnabled() { 178 bool CastMetricsServiceClient::IsReportingEnabled() {
175 return PlatformIsReportingEnabled(); 179 return PlatformIsReportingEnabled();
176 } 180 }
177 181
178 } // namespace metrics 182 } // namespace metrics
179 } // namespace chromecast 183 } // namespace chromecast
OLDNEW
« no previous file with comments | « chromecast/metrics/cast_metrics_prefs.cc ('k') | chromecast/metrics/cast_metrics_service_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698