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

Side by Side Diff: chrome/browser/metrics/chrome_metrics_services_manager_client.cc

Issue 1411863002: Use kMetricsReportingEnabled instead of kStatsReporingPref on metrics side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix callback Created 5 years, 1 month 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 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 #include "chrome/browser/metrics/chrome_metrics_services_manager_client.h" 5 #include "chrome/browser/metrics/chrome_metrics_services_manager_client.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "chrome/browser/browser_process.h" 10 #include "chrome/browser/browser_process.h"
(...skipping 18 matching lines...) Expand all
29 FROM_HERE, 29 FROM_HERE,
30 base::Bind(&GoogleUpdateSettings::StoreMetricsClientInfo, client_info)); 30 base::Bind(&GoogleUpdateSettings::StoreMetricsClientInfo, client_info));
31 } 31 }
32 32
33 } // namespace 33 } // namespace
34 34
35 ChromeMetricsServicesManagerClient::ChromeMetricsServicesManagerClient( 35 ChromeMetricsServicesManagerClient::ChromeMetricsServicesManagerClient(
36 PrefService* local_state) 36 PrefService* local_state)
37 : local_state_(local_state) { 37 : local_state_(local_state) {
38 DCHECK(local_state); 38 DCHECK(local_state);
39
40 SetupMetricsStateForChromeOS();
39 } 41 }
40 42
41 ChromeMetricsServicesManagerClient::~ChromeMetricsServicesManagerClient() {} 43 ChromeMetricsServicesManagerClient::~ChromeMetricsServicesManagerClient() {}
42 44
43 scoped_ptr<rappor::RapporService> 45 scoped_ptr<rappor::RapporService>
44 ChromeMetricsServicesManagerClient::CreateRapporService() { 46 ChromeMetricsServicesManagerClient::CreateRapporService() {
45 DCHECK(thread_checker_.CalledOnValidThread()); 47 DCHECK(thread_checker_.CalledOnValidThread());
46 return make_scoped_ptr(new rappor::RapporService( 48 return make_scoped_ptr(new rappor::RapporService(
47 local_state_, base::Bind(&chrome::IsOffTheRecordSessionActive))); 49 local_state_, base::Bind(&chrome::IsOffTheRecordSessionActive)));
48 } 50 }
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 if (!metrics_state_manager_) { 104 if (!metrics_state_manager_) {
103 metrics_state_manager_ = metrics::MetricsStateManager::Create( 105 metrics_state_manager_ = metrics::MetricsStateManager::Create(
104 local_state_, 106 local_state_,
105 base::Bind( 107 base::Bind(
106 &ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled), 108 &ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled),
107 base::Bind(&PostStoreMetricsClientInfo), 109 base::Bind(&PostStoreMetricsClientInfo),
108 base::Bind(&GoogleUpdateSettings::LoadMetricsClientInfo)); 110 base::Bind(&GoogleUpdateSettings::LoadMetricsClientInfo));
109 } 111 }
110 return metrics_state_manager_.get(); 112 return metrics_state_manager_.get();
111 } 113 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698