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

Side by Side Diff: android_webview/browser/aw_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 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 ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_ 5 #ifndef ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_
6 #define ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_ 6 #define ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 const base::FilePath guid_file_path); 50 const base::FilePath guid_file_path);
51 void SetMetricsEnabled(bool enabled); 51 void SetMetricsEnabled(bool enabled);
52 52
53 // metrics::EnabledStateProvider: 53 // metrics::EnabledStateProvider:
54 bool IsConsentGiven() override; 54 bool IsConsentGiven() override;
55 55
56 // These implement metrics::MetricsServiceClient. They must not be called 56 // These implement metrics::MetricsServiceClient. They must not be called
57 // until initialization has asynchronously finished. 57 // until initialization has asynchronously finished.
58 metrics::MetricsService* GetMetricsService() override; 58 metrics::MetricsService* GetMetricsService() override;
59 void SetMetricsClientId(const std::string& client_id) override; 59 void SetMetricsClientId(const std::string& client_id) override;
60 void OnRecordingDisabled() override;
61 bool IsOffTheRecordSessionActive() override; 60 bool IsOffTheRecordSessionActive() override;
62 int32_t GetProduct() override; 61 int32_t GetProduct() override;
63 std::string GetApplicationLocale() override; 62 std::string GetApplicationLocale() override;
64 bool GetBrand(std::string* brand_code) override; 63 bool GetBrand(std::string* brand_code) override;
65 metrics::SystemProfileProto::Channel GetChannel() override; 64 metrics::SystemProfileProto::Channel GetChannel() override;
66 std::string GetVersionString() override; 65 std::string GetVersionString() override;
67 void OnLogUploadComplete() override; 66 void OnLogUploadComplete() override;
68 void InitializeSystemProfileMetrics( 67 void InitializeSystemProfileMetrics(
69 const base::Closure& done_callback) override; 68 const base::Closure& done_callback) override;
70 void CollectFinalMetricsForLog(const base::Closure& done_callback) override; 69 void CollectFinalMetricsForLog(const base::Closure& done_callback) override;
(...skipping 13 matching lines...) Expand all
84 net::URLRequestContextGetter* request_context_; 83 net::URLRequestContextGetter* request_context_;
85 std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_; 84 std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_;
86 std::unique_ptr<metrics::MetricsService> metrics_service_; 85 std::unique_ptr<metrics::MetricsService> metrics_service_;
87 86
88 DISALLOW_COPY_AND_ASSIGN(AwMetricsServiceClient); 87 DISALLOW_COPY_AND_ASSIGN(AwMetricsServiceClient);
89 }; 88 };
90 89
91 } // namespace android_webview 90 } // namespace android_webview
92 91
93 #endif // ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_ 92 #endif // ANDROID_WEBVIEW_BROWSER_AW_METRICS_SERVICE_CLIENT_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698