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

Unified Diff: blimp/engine/app/blimp_metrics_service_client.h

Issue 2629743003: Remove all blimp engine code (Closed)
Patch Set: Use consistent comment style in //chrome 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « blimp/engine/app/blimp_main.cc ('k') | blimp/engine/app/blimp_metrics_service_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/engine/app/blimp_metrics_service_client.h
diff --git a/blimp/engine/app/blimp_metrics_service_client.h b/blimp/engine/app/blimp_metrics_service_client.h
deleted file mode 100644
index 6c2fa981a0221cafdb837e30c99e8b2a47b8530a..0000000000000000000000000000000000000000
--- a/blimp/engine/app/blimp_metrics_service_client.h
+++ /dev/null
@@ -1,83 +0,0 @@
-// Copyright 2016 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 BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
-#define BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
-
-#include <stdint.h>
-
-#include <string>
-
-#include "base/macros.h"
-#include "base/memory/ref_counted.h"
-#include "components/metrics/enabled_state_provider.h"
-#include "components/metrics/metrics_service_client.h"
-
-class PrefService;
-
-namespace metrics {
-class MetricsService;
-class MetricsStateManager;
-class SystemProfileProto;
-}
-
-namespace net {
-class URLRequestContextGetter;
-} // namespace net
-
-namespace blimp {
-namespace engine {
-
-// BlimpMetricsServiceClient provides an implementation of MetricsServiceClient
-// tailored for the Blimp engine to support the upload of metrics information
-// from the engine.
-// Metrics are always turned on.
-class BlimpMetricsServiceClient : public metrics::MetricsServiceClient,
- public metrics::EnabledStateProvider {
- public:
- // PrefService ownership is retained by the caller.
- // The request_context_getter is a system request context.
- // Both must remain valid for client lifetime.
- BlimpMetricsServiceClient(
- PrefService* pref_service,
- scoped_refptr<net::URLRequestContextGetter> request_context_getter);
- ~BlimpMetricsServiceClient() override;
-
- // metrics::MetricsServiceClient implementation.
- metrics::MetricsService* GetMetricsService() override;
- void SetMetricsClientId(const std::string& client_id) override;
- int32_t GetProduct() override;
- std::string GetApplicationLocale() override;
- bool GetBrand(std::string* brand_code) override;
- metrics::SystemProfileProto::Channel GetChannel() override;
- std::string GetVersionString() override;
- void InitializeSystemProfileMetrics(
- const base::Closure& done_callback) override;
- void CollectFinalMetricsForLog(const base::Closure& done_callback) override;
- std::unique_ptr<metrics::MetricsLogUploader> CreateUploader(
- const std::string& server_url,
- const std::string& mime_type,
- const base::Callback<void(int)>& on_upload_complete) override;
- base::TimeDelta GetStandardUploadInterval() override;
- metrics::EnableMetricsDefault GetMetricsReportingDefaultState() override;
-
- // metrics::EnabledStateProvider implementation.
- // Returns if consent is given for the MetricsService to record metrics
- // information for the client. Always true.
- bool IsConsentGiven() override;
-
- private:
- // Used by NetMetricsLogUploader to create log-upload requests.
- scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
-
- std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_;
- std::unique_ptr<metrics::MetricsService> metrics_service_;
-
- DISALLOW_COPY_AND_ASSIGN(BlimpMetricsServiceClient);
-};
-
-} // namespace engine
-} // namespace blimp
-
-#endif // BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
« no previous file with comments | « blimp/engine/app/blimp_main.cc ('k') | blimp/engine/app/blimp_metrics_service_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698