Add more Rappor metrics

components/rappor/rappor_utils.cc

Index: components/rappor/rappor_utils.cc
diff --git a/components/rappor/rappor_utils.cc b/components/rappor/rappor_utils.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fc0a76fcd7d7922f08cc3e0644db4674f9050815
--- /dev/null
+++ b/components/rappor/rappor_utils.cc
@@ -0,0 +1,36 @@
+// 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