Add RAPPOR metrics of ServiceWorker.ControlledPage

content/browser/service_worker/service_worker_metrics.cc

Index: content/browser/service_worker/service_worker_metrics.cc
diff --git a/content/browser/service_worker/service_worker_metrics.cc b/content/browser/service_worker/service_worker_metrics.cc
index 6e3ddf1d7aaded7762fd92449e45077a38047d8d..d984384988248f3a051d87a21d207a8a7eea8f8f 100644
--- a/content/browser/service_worker/service_worker_metrics.cc
+++ b/content/browser/service_worker/service_worker_metrics.cc
@@ -6,10 +6,24 @@
 
 #include "base/metrics/histogram.h"
 #include "base/metrics/user_metrics_action.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/user_metrics.h"
+#include "content/public/common/content_client.h"
 
 namespace content {
 
+namespace {
+
+void SampleDomainAndRegistryFromGURLOnUI(const GURL& url) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  GetContentClient()->browser()->SampleDomainAndRegistryFromGURL(
+      "ServiceWorker.ControlledPage",

[falken, 2015/02/25 03:36:58]

just bikeshedding now... "ControlledPageUrl" would be clearer for me to
understand the diff between ControlledPageLoad.

[horo, 2015/02/25 04:30:04]

Done.

+      url);
+}
+
+}  // namespace
+
 // static
 void ServiceWorkerMetrics::CountInitDiskCacheResult(bool result) {
   UMA_HISTOGRAM_BOOLEAN("ServiceWorker.DiskCache.InitResult", result);
@@ -51,8 +65,12 @@ void ServiceWorkerMetrics::CountWriteDatabaseResult(
 }
 
 // static
-void ServiceWorkerMetrics::CountControlledPageLoad() {
+void ServiceWorkerMetrics::CountControlledPageLoad(const GURL& url) {
   RecordAction(base::UserMetricsAction("ServiceWorker.ControlledPageLoad"));
+  BrowserThread::PostTask(
+      BrowserThread::UI,
+      FROM_HERE,
+      base::Bind(&SampleDomainAndRegistryFromGURLOnUI, url));
 }
 
 }  // namespace content