Declient calculator test mojom.

mojo/public/cpp/bindings/callback.h

 // Copyright 2014 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.
 
 #ifndef MOJO_PUBLIC_CPP_BINDINGS_CALLBACK_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_CALLBACK_H_
 
 #include "mojo/public/cpp/bindings/lib/callback_internal.h"
 #include "mojo/public/cpp/bindings/lib/shared_ptr.h"
 #include "mojo/public/cpp/bindings/lib/template_util.h"
@@ skipping 44 matching lines @@
       sink.Run(internal::Forward(args)...);
     }
     Sink sink;
   };
 
   internal::SharedPtr<Runnable> sink_;
 };
 
 typedef Callback<void()> Closure;
 
+namespace {

[jamesr, 2015/01/22 22:45:24]

anonymous namespace inside a header is almost never what you want - it means
that if two translation units #include this header they will not have a shared
type that they can refer to

[Aaron Boodman, 2015/01/22 22:49:08]

You're right, thanks.

+
+template <typename Method, typename Class>
+class RunnableImpl {

[jamesr, 2015/01/22 22:45:24]

don't put this in public. If you want it in your test feel free to put it there,
but I think if we're going to expand the public Callback we should have it
actually accept real functors for Sink.  The c++11 patch I linked earlier does
this and it's possible to do so in c++98 with SFINAE but nobody's got around to
doing that.  This adds another confusing layer without letting you use the
syntax you really want.

[Aaron Boodman, 2015/01/22 22:49:08]

done.

+ public:
+  RunnableImpl(Method method, Class instance)
+      : method_(method), instance_(instance) {}
+  template <typename... Args>
+  void Run(Args... args) const {
+    (instance_->*method_)(args...);
+  }
+
+ private:
+  Method method_;
+  Class instance_;
+};
+
+}  // namespace
+
+// Convenience to create a create a runnable for any method.
+// |Class| can be any copyable type with an -> operator.
+template <typename Method, typename Class>
+RunnableImpl<Method, Class> MakeRunnable(Method method, Class object) {
+  return RunnableImpl<Method, Class>(method, object);
+}
+
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_CALLBACK_H_