Mojo C++ bindings: Generate InterfaceHandle<> instead of InterfacePtr<>.

services/http_server/http_server_impl.cc

Index: services/http_server/http_server_impl.cc
diff --git a/services/http_server/http_server_impl.cc b/services/http_server/http_server_impl.cc
index fa7f5b34215cfb7dcabf0d53d94a50f24e59e731..c05f39cfa07bc84151ad03faf3516941105f5fb7 100644
--- a/services/http_server/http_server_impl.cc
+++ b/services/http_server/http_server_impl.cc
@@ -4,6 +4,8 @@
 
 #include "services/http_server/http_server_impl.h"
 
+#include <utility>
+
 #include "base/bind.h"
 #include "base/logging.h"
 #include "mojo/public/cpp/application/application_impl.h"
@@ -33,14 +35,15 @@ void HttpServerImpl::AddBinding(mojo::InterfaceRequest<HttpServer> request) {
 }
 
 void HttpServerImpl::SetHandler(const mojo::String& path,
-                                HttpHandlerPtr http_handler,
+                                mojo::InterfaceHandle<HttpHandler> http_handler,
                                 const mojo::Callback<void(bool)>& callback) {
   for (const auto& handler : handlers_) {
     if (handler->pattern->pattern() == path)
       callback.Run(false);
   }
 
-  Handler* handler = new Handler(path, http_handler.Pass());
+  Handler* handler =
+      new Handler(path, HttpHandlerPtr::Create(std::move(http_handler)));

[viettrungluu, 2016/02/11 18:26:29]

Maybe make Handler's ctor take an IH instead? <shrug>

[vardhan, 2016/02/11 22:47:53]

Handler more or less just holds data (if you look at how this http handler is
accessed).  I think it makes sense to convert InterfaceHandle<> to
InterfacePtr<> here.

   handler->http_handler.set_connection_error_handler(
       [this, handler]() { OnHandlerConnectionError(handler); });
   handlers_.push_back(handler);