Alternative Multi-dimensional Rappor

components/rappor/bloom_filter_unittest.cc

Index: components/rappor/bloom_filter_unittest.cc
diff --git a/components/rappor/bloom_filter_unittest.cc b/components/rappor/bloom_filter_unittest.cc
index 25f965f8402835ea354ea5f5f391a839baa9d456..679ffdc50639d398ab16be145faf0e07f87d66cd 100644
--- a/components/rappor/bloom_filter_unittest.cc
+++ b/components/rappor/bloom_filter_unittest.cc
@@ -51,4 +51,9 @@ TEST(BloomFilterTest, HugeFilter) {
   EXPECT_EQ(1, CountBits(filter.bytes()));
 }
 
+TEST(BloomFilterTest, GetFunction) {

[Alexei Svitkine (slow), 2015/04/23 21:42:34]

Call the test GetBloomBits?

[Steven Holte, 2015/04/24 16:59:05]

Done.

+  uint64_t bytes_from_get = internal::GetBloomBits(1u, 4u, 0u, "Bar");
+  EXPECT_EQ(uint64_t(0xa8), bytes_from_get);

[Alexei Svitkine (slow), 2015/04/23 21:42:34]

This style cast is against style guide - either static_cast or maybe you can
express the literal with a 'u' suffix to not need the cast?

[Steven Holte, 2015/04/24 16:59:05]

Done.

+}
+
 }  // namespace rappor