Measure usage metrics to prepare for adaptive fetching rates in M55

components/ntp_snippets/content_suggestions_service_unittest.cc

Index: components/ntp_snippets/content_suggestions_service_unittest.cc
diff --git a/components/ntp_snippets/content_suggestions_service_unittest.cc b/components/ntp_snippets/content_suggestions_service_unittest.cc
index 59d721dea1683fac274a0b76566dffc1d1dbfb95..3627c3785d4bed77675820e50c770333f327e9f5 100644
--- a/components/ntp_snippets/content_suggestions_service_unittest.cc
+++ b/components/ntp_snippets/content_suggestions_service_unittest.cc
@@ -205,8 +205,9 @@ class ContentSuggestionsServiceTest : public testing::Test {
   void CreateContentSuggestionsService(
       ContentSuggestionsService::State enabled) {
     ASSERT_FALSE(service_);
-    service_.reset(
-        new ContentSuggestionsService(enabled, /*history_service=*/nullptr));
+    service_.reset(new ContentSuggestionsService(enabled,
+                                                 /*history_service=*/nullptr,

[Bernhard Bauer, 2016/09/07 13:46:45]

Could you add spaces between the comment delimiters? This all looks a bit
cramped.

[jkrcal, 2016/09/07 14:19:07]

By chromium code search, this seems the way more popular way to format it. Do
you insist?

[Bernhard Bauer, 2016/09/07 17:21:13]

Actually, what seems to be the most common style is `value /* variable_name */`,
which I would not be averse to :)

[jkrcal, 2016/09/07 19:09:35]

Done.

+                                                 /*pref_service=*/nullptr));
   }
 
   ContentSuggestionsService* service() { return service_.get(); }