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..ef89ae82d46fe48e5761dd8ab9a1983ae1b779f4 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) {

[falken, 2015/02/25 01:43:59]

missing space before url (weird, does that compile?)

[horo, 2015/02/25 02:27:00]

Done.

+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  GetContentClient()->browser()->SampleDomainAndRegistryFromGURL(
+      "ServiceWorker.ControlledPageLoad",

[falken, 2015/02/25 01:43:59]

I might not understand RAPPOR enough, but I feel we shouldn't use the same
metric name for the UMA count and the RAPPOR one? Or are they supposed to be
linked together somehow?

[horo, 2015/02/25 02:27:00]

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