commit 6f09ae20bd3f41eb9ccfab45ca8d039395c007ab
author Danil Chapovalov <danilchap@webrtc.org> Thu Oct 12 14:39:25 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Fri Oct 13 08:56:06 2017 +0000
tree b1e92836a15bcce6cb97089420f8decbd8f0dd58
parent 75dfd1c174933a139ff08f49f174c2db78e8e704

Support move-only closures in TaskQueue

it allows to create less boil-plate code when marshaling call
with move-only parameters (e.g. unique_ptr) to TaskQueue

Bug: None
Change-Id: I97ddf4f8409af2f83d69fd33267e9a87fb60d901
Reviewed-on: https://webrtc-review.googlesource.com/7619
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20271}

rtc_base/task_queue_win.cc

diff --git a/rtc_base/task_queue_win.cc b/rtc_base/task_queue_win.cc
index 1d069e6..082bf9d 100644
--- a/rtc_base/task_queue_win.cc
+++ b/rtc_base/task_queue_win.cc
@@ -15,6 +15,7 @@
 
 #include <algorithm>
 #include <queue>
+#include <utility>
 
 #include "rtc_base/arraysize.h"
 #include "rtc_base/checks.h"
@@ -170,10 +171,11 @@
   bool IsCurrent() const;
 
   template <class Closure,
-            typename std::enable_if<
-                std::is_copy_constructible<Closure>::value>::type* = nullptr>
-  void PostTask(const Closure& closure) {
-    PostTask(std::unique_ptr<QueuedTask>(new ClosureTask<Closure>(closure)));
+            typename std::enable_if<!std::is_convertible<
+                Closure,
+                std::unique_ptr<QueuedTask>>::value>::type* = nullptr>
+  void PostTask(Closure&& closure) {
+    PostTask(NewClosure(std::forward<Closure>(closure)));
   }
 
   void PostTask(std::unique_ptr<QueuedTask> task);