Modify rappor parameters.

components/rappor/rappor_parameters.cc

Index: components/rappor/rappor_parameters.cc
diff --git a/components/rappor/rappor_parameters.cc b/components/rappor/rappor_parameters.cc
index ede7636661e873e2ada2a3f621e5615b88a7b8c5..a69ac61f0101b07d35812a5fdc39dceb5af42890 100644
--- a/components/rappor/rappor_parameters.cc
+++ b/components/rappor/rappor_parameters.cc
@@ -4,12 +4,14 @@
 
 #include "components/rappor/rappor_parameters.h"
 
+#include "base/compiler_specific.h"
 #include "base/strings/stringprintf.h"
 
 namespace rappor {
 
 std::string RapporParameters::ToString() const {
-  return base::StringPrintf("{ %d, %d, %d, %d, %d, %d }",
+  return base::StringPrintf("{ %d, %d, %d, %d, %d, %d, %d }",
+      num_cohorts,
       bloom_filter_size_bytes,
       bloom_filter_hash_function_count,
       fake_prob,
@@ -18,4 +20,6 @@ std::string RapporParameters::ToString() const {
       zero_coin_prob);
 }
 
+STATIC_CONST_MEMBER_DEFINITION const int RapporParameters::kMaxCohorts;

[Alexei Svitkine (slow), 2014/05/21 09:13:37]

Hmm, I'd prefer if this just used the more standard syntax of:

.cc file:

// static
bool MetricsStateManager::kMaxCohorts = 512;

header file:

static const int kMaxCohorts;

[Steven Holte, 2014/05/21 19:02:57]

Done.

+
 }  // namespace rappor