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

Unified Diff: chrome/browser/metrics/metrics_services_manager.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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metrics/metrics_services_manager.h
diff --git a/chrome/browser/metrics/metrics_services_manager.h b/chrome/browser/metrics/metrics_services_manager.h
deleted file mode 100644
index d11a3c9706492e1d345d6a5ca1cd848076ae4333..0000000000000000000000000000000000000000
--- a/chrome/browser/metrics/metrics_services_manager.h
+++ /dev/null
@@ -1,100 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_
-#define CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_
-
-#include "base/basictypes.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/threading/thread_checker.h"
-
-class MetricsServicesManagerClient;
-
-namespace base {
-class FilePath;
-}
-
-namespace metrics {
-class MetricsService;
-class MetricsServiceClient;
-class MetricsStateManager;
-}
-
-namespace rappor {
-class RapporService;
-}
-
-namespace variations {
-class VariationsService;
-}
-
-// MetricsServicesManager is a helper class for embedders that use the various
-// metrics-related services in a Chrome-like fashion: MetricsService (via its
-// client), RapporService and VariationsService.
-class MetricsServicesManager {
- public:
- // Creates the MetricsServicesManager with the given client.
- explicit MetricsServicesManager(
- scoped_ptr<MetricsServicesManagerClient> client);
- virtual ~MetricsServicesManager();
-
- // Returns the MetricsService, creating it if it hasn't been created yet (and
- // additionally creating the MetricsServiceClient in that case).
- metrics::MetricsService* GetMetricsService();
-
- // Returns the RapporService, creating it if it hasn't been created yet.
- rappor::RapporService* GetRapporService();
-
- // Returns the VariationsService, creating it if it hasn't been created yet.
- variations::VariationsService* GetVariationsService();
-
- // Should be called when a plugin loading error occurs.
- void OnPluginLoadingError(const base::FilePath& plugin_path);
-
- // Update the managed services when permissions for recording/uploading
- // metrics change.
- void UpdatePermissions(bool may_record, bool may_upload);
-
- // Update the managed services when permissions for uploading metrics change.
- void UpdateUploadPermissions(bool may_upload);
-
- private:
- // Update the managed services when permissions for recording/uploading
- // metrics change.
- void UpdateRapporService();
-
- // Returns the MetricsServiceClient, creating it if it hasn't been
- // created yet (and additionally creating the MetricsService in that case).
- metrics::MetricsServiceClient* GetMetricsServiceClient();
-
- metrics::MetricsStateManager* GetMetricsStateManager();
-
- // Update which services are running to match current permissions.
- void UpdateRunningServices();
-
- // The client passed in from the embedder.
- scoped_ptr<MetricsServicesManagerClient> client_;
-
- // Ensures that all functions are called from the same thread.
- base::ThreadChecker thread_checker_;
-
- // The current metrics reporting setting.
- bool may_upload_;
-
- // The current metrics recording setting.
- bool may_record_;
-
- // The MetricsServiceClient. Owns the MetricsService.
- scoped_ptr<metrics::MetricsServiceClient> metrics_service_client_;
-
- // The RapporService, for RAPPOR metric uploads.
- scoped_ptr<rappor::RapporService> rappor_service_;
-
- // The VariationsService, for server-side experiments infrastructure.
- scoped_ptr<variations::VariationsService> variations_service_;
-
- DISALLOW_COPY_AND_ASSIGN(MetricsServicesManager);
-};
-
-#endif // CHROME_BROWSER_METRICS_METRICS_SERVICES_MANAGER_H_

Powered by Google App Engine
This is Rietveld 408576698