RAPPOR implementation

components/rappor/rappor_registry.cc

Index: components/rappor/rappor_registry.cc
diff --git a/components/rappor/rappor_registry.cc b/components/rappor/rappor_registry.cc
new file mode 100644
index 0000000000000000000000000000000000000000..af36d9e312371fe1dfd14f61026a0110d37226a8
--- /dev/null
+++ b/components/rappor/rappor_registry.cc
@@ -0,0 +1,23 @@
+// 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_registry.h"
+
+#include "components/rappor/rappor_parameters.h"
+
+namespace rappor {
+
+extern const RapporParameters g_rappor_registry[NUM_RAPPOR_REGISTRY_ENTRIES] = {
+    {  // HOME_PAGE_RAPPOR
+        "Settings.HomePage" /* Metric name */,

[Alexei Svitkine (slow), 2014/02/04 19:20:12]

This wasn't quite what I meant.

I think the name should still be passed to the API, but the other params should
be in the registry referenced by the enum.

(And the enum name should be more generic - e.g. RAPPOR_URL rather than
RAPPOR_HOMEPAGE).

To avoid having to refactor things too much, maybe you can still keep the name
in RapporParameters, but have an empty name for all of these pre-defined ones
and have a call to RapporService copy the params over and fill the name.

[Steven Holte, 2014/02/04 21:41:21]

Split name and parameters.  PTAL.

+        16 /* Bloom filter size bytes */,
+        2 /* Bloom filter hash count */,
+        rappor::PROBABILITY_75 /* Fake data probability */,
+        rappor::PROBABILITY_50 /* Fake one probability */,
+        rappor::PROBABILITY_75 /* One coin probability */,
+        rappor::PROBABILITY_50 /* Zero coin probability */
+    },
+};
+
+}  // namespace rappor