[Remote suggestions] Remove the unique_ptr wrapper around callbacks

components/ntp_snippets/remote/remote_suggestions_scheduler_impl_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/remote/remote_suggestions_scheduler_impl.h"
 
 #include <memory>
 #include <set>
 #include <string>
 #include <utility>
@@ skipping 54 matching lines @@
                bool(base::TimeDelta period_wifi,
                     base::TimeDelta period_fallback));
   MOCK_METHOD0(Unschedule, bool());
 };
 
 // TODO(jkrcal): Move into its own library to reuse in other unit-tests?
 class MockRemoteSuggestionsProvider : public RemoteSuggestionsProvider {
  public:
   MockRemoteSuggestionsProvider(Observer* observer)
       : RemoteSuggestionsProvider(observer) {}
-  // Move-only params are not supported by GMock. We want to mock out
-  // RefetchInTheBackground() which takes a unique_ptr<>. Instead, we add a new
-  // mock function which takes a copy of the callback and override the
-  // RemoteSuggestionsProvider's method to forward the call into the new mock
-  // function.
-  void RefetchInTheBackground(
-      std::unique_ptr<RemoteSuggestionsProvider::FetchStatusCallback> callback)
-      override {
-    RefetchInTheBackground(*callback);
-  }
   MOCK_METHOD1(RefetchInTheBackground,
-               void(RemoteSuggestionsProvider::FetchStatusCallback));
-
+               void(const RemoteSuggestionsProvider::FetchStatusCallback&));
   MOCK_CONST_METHOD0(suggestions_fetcher_for_debugging,
                      const RemoteSuggestionsFetcher*());
-
   MOCK_METHOD1(GetCategoryStatus, CategoryStatus(Category));
   MOCK_METHOD1(GetCategoryInfo, CategoryInfo(Category));
   MOCK_METHOD3(ClearHistory,
                void(base::Time begin,
                     base::Time end,
                     const base::Callback<bool(const GURL& url)>& filter));
   MOCK_METHOD3(Fetch,
                void(const Category&,
                     const std::set<std::string>&,
                     const FetchDoneCallback&));
@@ skipping 678 matching lines @@
   for (int x = 0; x < 5; ++x) {
     scheduler()->OnPersistentSchedulerWakeUp();
     signal_fetch_done.Run(Status::Success());
   }
 
   // For the 6th time, it is blocked by the scheduling provider.
   scheduler()->OnPersistentSchedulerWakeUp();
 }
 
 }  // namespace ntp_snippets