Update GPU memory buffers to use StrideInBytes internally.

ui/gl/gl_image_memory.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 "ui/gl/gl_image_memory.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "ui/gl/gl_bindings.h"
 #include "ui/gl/scoped_binders.h"
@@ skipping 82 matching lines @@
 
 GLImageMemory::~GLImageMemory() {
 #if defined(OS_WIN) || defined(USE_X11) || defined(OS_ANDROID) || \
     defined(USE_OZONE)
   DCHECK_EQ(EGL_NO_IMAGE_KHR, egl_image_);
   DCHECK_EQ(0u, egl_texture_id_);
 #endif
 }
 
 // static
-size_t GLImageMemory::BytesPerPixel(gfx::GpuMemoryBuffer::Format format) {
+size_t GLImageMemory::StrideInBytes(
+    size_t width, gfx::GpuMemoryBuffer::Format format) {
   switch (format) {
     case gfx::GpuMemoryBuffer::RGBA_8888:
     case gfx::GpuMemoryBuffer::BGRA_8888:
-      return 4;
+      return width * 4;

[reveman, 2014/12/18 18:49:57]

Can this overflow? Should it be using CheckedNumeric just like SizeInBytes in
gl_image_shared_memory.cc?

[christiank, 2015/01/12 10:35:23]

Yes, think it can. It's now updated to use CheckedNumeric.

     case gfx::GpuMemoryBuffer::RGBX_8888:
       NOTREACHED();
       return 0;
   }
 
   NOTREACHED();
   return 0;
 }
 
 bool GLImageMemory::Initialize(const unsigned char* memory,
@@ skipping 162 matching lines @@
                TextureFormat(format_),
                size_.width(),
                size_.height(),
                0,  // border
                DataFormat(format_),
                DataType(format_),
                memory_);
 }
 
 }  // namespace gfx