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

services/ui/gpu/gpu_service_impl.cc

Index: services/ui/gpu/gpu_service_impl.cc
diff --git a/services/ui/gpu/gpu_service_impl.cc b/services/ui/gpu/gpu_service_impl.cc
index f0e6013217e9cfce2a69e4126ecdd3117dd32999..adf7a1b282da22c0c615865f04be0cce591e6228 100644
--- a/services/ui/gpu/gpu_service_impl.cc
+++ b/services/ui/gpu/gpu_service_impl.cc
@@ -17,7 +17,8 @@ void EstablishGpuChannelDone(
     int32_t client_id,
     const IPC::ChannelHandle& channel_handle) {
   // TODO(penghuang): Send the real GPUInfo to the client.
-  callback.Run(client_id, mojom::ChannelHandle::From(channel_handle),
+  callback.Run(client_id,
+               mojo::ScopedMessagePipeHandle(channel_handle.mojo_handle),

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

Does ChannelHandle not have strong ownership of its mojo_handle? I'm not
familiar with this part of the system, so I'm trying to understand how it works.

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

IPC::ChannelHandle is more like a data struct instead of class. It doesn't
maintain the life cycle of its members. It is just used for passing data I
think.

In here, the mojo::ScopedMessagePipleHandle() will take the ownership of the
channel_handle.mojo_handle.

                gpu::GPUInfo());
 }
 }