Remove stray std::move()s from reporter_runner_win.cc

This CL replaces std::move()s whose result is not used, as clang starts
warning that on C++17 mode.

Bug: 752720
Change-Id: I843ba7d9356fdb14fe5cc0c258018fae201a32c6
Reviewed-on: https://chromium-review.googlesource.com/1068639Reviewed-by: Joe Mason <joenotcharles@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560708}

chrome/browser/safe_browsing/chrome_cleaner/reporter_runner_win.cc

@@ -634,7 +634,7 @@ class ReporterRunner {
 
     // The reporter sequence has been scheduled to run, so don't notify that
     // it has not been scheduled.
-    std::move(scoped_runner.Release());
+    ignore_result(scoped_runner.Release());
   }
 
  private:
@@ -765,8 +765,8 @@ class ReporterRunner {
     if (!invocations_.container().empty()) {
       // If there are other invocations to start, then we shouldn't finalize
       // this object. ScopedClosureRunner::Release requires its return value to
-      // be used, so simply std::move it, since it will not be needed.
-      std::move(scoped_runner.Release());
+      // be used, so simply ignore_result it, since it will not be needed.
+      ignore_result(scoped_runner.Release());
       PostNextInvocation();
     }