Simplify ppapi Graphics3D size propagation a bit

ppapi/shared_impl/ppb_graphics_3d_shared.cc

 // Copyright (c) 2012 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 "ppapi/shared_impl/ppb_graphics_3d_shared.h"
 
 #include "base/logging.h"
 #include "gpu/command_buffer/client/gles2_cmd_helper.h"
 #include "gpu/command_buffer/client/gles2_implementation.h"
 #include "gpu/command_buffer/client/shared_memory_limits.h"
 #include "gpu/command_buffer/client/transfer_buffer.h"
 #include "gpu/command_buffer/common/sync_token.h"
 #include "ppapi/c/pp_errors.h"
 
 namespace ppapi {
 
 PPB_Graphics3D_Shared::PPB_Graphics3D_Shared(PP_Instance instance)
     : Resource(OBJECT_IS_IMPL, instance) {}
 
-PPB_Graphics3D_Shared::PPB_Graphics3D_Shared(const HostResource& host_resource)
-    : Resource(OBJECT_IS_PROXY, host_resource) {}
+PPB_Graphics3D_Shared::PPB_Graphics3D_Shared(const HostResource& host_resource,
+                                             const gfx::Size& size)
+    : Resource(OBJECT_IS_PROXY, host_resource), size_(size) {}
 
 PPB_Graphics3D_Shared::~PPB_Graphics3D_Shared() {
   // Make sure that GLES2 implementation has already been destroyed.
   DCHECK(!gles2_helper_.get());
   DCHECK(!transfer_buffer_.get());
   DCHECK(!gles2_impl_.get());
 }
 
 thunk::PPB_Graphics3D_API* PPB_Graphics3D_Shared::AsPPB_Graphics3D_API() {
   return this;
@@ skipping 12 matching lines @@
 int32_t PPB_Graphics3D_Shared::GetError() {
   // TODO(alokp): Implement me.
   return PP_ERROR_FAILED;
 }
 
 int32_t PPB_Graphics3D_Shared::ResizeBuffers(int32_t width, int32_t height) {
   if ((width < 0) || (height < 0))
     return PP_ERROR_BADARGUMENT;
 
   gles2_impl()->ResizeCHROMIUM(width, height, 1.f, true);
-  width_ = width;
-  height_ = height;
+  size_ = gfx::Size(width, height);
   // TODO(alokp): Check if resize succeeded and return appropriate error code.
   return PP_OK;
 }
 
 int32_t PPB_Graphics3D_Shared::SwapBuffers(
     scoped_refptr<TrackedCallback> callback) {
-  return SwapBuffersWithSyncToken(callback, gpu::SyncToken(), width_, height_);
+  return SwapBuffersWithSyncToken(callback, gpu::SyncToken(), size_);
 }
 
 int32_t PPB_Graphics3D_Shared::SwapBuffersWithSyncToken(
     scoped_refptr<TrackedCallback> callback,
     const gpu::SyncToken& sync_token,
-    int32_t width,
-    int32_t height) {
+    const gfx::Size& size) {
   if (HasPendingSwap()) {
     Log(PP_LOGLEVEL_ERROR,
         "PPB_Graphics3D.SwapBuffers: Plugin attempted swap "
         "with previous swap still pending.");
     // Already a pending SwapBuffers that hasn't returned yet.
     return PP_ERROR_INPROGRESS;
   }
 
   swap_callback_ = callback;
-  return DoSwapBuffers(sync_token, width, height);
+  return DoSwapBuffers(sync_token, size);
 }
 
 int32_t PPB_Graphics3D_Shared::GetAttribMaxValue(int32_t attribute,
                                                  int32_t* value) {
   // TODO(alokp): Implement me.
   return PP_ERROR_FAILED;
 }
 
 void* PPB_Graphics3D_Shared::MapTexSubImage2DCHROMIUM(GLenum target,
                                                       GLint level,
@@ skipping 69 matching lines @@
   return true;
 }
 
 void PPB_Graphics3D_Shared::DestroyGLES2Impl() {
   gles2_impl_.reset();
   transfer_buffer_.reset();
   gles2_helper_.reset();
 }
 
 }  // namespace ppapi