Add more Rappor metrics

components/rappor/rappor_utils.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 "components/rappor/rappor_utils.h"
+
+#include "components/rappor/rappor_service.h"
+#include "net/base/registry_controlled_domains/registry_controlled_domain.h"
+
+namespace rappor {
+
+void SampleDomainAndRegistryFromGURL(RapporService* service,
+                                     const std::string& metric,
+                                     const GURL& gurl) {
+  if (!service)
+    return;
+  service->RecordSample(
+      metric,
+      rappor::ETLD_PLUS_ONE_RAPPOR_TYPE,
+      net::registry_controlled_domains::GetDomainAndRegistry(
+          gurl, net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES));

[battre, 2014/06/06 09:49:43]

please be aware that this does not record anything meaningful for
chrome-extensions://extensionID/ type of domains.

[Steven Holte, 2014/06/06 21:00:50]

Noted, though we should still be able to determine a limit on the number of URLs
we see of this type.

+}
+
+void SampleDomainAndRegistryFromHost(RapporService* service,
+                                     const std::string& metric,
+                                     const std::string& host) {
+  if (!service)
+    return;
+  service->RecordSample(
+      metric,
+      rappor::ETLD_PLUS_ONE_RAPPOR_TYPE,
+      net::registry_controlled_domains::GetDomainAndRegistry(
+          host, net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES));
+}
+
+}  // namespace rappor