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

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

Issue 1412113002: Componentize MetricsServicesManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@refactor_metrics_services_manager
Patch Set: Rebase Created 5 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 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 #ifndef CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_ 5 #ifndef CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_
6 #define CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_ 6 #define CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/threading/thread_checker.h" 10 #include "base/threading/thread_checker.h"
11 #include "chrome/browser/metrics/metrics_services_manager_client.h"
12 #include "chrome/browser/safe_browsing/safe_browsing_service.h" 11 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
12 #include "components/metrics_services_manager/metrics_services_manager_client.h"
13 13
14 class PrefService; 14 class PrefService;
15 15
16 namespace metrics { 16 namespace metrics {
17 class MetricsStateManager; 17 class MetricsStateManager;
18 } 18 }
19 19
20 // Provides a //chrome-specific implementation of MetricsServicesManagerClient. 20 // Provides a //chrome-specific implementation of MetricsServicesManagerClient.
21 class ChromeMetricsServicesManagerClient : public MetricsServicesManagerClient { 21 class ChromeMetricsServicesManagerClient
22 : public metrics_services_manager::MetricsServicesManagerClient {
22 public: 23 public:
23 explicit ChromeMetricsServicesManagerClient(PrefService* local_state); 24 explicit ChromeMetricsServicesManagerClient(PrefService* local_state);
24 ~ChromeMetricsServicesManagerClient() override; 25 ~ChromeMetricsServicesManagerClient() override;
25 26
26 private: 27 private:
27 // MetricsServicesManagerClient: 28 // metrics_services_manager::MetricsServicesManagerClient:
28 scoped_ptr<rappor::RapporService> CreateRapporService() override; 29 scoped_ptr<rappor::RapporService> CreateRapporService() override;
29 scoped_ptr<variations::VariationsService> CreateVariationsService() override; 30 scoped_ptr<variations::VariationsService> CreateVariationsService() override;
30 scoped_ptr<metrics::MetricsServiceClient> CreateMetricsServiceClient() 31 scoped_ptr<metrics::MetricsServiceClient> CreateMetricsServiceClient()
31 override; 32 override;
32 net::URLRequestContextGetter* GetURLRequestContext() override; 33 net::URLRequestContextGetter* GetURLRequestContext() override;
33 bool IsSafeBrowsingEnabled(const base::Closure& on_update_callback) override; 34 bool IsSafeBrowsingEnabled(const base::Closure& on_update_callback) override;
34 bool IsMetricsReportingEnabled() override; 35 bool IsMetricsReportingEnabled() override;
35 bool OnlyDoMetricsRecording() override; 36 bool OnlyDoMetricsRecording() override;
36 37
37 // Gets the MetricsStateManager, creating it if it has not already been 38 // Gets the MetricsStateManager, creating it if it has not already been
38 // created. 39 // created.
39 metrics::MetricsStateManager* GetMetricsStateManager(); 40 metrics::MetricsStateManager* GetMetricsStateManager();
40 41
41 // MetricsStateManager which is passed as a parameter to service constructors. 42 // MetricsStateManager which is passed as a parameter to service constructors.
42 scoped_ptr<metrics::MetricsStateManager> metrics_state_manager_; 43 scoped_ptr<metrics::MetricsStateManager> metrics_state_manager_;
43 44
44 // Ensures that all functions are called from the same thread. 45 // Ensures that all functions are called from the same thread.
45 base::ThreadChecker thread_checker_; 46 base::ThreadChecker thread_checker_;
46 47
47 // Weak pointer to the local state prefs store. 48 // Weak pointer to the local state prefs store.
48 PrefService* local_state_; 49 PrefService* local_state_;
49 50
50 // Subscription to SafeBrowsing service state changes. 51 // Subscription to SafeBrowsing service state changes.
51 scoped_ptr<SafeBrowsingService::StateSubscription> sb_state_subscription_; 52 scoped_ptr<SafeBrowsingService::StateSubscription> sb_state_subscription_;
52 53
53 DISALLOW_COPY_AND_ASSIGN(ChromeMetricsServicesManagerClient); 54 DISALLOW_COPY_AND_ASSIGN(ChromeMetricsServicesManagerClient);
54 }; 55 };
55 56
56 #endif // CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_ 57 #endif // CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698