content: Remove use of GLImage from GpuMemoryBufferImpl.

content/common/gpu/client/gpu_memory_buffer_impl.cc

 // Copyright 2013 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 "content/common/gpu/client/gpu_memory_buffer_impl.h"
 
 #include "base/logging.h"
 #include "base/numerics/safe_math.h"
 #include "content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.h"
 #include "ui/gl/gl_bindings.h"
-#include "ui/gl/gl_image_memory.h"
 
 #if defined(OS_MACOSX)
 #include "content/common/gpu/client/gpu_memory_buffer_impl_io_surface.h"
 #endif
 
 #if defined(OS_ANDROID)
 #include "content/common/gpu/client/gpu_memory_buffer_impl_surface_texture.h"
 #endif
 
 #if defined(USE_OZONE)
 #include "content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h"
 #endif
 
 namespace content {
 
 GpuMemoryBufferImpl::GpuMemoryBufferImpl(gfx::GpuMemoryBufferId id,
                                          const gfx::Size& size,
                                          Format format,
                                          const DestructionCallback& callback)
     : id_(id),
       size_(size),
       format_(format),
       callback_(callback),
       mapped_(false),
       destruction_sync_point_(0) {
-  DCHECK(gfx::GLImageMemory::ValidSize(size, format));

[piman, 2015/03/03 20:48:21]

nit: it'd be nice to keep this DCHECK which is in the call stack of when the
buffer is originally created.

[reveman, 2015/03/03 23:46:26]

Added size checks to the GMB code. And validation on the browser side. It's not
required for security but if we're going to have a DCHECK on the client side
then I think we should have validation on the browser side too.

 }
 
 GpuMemoryBufferImpl::~GpuMemoryBufferImpl() {
   callback_.Run(destruction_sync_point_);
 }
 
 // static
 scoped_ptr<GpuMemoryBufferImpl> GpuMemoryBufferImpl::CreateFromHandle(
     const gfx::GpuMemoryBufferHandle& handle,
     const gfx::Size& size,
@@ skipping 71 matching lines @@
 
 bool GpuMemoryBufferImpl::IsMapped() const {
   return mapped_;
 }
 
 ClientBuffer GpuMemoryBufferImpl::AsClientBuffer() {
   return reinterpret_cast<ClientBuffer>(this);
 }
 
 }  // namespace content