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

Unified Diff: android_webview/browser/aw_browser_context.cc

Issue 1563093005: Keep per-app GUID for WebView Metrics (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: get GUID on FILE thread Created 4 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
Index: android_webview/browser/aw_browser_context.cc
diff --git a/android_webview/browser/aw_browser_context.cc b/android_webview/browser/aw_browser_context.cc
index 098b95577fb4ae4fdfb9d5ce9bf609f6d9838e1a..8cf20eaec7b46b5dd7e209e87ef558289a8ad9b4 100644
--- a/android_webview/browser/aw_browser_context.cc
+++ b/android_webview/browser/aw_browser_context.cc
@@ -256,8 +256,15 @@ void AwBrowserContext::PreMainMessageLoopRun() {
blacklist_manager_.reset(CreateURLBlackListManager(user_pref_service_.get()));
+ // UMA uses randomly-generated GUIDs (globally unique identifiers) to
+ // anonymously identify logs. Every WebView-using app on every device
+ // is given a GUID, stored in this file in the app's data directory.
+ const FilePath guid_file_path =
+ GetPath().Append(FILE_PATH_LITERAL("metrics_guid"));
+
AwMetricsServiceClient::GetInstance()->Initialize(user_pref_service_.get(),
- GetRequestContext());
+ GetRequestContext(),
+ guid_file_path);
}
void AwBrowserContext::PostMainMessageLoopRun() {

Powered by Google App Engine
This is Rietveld 408576698