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

Side by Side Diff: ios/chrome/browser/metrics/ios_chrome_metrics_services_manager_client.mm

Issue 2615003002: Use ChromeBrowserStateManager instead of BrowserListIOS. (Closed)
Patch Set: Fix ios_chrome_perftests. 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 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 "ios/chrome/browser/metrics/ios_chrome_metrics_services_manager_client. h" 5 #include "ios/chrome/browser/metrics/ios_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/memory/ptr_util.h" 9 #include "base/memory/ptr_util.h"
10 #include "components/metrics/enabled_state_provider.h" 10 #include "components/metrics/enabled_state_provider.h"
11 #include "components/metrics/metrics_state_manager.h" 11 #include "components/metrics/metrics_state_manager.h"
12 #include "components/prefs/pref_service.h" 12 #include "components/prefs/pref_service.h"
13 #include "components/rappor/rappor_service_impl.h" 13 #include "components/rappor/rappor_service_impl.h"
14 #include "components/variations/service/variations_service.h" 14 #include "components/variations/service/variations_service.h"
15 #include "ios/chrome/browser/application_context.h" 15 #include "ios/chrome/browser/application_context.h"
16 #include "ios/chrome/browser/chrome_switches.h" 16 #include "ios/chrome/browser/chrome_switches.h"
17 #include "ios/chrome/browser/metrics/ios_chrome_metrics_service_accessor.h" 17 #include "ios/chrome/browser/metrics/ios_chrome_metrics_service_accessor.h"
18 #include "ios/chrome/browser/metrics/ios_chrome_metrics_service_client.h" 18 #include "ios/chrome/browser/metrics/ios_chrome_metrics_service_client.h"
19 #include "ios/chrome/browser/ui/browser_list_ios.h" 19 #include "ios/chrome/browser/tabs/tab_model_list.h"
20 #include "ios/chrome/browser/variations/ios_chrome_variations_service_client.h" 20 #include "ios/chrome/browser/variations/ios_chrome_variations_service_client.h"
21 #include "ios/chrome/browser/variations/ios_ui_string_overrider_factory.h" 21 #include "ios/chrome/browser/variations/ios_ui_string_overrider_factory.h"
22 #include "ios/public/provider/chrome/browser/chrome_browser_provider.h" 22 #include "ios/public/provider/chrome/browser/chrome_browser_provider.h"
23 23
24 #if !defined(__has_feature) || !__has_feature(objc_arc) 24 #if !defined(__has_feature) || !__has_feature(objc_arc)
25 #error "This file requires ARC support." 25 #error "This file requires ARC support."
26 #endif 26 #endif
27 27
28 namespace { 28 namespace {
29 29
(...skipping 26 matching lines...) Expand all
56 DCHECK(local_state); 56 DCHECK(local_state);
57 } 57 }
58 58
59 IOSChromeMetricsServicesManagerClient:: 59 IOSChromeMetricsServicesManagerClient::
60 ~IOSChromeMetricsServicesManagerClient() = default; 60 ~IOSChromeMetricsServicesManagerClient() = default;
61 61
62 std::unique_ptr<rappor::RapporServiceImpl> 62 std::unique_ptr<rappor::RapporServiceImpl>
63 IOSChromeMetricsServicesManagerClient::CreateRapporServiceImpl() { 63 IOSChromeMetricsServicesManagerClient::CreateRapporServiceImpl() {
64 DCHECK(thread_checker_.CalledOnValidThread()); 64 DCHECK(thread_checker_.CalledOnValidThread());
65 return base::MakeUnique<rappor::RapporServiceImpl>( 65 return base::MakeUnique<rappor::RapporServiceImpl>(
66 local_state_, base::Bind(&BrowserListIOS::IsOffTheRecordSessionActive)); 66 local_state_, base::Bind(&::IsOffTheRecordSessionActive));
67 } 67 }
68 68
69 std::unique_ptr<variations::VariationsService> 69 std::unique_ptr<variations::VariationsService>
70 IOSChromeMetricsServicesManagerClient::CreateVariationsService() { 70 IOSChromeMetricsServicesManagerClient::CreateVariationsService() {
71 DCHECK(thread_checker_.CalledOnValidThread()); 71 DCHECK(thread_checker_.CalledOnValidThread());
72 72
73 // NOTE: On iOS, disabling background networking is not supported, so pass in 73 // NOTE: On iOS, disabling background networking is not supported, so pass in
74 // a dummy value for the name of the switch that disables background 74 // a dummy value for the name of the switch that disables background
75 // networking. 75 // networking.
76 return variations::VariationsService::Create( 76 return variations::VariationsService::Create(
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 IOSChromeMetricsServicesManagerClient::GetMetricsStateManager() { 114 IOSChromeMetricsServicesManagerClient::GetMetricsStateManager() {
115 DCHECK(thread_checker_.CalledOnValidThread()); 115 DCHECK(thread_checker_.CalledOnValidThread());
116 if (!metrics_state_manager_) { 116 if (!metrics_state_manager_) {
117 metrics_state_manager_ = metrics::MetricsStateManager::Create( 117 metrics_state_manager_ = metrics::MetricsStateManager::Create(
118 local_state_, enabled_state_provider_.get(), 118 local_state_, enabled_state_provider_.get(),
119 base::Bind(&PostStoreMetricsClientInfo), 119 base::Bind(&PostStoreMetricsClientInfo),
120 base::Bind(&LoadMetricsClientInfo)); 120 base::Bind(&LoadMetricsClientInfo));
121 } 121 }
122 return metrics_state_manager_.get(); 122 return metrics_state_manager_.get();
123 } 123 }
OLDNEW
« no previous file with comments | « ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm ('k') | ios/chrome/browser/metrics/new_tab_page_uma.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698