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

Unified Diff: chrome/browser/metrics/chrome_metrics_service_client.cc

Issue 2893943004: Refactor UKM interface for mojo-ification (Closed)
Patch Set: Fixed contextualsearch Created 3 years, 7 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: chrome/browser/metrics/chrome_metrics_service_client.cc
diff --git a/chrome/browser/metrics/chrome_metrics_service_client.cc b/chrome/browser/metrics/chrome_metrics_service_client.cc
index 96ec8134463afdeb0611305ebf740368b279c1aa..ce0c71ad903b26deaa648d09dba9ababcf83a254 100644
--- a/chrome/browser/metrics/chrome_metrics_service_client.cc
+++ b/chrome/browser/metrics/chrome_metrics_service_client.cc
@@ -281,8 +281,11 @@ void GetExecutableVersionDetails(base::string16* product_name,
}
#endif // OS_WIN
-ukm::UkmService* BindableGetUkmService() {
- return g_browser_process->ukm_service();
+ukm::UkmService* BindableGetUkmService(
+ base::WeakPtr<ChromeMetricsServiceClient> weak_ptr) {
+ if (!weak_ptr)
+ return nullptr;
+ return weak_ptr->GetUkmService();
}
} // namespace
@@ -935,7 +938,8 @@ void ChromeMetricsServiceClient::RegisterForNotifications() {
void ChromeMetricsServiceClient::RegisterForProfileEvents(Profile* profile) {
// Register chrome://ukm handler
content::URLDataSource::Add(
- profile, new ukm::debug::DebugPage(base::Bind(&BindableGetUkmService)));
+ profile, new ukm::debug::DebugPage(base::Bind(
+ &BindableGetUkmService, weak_ptr_factory_.GetWeakPtr())));
#if defined(OS_CHROMEOS)
// Ignore the signin profile for sync disables / history deletion.
if (chromeos::ProfileHelper::IsSigninProfile(profile))
« no previous file with comments | « chrome/browser/browser_process_impl.cc ('k') | chrome/browser/page_load_metrics/observers/ukm_page_load_metrics_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698