Measure usage metrics to prepare for adaptive fetching rates in M55

components/ntp_snippets/content_suggestions_service_unittest.cc

 // Copyright 2016 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/ntp_snippets/content_suggestions_service.h"
 
 #include <memory>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 187 matching lines @@
     service()->RegisterProvider(std::move(provider));
     return result;
   }
 
   MOCK_METHOD1(OnImageFetched, void(const gfx::Image&));
 
  protected:
   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(); }
 
  private:
   std::unique_ptr<ContentSuggestionsService> service_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentSuggestionsServiceTest);
 };
 
@@ skipping 354 matching lines @@
   Category category = FromKnownCategory(KnownCategories::DOWNLOADS);
   MockProvider* provider = RegisterProvider(category);
   base::Time begin = base::Time::FromTimeT(123),
              end = base::Time::FromTimeT(456);
   EXPECT_CALL(*provider, ClearHistory(begin, end, _));
   base::Callback<bool(const GURL& url)> filter;
   service()->ClearHistory(begin, end, filter);
 }
 
 }  // namespace ntp_snippets