Change mojo geometry structs from using type converters to StructTraits.

components/mus/public/cpp/lib/command_buffer_client_impl.cc

 // 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.
 
 #include "components/mus/public/cpp/lib/command_buffer_client_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <limits>
@@ skipping 195 matching lines @@
 gpu::Capabilities CommandBufferClientImpl::GetCapabilities() {
   return capabilities_;
 }
 
 int32_t CommandBufferClientImpl::CreateImage(ClientBuffer buffer,
                                              size_t width,
                                              size_t height,
                                              unsigned internalformat) {
   int32_t new_id = ++next_image_id_;
 
-  mojo::SizePtr size = mojo::Size::New();
-  size->width = static_cast<int32_t>(width);
-  size->height = static_cast<int32_t>(height);
+  gfx::Size size(static_cast<int32_t>(width), static_cast<int32_t>(height));

[dcheng, 2016/05/26 22:20:40]

Not related to your change but I wonder why this function takes size_t's when
there's many assumptions elsewhere that the parameters need to fit in an int...

 
   mus::MojoGpuMemoryBufferImpl* gpu_memory_buffer =
       mus::MojoGpuMemoryBufferImpl::FromClientBuffer(buffer);
   gfx::GpuMemoryBufferHandle handle = gpu_memory_buffer->GetHandle();
 
   bool requires_sync_point = false;
   if (handle.type != gfx::SHARED_MEMORY_BUFFER) {
     requires_sync_point = true;
     NOTIMPLEMENTED();
     return -1;
@@ skipping 140 matching lines @@
 
   // It is also safe to wait on the same context.
   if (sync_token->namespace_id() == gpu::CommandBufferNamespace::MOJO &&
       sync_token->command_buffer_id() == GetCommandBufferID())
     return true;
 
   return false;
 }
 
 }  // namespace mus