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

Unified Diff: android_webview/browser/aw_metrics_service_client.cc

Issue 1474483004: WebView Metrics client implementation (Chromium part) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move client back to browser layer, make it a Leaky global, and some cleanup Created 5 years 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: android_webview/browser/aw_metrics_service_client.cc
diff --git a/android_webview/browser/aw_metrics_service_client.cc b/android_webview/browser/aw_metrics_service_client.cc
new file mode 100644
index 0000000000000000000000000000000000000000..90f432cabe03304191cae7f6a164f76da8c43dd9
--- /dev/null
+++ b/android_webview/browser/aw_metrics_service_client.cc
@@ -0,0 +1,193 @@
+// Copyright 2015 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.
+
+#include "android_webview/browser/aw_metrics_service_client.h"
+
+#include "base/bind.h"
+#include "base/i18n/rtl.h"
+#include "base/prefs/pref_service.h"
+#include "components/metrics/metrics_service.h"
+#include "components/metrics/metrics_state_manager.h"
+#include "components/metrics/net/net_metrics_log_uploader.h"
+#include "components/metrics/ui/screen_info_metrics_provider.h"
+#include "components/metrics/url_constants.h"
+
+namespace android_webview {
+
+base::LazyInstance<AwMetricsServiceClient>::Leaky g_lazy_instance_;
+
+// static
+AwMetricsServiceClient* AwMetricsServiceClient::GetInstance() {
+ LOG(INFO) << "metrix AwMetricsServiceClient::GetInstance";
+ return g_lazy_instance_.Pointer();
+}
+
+// static
+void AwMetricsServiceClient::SetMetricsEnabled(bool enabled) {
+ LOG(INFO) << "metrix AwMetricsServiceClient::SetMetricsEnabled " << enabled;
+
+ if (s_metrics_enabled_ != enabled) {
+ // If the client is initialized, apply the setting immediately. Otherwise,
+ // it will be applied on initialization.
+ AwMetricsServiceClient* client = GetInstance();
+ if (client->IsInitialized()) {
+ if (enabled) {
+ client->metrics_service_->Start();
+ } else {
+ client->metrics_service_->Stop();
+ }
+ }
+ }
+ s_metrics_enabled_ = enabled;
+}
+
+void AwMetricsServiceClient::Initialize(
+ PrefService* pref_service,
+ net::URLRequestContextGetter* request_context) {
+ DCHECK(!is_initialized_);
+
+ pref_service_ = pref_service;
+ request_context_ = request_context;
+
+ LOG(INFO) << "metrix Initialize";
+ metrics_state_manager_ = metrics::MetricsStateManager::Create(
+ pref_service_,
+ base::Bind(&AwMetricsServiceClient::IsReportingEnabled,
+ base::Unretained(this)),
+ base::Bind(&AwMetricsServiceClient::StoreClientInfo,
+ base::Unretained(this)),
+ base::Bind(&AwMetricsServiceClient::LoadClientInfo,
+ base::Unretained(this)));
+ metrics_service_.reset(new ::metrics::MetricsService(
+ metrics_state_manager_.get(), this, pref_service_));
+
+ metrics_service_->RegisterMetricsProvider(
+ scoped_ptr<metrics::MetricsProvider>(
+ new metrics::ScreenInfoMetricsProvider));
+ metrics_service_->InitializeMetricsRecordingState();
+
+ is_initialized_ = true;
+
+ if (IsReportingEnabled())
+ metrics_service_->Start();
+}
+
+void AwMetricsServiceClient::Finalize() {
+ DCHECK(is_initialized_);
+ metrics_service_->Stop();
+}
+
+metrics::MetricsService* AwMetricsServiceClient::GetMetricsService() {
+ return metrics_service_.get();
+}
+
+void AwMetricsServiceClient::SetMetricsClientId(
+ const std::string& client_id) {
+ LOG(INFO) << "metrix SetMetricsClientId " << client_id;
+}
+
+void AwMetricsServiceClient::OnRecordingDisabled() {
+ LOG(INFO) << "metrix OnRecordingDisabled";
+}
+
+bool AwMetricsServiceClient::IsOffTheRecordSessionActive() {
+ LOG(INFO) << "metrix IsOffTheRecordSessionActive";
+ return false;
+}
+
+int32_t AwMetricsServiceClient::GetProduct() {
+ LOG(INFO) << "metrix GetProduct";
+ return 42;
sgurun-gerrit only 2015/12/02 00:41:43 ?
paulmiller 2015/12/09 00:50:40 fixed
+}
+
+std::string AwMetricsServiceClient::GetApplicationLocale() {
+ LOG(INFO) << "metrix GetApplicationLocale";
+ return base::i18n::GetConfiguredLocale();
+}
+
+bool AwMetricsServiceClient::GetBrand(std::string* brand_code) {
+ LOG(INFO) << "metrix GetBrand " << brand_code;
+ return false;
+}
+
+metrics::SystemProfileProto::Channel AwMetricsServiceClient::GetChannel() {
+ LOG(INFO) << "metrix GetChannel";
+ return metrics::SystemProfileProto::CHANNEL_UNKNOWN;
+}
+
+std::string AwMetricsServiceClient::GetVersionString() {
+ LOG(INFO) << "metrix GetVersionString";
+ return "version 42";
+}
+
+void AwMetricsServiceClient::OnLogUploadComplete() {
+ LOG(INFO) << "metrix OnLogUploadComplete";
+}
+
+void AwMetricsServiceClient::InitializeSystemProfileMetrics(
+ const base::Closure& done_callback) {
+ LOG(INFO) << "metrix InitializeSystemProfileMetrics";
+ done_callback.Run();
+}
+
+void AwMetricsServiceClient::CollectFinalMetricsForLog(
+ const base::Closure& done_callback) {
+ LOG(INFO) << "metrix CollectFinalMetricsForLog";
+ done_callback.Run();
+}
+
+scoped_ptr<metrics::MetricsLogUploader> AwMetricsServiceClient::CreateUploader(
+ const base::Callback<void(int)>& on_upload_complete) {
+ LOG(INFO) << "metrix CreateUploader";
+ return scoped_ptr<::metrics::MetricsLogUploader>(
+ new metrics::NetMetricsLogUploader(
+ request_context_,
+ /*metrics::kDefaultMetricsServerUrl,*/ "http://localhost:10000",
+ metrics::kDefaultMetricsMimeType,
+ on_upload_complete
+ )
+ );
+}
+
+base::TimeDelta AwMetricsServiceClient::GetStandardUploadInterval() {
+ LOG(INFO) << "metrix GetStandardUploadInterval";
+ return base::TimeDelta::FromSeconds(10);
+}
+
+AwMetricsServiceClient::AwMetricsServiceClient()
+ : is_initialized_(false),
+ pref_service_(nullptr),
+ request_context_(nullptr) {
+ LOG(INFO) << "metrix AwMetricsServiceClient";
+}
+
+AwMetricsServiceClient::~AwMetricsServiceClient() {
+ LOG(INFO) << "metrix ~AwMetricsServiceClient";
+}
+
+bool AwMetricsServiceClient::IsInitialized() {
+ return is_initialized_;
+}
+
+bool AwMetricsServiceClient::IsReportingEnabled() {
+ LOG(INFO) << "metrix IsReportingEnabled " << s_metrics_enabled_;
+ return s_metrics_enabled_;
+}
+
+void AwMetricsServiceClient::StoreClientInfo(
+ const metrics::ClientInfo& client_info){
+ LOG(INFO) << "metrix StoreClientInfo id=" << client_info.client_id;
+}
+
+scoped_ptr<metrics::ClientInfo> AwMetricsServiceClient::LoadClientInfo() {
+ LOG(INFO) << "metrix LoadClientInfo";
+ scoped_ptr<metrics::ClientInfo> client_info(new ::metrics::ClientInfo);
+ client_info->client_id = "00000000-0000-4000-8000-000000000000";
+ return client_info.Pass();
+}
+
+// static
+bool AwMetricsServiceClient::s_metrics_enabled_ = false;
sgurun-gerrit only 2015/12/02 00:41:43 drop
paulmiller 2015/12/09 00:50:40 fixed
+
+} // namespace android_webview

Powered by Google App Engine
This is Rietveld 408576698