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

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

Issue 293393010: Move initial metrics gathering tasks out of MetricsService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes Created 6 years, 7 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 #ifndef CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_ 5 #ifndef CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_
6 #define CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_ 6 #define CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_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 11
12 class ChromeMetricsServiceClient; 12 class ChromeMetricsServiceClient;
13 class MetricsService; 13 class MetricsService;
14 class PluginMetricsProvider;
14 class PrefService; 15 class PrefService;
15 16
16 namespace metrics { 17 namespace metrics {
17 class MetricsStateManager; 18 class MetricsStateManager;
18 } 19 }
19 20
20 namespace rappor { 21 namespace rappor {
21 class RapporService; 22 class RapporService;
22 } 23 }
23 24
24 namespace chrome_variations { 25 namespace chrome_variations {
25 class VariationsService; 26 class VariationsService;
26 } 27 }
27 28
28 // MetricsServicesManager is a helper class that has ownership over the various 29 // MetricsServicesManager is a helper class that has ownership over the various
29 // metrics-related services in Chrome: MetricsService (via its client), 30 // metrics-related services in Chrome: MetricsService (via its client),
30 // RapporService and VariationsService. 31 // RapporService and VariationsService.
31 class MetricsServicesManager { 32 class MetricsServicesManager {
32 public: 33 public:
33 // Creates the MetricsServicesManager with the |local_state| prefs service. 34 // Creates the MetricsServicesManager with the |local_state| prefs service.
34 explicit MetricsServicesManager(PrefService* local_state); 35 explicit MetricsServicesManager(PrefService* local_state);
35 virtual ~MetricsServicesManager(); 36 virtual ~MetricsServicesManager();
36 37
37 // Returns the MetricsService, creating it if it hasn't been created yet. 38 // Returns the MetricsService, creating it if it hasn't been created yet.
38 MetricsService* GetMetricsService(); 39 MetricsService* GetMetricsService();
39 40
41 // Returns the PluginMetricsProvider or NULL if plugins aren't enabled.
42 PluginMetricsProvider* GetPluginMetricsProvider();
43
40 // Returns the GetRapporService, creating it if it hasn't been created yet. 44 // Returns the GetRapporService, creating it if it hasn't been created yet.
41 rappor::RapporService* GetRapporService(); 45 rappor::RapporService* GetRapporService();
42 46
43 // Returns the VariationsService, creating it if it hasn't been created yet. 47 // Returns the VariationsService, creating it if it hasn't been created yet.
44 chrome_variations::VariationsService* GetVariationsService(); 48 chrome_variations::VariationsService* GetVariationsService();
45 49
46 private: 50 private:
47 metrics::MetricsStateManager* GetMetricsStateManager(); 51 metrics::MetricsStateManager* GetMetricsStateManager();
48 52
53 ChromeMetricsServiceClient* GetMetricsServiceClient();
54
49 // Returns true iff metrics reporting is enabled. 55 // Returns true iff metrics reporting is enabled.
50 bool IsMetricsReportingEnabled() const; 56 bool IsMetricsReportingEnabled() const;
51 57
52 // Ensures that all functions are called from the same thread. 58 // Ensures that all functions are called from the same thread.
53 base::ThreadChecker thread_checker_; 59 base::ThreadChecker thread_checker_;
54 60
55 // Weak pointer to the local state prefs store. 61 // Weak pointer to the local state prefs store.
56 PrefService* local_state_; 62 PrefService* local_state_;
57 63
58 // MetricsStateManager which is passed as a parameter to service constructors. 64 // MetricsStateManager which is passed as a parameter to service constructors.
59 scoped_ptr<metrics::MetricsStateManager> metrics_state_manager_; 65 scoped_ptr<metrics::MetricsStateManager> metrics_state_manager_;
60 66
61 // Chrome embedder implementation of the MetricsServiceClient. Owns the 67 // Chrome embedder implementation of the MetricsServiceClient. Owns the
62 // MetricsService. 68 // MetricsService.
63 scoped_ptr<ChromeMetricsServiceClient> metrics_service_client_; 69 scoped_ptr<ChromeMetricsServiceClient> metrics_service_client_;
64 70
65 // The RapporService, for RAPPOR metric uploads. 71 // The RapporService, for RAPPOR metric uploads.
66 scoped_ptr<rappor::RapporService> rappor_service_; 72 scoped_ptr<rappor::RapporService> rappor_service_;
67 73
68 // The VariationsService, for server-side experiments infrastructure. 74 // The VariationsService, for server-side experiments infrastructure.
69 scoped_ptr<chrome_variations::VariationsService> variations_service_; 75 scoped_ptr<chrome_variations::VariationsService> variations_service_;
70 76
71 DISALLOW_COPY_AND_ASSIGN(MetricsServicesManager); 77 DISALLOW_COPY_AND_ASSIGN(MetricsServicesManager);
72 }; 78 };
73 79
74 #endif // CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_ 80 #endif // CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698