Add more Rappor metrics

components/rappor/rappor_utils.h

+// 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.
+
+#ifndef COMPONENTS_RAPPOR_RAPPOR_UTILS_H_
+#define COMPONENTS_RAPPOR_RAPPOR_UTILS_H_
+
+#include <string>
+
+class GURL;
+
+namespace rappor {
+
+class RapporService;
+
+// Record the ETLD+1 of a url to a Rappor metric.

[Alexei Svitkine (slow), 2014/06/05 14:23:35]

Nit: Mention that service may be NULL and what happens in that case. Same below.

[Steven Holte, 2014/06/05 20:17:07]

Done.

+void SampleETLDp1(RapporService* service,
+                  const std::string& metric,
+                  const GURL& gurl);
+
+// Record the ETLD+1 of a host to a Rappor metric.
+void SampleETLDp1(RapporService* service,

[Alexei Svitkine (slow), 2014/06/05 14:23:35]

Don't overload function names. Use a different name for this one.

[Steven Holte, 2014/06/05 20:17:07]

Done.

+                  const std::string& metric,
+                  const std::string& host);
+
+}  // namespace rappor
+
+#endif  // COMPONENTS_RAPPOR_RAPPOR_UTILS_H_