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

Side by Side Diff: blimp/engine/app/blimp_metrics_service_client.cc

Issue 2605553002: Add EffectiveConnectionType enum to the system profile proto (Closed)
Patch Set: Update proto, Use interface, pass in ctor 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 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 "blimp/engine/app/blimp_metrics_service_client.h" 5 #include "blimp/engine/app/blimp_metrics_service_client.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/i18n/rtl.h" 8 #include "base/i18n/rtl.h"
9 #include "base/lazy_instance.h" 9 #include "base/lazy_instance.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 DCHECK(metrics_state_manager_); 57 DCHECK(metrics_state_manager_);
58 58
59 metrics_service_.reset(new metrics::MetricsService( 59 metrics_service_.reset(new metrics::MetricsService(
60 metrics_state_manager_.get(), this, pref_service)); 60 metrics_state_manager_.get(), this, pref_service));
61 metrics_service_->RegisterMetricsProvider( 61 metrics_service_->RegisterMetricsProvider(
62 base::WrapUnique<metrics::MetricsProvider>( 62 base::WrapUnique<metrics::MetricsProvider>(
63 new BlimpStabilityMetricsProvider(pref_service))); 63 new BlimpStabilityMetricsProvider(pref_service)));
64 metrics_service_->RegisterMetricsProvider( 64 metrics_service_->RegisterMetricsProvider(
65 base::WrapUnique<metrics::MetricsProvider>( 65 base::WrapUnique<metrics::MetricsProvider>(
66 new metrics::NetworkMetricsProvider( 66 new metrics::NetworkMetricsProvider(
67 nullptr /* network_quality_task_runner */,
68 nullptr /* network_quality_estimator_provider */,
67 content::BrowserThread::GetBlockingPool()))); 69 content::BrowserThread::GetBlockingPool())));
68 metrics_service_->RegisterMetricsProvider( 70 metrics_service_->RegisterMetricsProvider(
69 base::WrapUnique<metrics::MetricsProvider>( 71 base::WrapUnique<metrics::MetricsProvider>(
70 new metrics::GPUMetricsProvider)); 72 new metrics::GPUMetricsProvider));
71 metrics_service_->RegisterMetricsProvider( 73 metrics_service_->RegisterMetricsProvider(
72 base::WrapUnique<metrics::MetricsProvider>( 74 base::WrapUnique<metrics::MetricsProvider>(
73 new metrics::ScreenInfoMetricsProvider)); 75 new metrics::ScreenInfoMetricsProvider));
74 metrics_service_->RegisterMetricsProvider( 76 metrics_service_->RegisterMetricsProvider(
75 base::WrapUnique<metrics::MetricsProvider>( 77 base::WrapUnique<metrics::MetricsProvider>(
76 new metrics::ProfilerMetricsProvider())); 78 new metrics::ProfilerMetricsProvider()));
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 BlimpMetricsServiceClient::GetMetricsReportingDefaultState() { 156 BlimpMetricsServiceClient::GetMetricsReportingDefaultState() {
155 return metrics::EnableMetricsDefault::OPT_IN; 157 return metrics::EnableMetricsDefault::OPT_IN;
156 } 158 }
157 159
158 bool BlimpMetricsServiceClient::IsConsentGiven() { 160 bool BlimpMetricsServiceClient::IsConsentGiven() {
159 return true; 161 return true;
160 } 162 }
161 163
162 } // namespace engine 164 } // namespace engine
163 } // namespace blimp 165 } // namespace blimp
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698