Add RAPPOR metrics of ServiceWorker.ControlledPage

content/browser/service_worker/service_worker_metrics.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/service_worker/service_worker_metrics.h"
 
 #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);
 }
 
 // static
 void ServiceWorkerMetrics::CountReadResponseResult(
     ServiceWorkerMetrics::ReadResponseResult result) {
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.DiskCache.ReadResponseResult",
                             result, NUM_READ_RESPONSE_RESULT_TYPES);
@@ skipping 21 matching lines @@
 }
 
 // static
 void ServiceWorkerMetrics::CountWriteDatabaseResult(
     ServiceWorkerDatabase::Status status) {
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.Database.WriteResult",
                             status, ServiceWorkerDatabase::STATUS_ERROR_MAX);
 }
 
 // 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