Pass Callback to TaskRunner by value and consume it on invocation (1)

components/subresource_filter/content/browser/content_ruleset_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/subresource_filter/content/browser/content_ruleset_service.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <tuple>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
+#include "base/callback_helpers.h"

[engedy, 2017/03/23 14:09:26]

nit: I think this isn't actually used here.

[tzik, 2017/03/24 09:25:55]

Removed. Right, it was needed by ResetAndReturn in the previous CL and is no
longer needed.

 #include "base/files/file.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/numerics/safe_conversions.h"
 #include "base/run_loop.h"
 #include "base/task_runner.h"
 #include "components/subresource_filter/content/common/subresource_filter_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/notification_service.h"
@@ skipping 14 matching lines @@
 using MockClosureTarget =
     ::testing::StrictMock<::testing::MockFunction<void()>>;
 
 class TestContentBrowserClient : public ::content::ContentBrowserClient {
  public:
   TestContentBrowserClient() {}
 
   // ::content::ContentBrowserClient:
   void PostAfterStartupTask(const tracked_objects::Location&,
                             const scoped_refptr<base::TaskRunner>& task_runner,
-                            const base::Closure& task) override {
+                            base::Closure task) override {
     scoped_refptr<base::TaskRunner> ui_task_runner =
         content::BrowserThread::GetTaskRunnerForThread(
             content::BrowserThread::UI);
     EXPECT_EQ(ui_task_runner, task_runner);
-    last_task_ = task;
+    last_task_ = std::move(task);
   }
 
   void RunAfterStartupTask() {
     if (!last_task_.is_null())
-      last_task_.Run();
+      std::move(last_task_).Run();
   }
 
  private:
   base::Closure last_task_;
 
   DISALLOW_COPY_AND_ASSIGN(TestContentBrowserClient);
 };
 
 class NotifyingMockRenderProcessHost : public content::MockRenderProcessHost {
  public:
@@ skipping 111 matching lines @@
 
   MockClosureTarget mock_closure_target;
   service.PostAfterStartupTask(base::Bind(
       &MockClosureTarget::Call, base::Unretained(&mock_closure_target)));
 
   EXPECT_CALL(mock_closure_target, Call()).Times(1);
   browser_client()->RunAfterStartupTask();
 }
 
 }  // namespace subresource_filter