services/ui: Get rid of IPC::ChannelHandle type converter

services/ui/public/cpp/gpu_service.cc

Index: services/ui/public/cpp/gpu_service.cc
diff --git a/services/ui/public/cpp/gpu_service.cc b/services/ui/public/cpp/gpu_service.cc
index 6c70440f940189eca61194151bf4c52152597284..a9c9b4d2b0b2cba4202a277cfb8b5a124a86b110 100644
--- a/services/ui/public/cpp/gpu_service.cc
+++ b/services/ui/public/cpp/gpu_service.cc
@@ -151,7 +151,7 @@ void GpuService::EstablishGpuChannelOnMainThreadSyncLocked() {
   DCHECK(!gpu_service_);
 
   int client_id = 0;
-  mojom::ChannelHandlePtr channel_handle;
+  mojo::ScopedMessagePipeHandle channel_handle;
   gpu::GPUInfo gpu_info;
   connector_->ConnectToInterface("mojo:ui", &gpu_service_);
   {
@@ -172,7 +172,7 @@ void GpuService::EstablishGpuChannelOnMainThreadSyncLocked() {
 void GpuService::EstablishGpuChannelOnMainThreadDone(
     bool locked,
     int client_id,
-    mojom::ChannelHandlePtr channel_handle,
+    mojo::ScopedMessagePipeHandle channel_handle,
     const gpu::GPUInfo& gpu_info) {
   DCHECK(IsMainThread());
   scoped_refptr<gpu::GpuChannelHost> gpu_channel;
@@ -180,7 +180,7 @@ void GpuService::EstablishGpuChannelOnMainThreadDone(
     // TODO(penghuang): Get the real gpu info from mus.
     gpu_channel = gpu::GpuChannelHost::Create(
         this, client_id, gpu::GPUInfo(),
-        channel_handle.To<IPC::ChannelHandle>(), &shutdown_event_,
+        IPC::ChannelHandle(channel_handle.release()), &shutdown_event_,

[dcheng, 2016/08/17 19:38:29]

Nit: I think std::move() might work here.

[Peng, 2016/08/17 19:59:17]

Just tried. std::move() doesn't work. It is because IP::ChannelHandle() only
accepts mojo::MessagePipeHandle, it dons't
accept mojo::ScopedMessagePipeHandle.

         gpu_memory_buffer_manager_.get());
   }