Support move-only type on base::Callback::Run

base/callback.h

Index: base/callback.h
diff --git a/base/callback.h b/base/callback.h
index 3bf0008b6d33bf73e075f502523327817ddb1345..5f2fe8cf07be85c1115fff4f1ed6c3412d6d5373 100644
--- a/base/callback.h
+++ b/base/callback.h
@@ -386,18 +386,14 @@ class Callback<R(Args...)> : public internal::CallbackBase {
     return CallbackBase::Equals(other);
   }
 
-  R Run(typename internal::CallbackParamTraits<Args>::ForwardType... args)
-      const {
+  R Run(Args... args) const {

[dcheng, 2016/03/02 00:25:45]

So this costs an extra copy now, since we don't have enough information to do
perfect forwarding: this was determined when we instantiated the callback, when
we don't yet know how Run() will be called.

We can't make this use forwarding references, because we need to instantiate a
reference to this in the ctor. So this is the best we can do.

(Maybe we could have a comment in the CL description or in the code or in both
that documents why the seemingly-obvious perfect forwarding pattern doesn't work
here?)

[tzik, 2016/03/02 01:04:21]

Acknowledged. Updating.

[tzik, 2016/03/02 12:44:51]

Done.

     PolymorphicInvoke f =
         reinterpret_cast<PolymorphicInvoke>(polymorphic_invoke_);
-
-    return f(bind_state_.get(), internal::CallbackForward(args)...);
+    return f(bind_state_.get(), std::forward<Args>(args)...);
   }
 
  private:
-  using PolymorphicInvoke =
-      R(*)(internal::BindStateBase*,
-           typename internal::CallbackParamTraits<Args>::ForwardType...);
+  using PolymorphicInvoke = R (*)(internal::BindStateBase*, Args&&...);
 };
 
 }  // namespace base