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

Side by Side Diff: blimp/engine/app/blimp_metrics_service_client.h

Issue 2248793002: Remove OnRecordingDisabled() metrics client interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add helper function Created 4 years, 4 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_ 5 #ifndef BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
6 #define BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_ 6 #define BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 29 matching lines...) Expand all
40 // The request_context_getter is a system request context. 40 // The request_context_getter is a system request context.
41 // Both must remain valid for client lifetime. 41 // Both must remain valid for client lifetime.
42 BlimpMetricsServiceClient( 42 BlimpMetricsServiceClient(
43 PrefService* pref_service, 43 PrefService* pref_service,
44 scoped_refptr<net::URLRequestContextGetter> request_context_getter); 44 scoped_refptr<net::URLRequestContextGetter> request_context_getter);
45 ~BlimpMetricsServiceClient() override; 45 ~BlimpMetricsServiceClient() override;
46 46
47 // metrics::MetricsServiceClient implementation. 47 // metrics::MetricsServiceClient implementation.
48 metrics::MetricsService* GetMetricsService() override; 48 metrics::MetricsService* GetMetricsService() override;
49 void SetMetricsClientId(const std::string& client_id) override; 49 void SetMetricsClientId(const std::string& client_id) override;
50 void OnRecordingDisabled() override;
51 bool IsOffTheRecordSessionActive() override; 50 bool IsOffTheRecordSessionActive() override;
52 int32_t GetProduct() override; 51 int32_t GetProduct() override;
53 std::string GetApplicationLocale() override; 52 std::string GetApplicationLocale() override;
54 bool GetBrand(std::string* brand_code) override; 53 bool GetBrand(std::string* brand_code) override;
55 metrics::SystemProfileProto::Channel GetChannel() override; 54 metrics::SystemProfileProto::Channel GetChannel() override;
56 std::string GetVersionString() override; 55 std::string GetVersionString() override;
57 void OnLogUploadComplete() override; 56 void OnLogUploadComplete() override;
58 void InitializeSystemProfileMetrics( 57 void InitializeSystemProfileMetrics(
59 const base::Closure& done_callback) override; 58 const base::Closure& done_callback) override;
60 void CollectFinalMetricsForLog(const base::Closure& done_callback) override; 59 void CollectFinalMetricsForLog(const base::Closure& done_callback) override;
(...skipping 14 matching lines...) Expand all
75 std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_; 74 std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_;
76 std::unique_ptr<metrics::MetricsService> metrics_service_; 75 std::unique_ptr<metrics::MetricsService> metrics_service_;
77 76
78 DISALLOW_COPY_AND_ASSIGN(BlimpMetricsServiceClient); 77 DISALLOW_COPY_AND_ASSIGN(BlimpMetricsServiceClient);
79 }; 78 };
80 79
81 } // namespace engine 80 } // namespace engine
82 } // namespace blimp 81 } // namespace blimp
83 82
84 #endif // BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_ 83 #endif // BLIMP_ENGINE_APP_BLIMP_METRICS_SERVICE_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698