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

Unified Diff: android_webview/browser/aw_metrics_service_client.cc

Issue 2611883002: Prepare to call GMS APIs from WebView (Closed)
Patch Set: explicit destructor for style checker 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 | « android_webview/browser/aw_metrics_service_client.h ('k') | android_webview/glue/glue.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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
index facd0fe4d23ad7b68f0bef109b2b6c6feb1d2139..c2bdad033e2c1583780c6433d29fb7f7117c3fa4 100644
--- a/android_webview/browser/aw_metrics_service_client.cc
+++ b/android_webview/browser/aw_metrics_service_client.cc
@@ -4,218 +4,9 @@
#include "android_webview/browser/aw_metrics_service_client.h"
-#include "android_webview/common/aw_version_info_values.h"
-#include "base/bind.h"
-#include "base/files/file_util.h"
-#include "base/guid.h"
-#include "base/i18n/rtl.h"
-#include "components/metrics/call_stack_profile_metrics_provider.h"
-#include "components/metrics/enabled_state_provider.h"
-#include "components/metrics/gpu/gpu_metrics_provider.h"
-#include "components/metrics/metrics_pref_names.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/profiler/profiler_metrics_provider.h"
-#include "components/metrics/ui/screen_info_metrics_provider.h"
-#include "components/metrics/url_constants.h"
-#include "components/prefs/pref_service.h"
-#include "content/public/browser/browser_thread.h"
-
namespace android_webview {
-base::LazyInstance<AwMetricsServiceClient>::Leaky g_lazy_instance_;
-
-namespace {
-
-const int kUploadIntervalMinutes = 30;
-
-// Callbacks for metrics::MetricsStateManager::Create. Store/LoadClientInfo
-// allow Windows Chrome to back up ClientInfo. They're no-ops for WebView.
-
-void StoreClientInfo(const metrics::ClientInfo& client_info) {}
-
-std::unique_ptr<metrics::ClientInfo> LoadClientInfo() {
- std::unique_ptr<metrics::ClientInfo> client_info;
- return client_info;
-}
-
-// A GUID in text form is composed of 32 hex digits and 4 hyphens.
-const size_t GUID_SIZE = 32 + 4;
-
-void GetOrCreateGUID(const base::FilePath guid_file_path, std::string* guid) {
- DCHECK_CURRENTLY_ON(content::BrowserThread::FILE);
-
- // Try to read an existing GUID.
- if (base::ReadFileToStringWithMaxSize(guid_file_path, guid, GUID_SIZE)) {
- if (base::IsValidGUID(*guid))
- return;
- else
- LOG(ERROR) << "Found invalid GUID";
- }
-
- // We must write a new GUID.
- *guid = base::GenerateGUID();
- if (!base::WriteFile(guid_file_path, guid->c_str(), guid->size()))
- LOG(ERROR) << "Failed to write new GUID";
- return;
-}
-
-} // namespace
-
-// static
-AwMetricsServiceClient* AwMetricsServiceClient::GetInstance() {
- DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- return g_lazy_instance_.Pointer();
-}
-
-void AwMetricsServiceClient::Initialize(
- PrefService* pref_service,
- net::URLRequestContextGetter* request_context,
- const base::FilePath guid_file_path) {
- DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- DCHECK(!is_initialized_);
-
- pref_service_ = pref_service;
- request_context_ = request_context;
-
- std::string* guid = new std::string;
- // Initialization happens on the UI thread, but getting the GUID should happen
- // on the file I/O thread. So we start to initialize, then post to get the
- // GUID, and then pick up where we left off, back on the UI thread, in
- // InitializeWithGUID.
- content::BrowserThread::PostTaskAndReply(
- content::BrowserThread::FILE,
- FROM_HERE,
- base::Bind(&GetOrCreateGUID, guid_file_path, guid),
- base::Bind(&AwMetricsServiceClient::InitializeWithGUID,
- base::Unretained(this), base::Owned(guid)));
-}
-
-void AwMetricsServiceClient::InitializeWithGUID(std::string* guid) {
- DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- DCHECK(!is_initialized_);
-
- pref_service_->SetString(metrics::prefs::kMetricsClientID, *guid);
-
- metrics_state_manager_ = metrics::MetricsStateManager::Create(
- pref_service_, this, base::Bind(&StoreClientInfo),
- base::Bind(&LoadClientInfo));
-
- metrics_service_.reset(new ::metrics::MetricsService(
- metrics_state_manager_.get(), this, pref_service_));
-
- metrics_service_->RegisterMetricsProvider(
- std::unique_ptr<metrics::MetricsProvider>(
- new metrics::NetworkMetricsProvider(
- content::BrowserThread::GetBlockingPool())));
-
- metrics_service_->RegisterMetricsProvider(
- std::unique_ptr<metrics::MetricsProvider>(
- new metrics::GPUMetricsProvider));
-
- metrics_service_->RegisterMetricsProvider(
- std::unique_ptr<metrics::MetricsProvider>(
- new metrics::ScreenInfoMetricsProvider));
-
- metrics_service_->RegisterMetricsProvider(
- std::unique_ptr<metrics::MetricsProvider>(
- new metrics::ProfilerMetricsProvider()));
-
- metrics_service_->RegisterMetricsProvider(
- std::unique_ptr<metrics::MetricsProvider>(
- new metrics::CallStackProfileMetricsProvider));
-
- metrics_service_->InitializeMetricsRecordingState();
-
- is_initialized_ = true;
-
- if (IsReportingEnabled())
- metrics_service_->Start();
-}
-
-void AwMetricsServiceClient::SetMetricsEnabled(bool enabled) {
- DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-
- // If the client is already initialized, apply the setting immediately.
- // Otherwise, it will be applied on initialization.
- if (is_initialized_ && is_enabled_ != enabled) {
- if (enabled)
- metrics_service_->Start();
- else
- metrics_service_->Stop();
- }
- is_enabled_ = enabled;
-}
-
-bool AwMetricsServiceClient::IsConsentGiven() {
- DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
- return is_enabled_;
-}
-
-metrics::MetricsService* AwMetricsServiceClient::GetMetricsService() {
- return metrics_service_.get();
-}
-
-// In Chrome, UMA and Breakpad are enabled/disabled together by the same
-// checkbox and they share the same client ID (a.k.a. GUID). SetMetricsClientId
-// is intended to provide the ID to Breakpad. In WebView, UMA and Breakpad are
-// independent, so this is a no-op.
-
-void AwMetricsServiceClient::SetMetricsClientId(const std::string& client_id) {}
-
-int32_t AwMetricsServiceClient::GetProduct() {
- return ::metrics::ChromeUserMetricsExtension::ANDROID_WEBVIEW;
-}
-
-std::string AwMetricsServiceClient::GetApplicationLocale() {
- return base::i18n::GetConfiguredLocale();
-}
-
-bool AwMetricsServiceClient::GetBrand(std::string* brand_code) {
- // WebView doesn't use brand codes.
- return false;
-}
-
-metrics::SystemProfileProto::Channel AwMetricsServiceClient::GetChannel() {
- // "Channel" means stable, beta, etc. WebView doesn't have channel info yet.
- // TODO(paulmiller) Update this once we have channel info.
- return metrics::SystemProfileProto::CHANNEL_UNKNOWN;
-}
-
-std::string AwMetricsServiceClient::GetVersionString() {
- return PRODUCT_VERSION;
-}
-
-void AwMetricsServiceClient::InitializeSystemProfileMetrics(
- const base::Closure& done_callback) {
- done_callback.Run();
-}
-
-void AwMetricsServiceClient::CollectFinalMetricsForLog(
- const base::Closure& done_callback) {
- done_callback.Run();
-}
-
-std::unique_ptr<metrics::MetricsLogUploader>
-AwMetricsServiceClient::CreateUploader(
- const std::string& server_url,
- const std::string& mime_type,
- const base::Callback<void(int)>& on_upload_complete) {
- return std::unique_ptr<::metrics::MetricsLogUploader>(
- new metrics::NetMetricsLogUploader(
- request_context_, server_url, mime_type, on_upload_complete));
-}
-
-base::TimeDelta AwMetricsServiceClient::GetStandardUploadInterval() {
- return base::TimeDelta::FromMinutes(kUploadIntervalMinutes);
-}
-
-AwMetricsServiceClient::AwMetricsServiceClient()
- : is_initialized_(false),
- is_enabled_(false),
- pref_service_(nullptr),
- request_context_(nullptr) {}
+AwMetricsServiceClient::AwMetricsServiceClient() {}
AwMetricsServiceClient::~AwMetricsServiceClient() {}
« no previous file with comments | « android_webview/browser/aw_metrics_service_client.h ('k') | android_webview/glue/glue.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698