glBindTexImage2DCHROMIUM should retain pixel format.

ui/gl/gl_image_egl.cc

 // Copyright (c) 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 "ui/gl/gl_image_egl.h"
 
 #include "ui/gl/egl_util.h"
 #include "ui/gl/gl_surface_egl.h"
 
 namespace gfx {
@@ skipping 33 matching lines @@
     if (result == EGL_FALSE) {
       DLOG(ERROR) << "Error destroying EGLImage: "
                   << ui::GetLastEGLErrorString();
     }
     egl_image_ = EGL_NO_IMAGE_KHR;
   }
 }
 
 gfx::Size GLImageEGL::GetSize() { return size_; }
 
+unsigned GLImageEGL::GetFormat() { return 0; }

[reveman, 2015/05/12 15:27:19]

Please return a valid default value here (say GL_RGBA) or not implement it as
part of GLImageEGL.

+
 bool GLImageEGL::BindTexImage(unsigned target) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_NE(EGL_NO_IMAGE_KHR, egl_image_);
   glEGLImageTargetTexture2DOES(target, egl_image_);
   DCHECK_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
   return true;
 }
 
 bool GLImageEGL::CopyTexImage(unsigned target) {
   return false;
 }
 
 bool GLImageEGL::ScheduleOverlayPlane(gfx::AcceleratedWidget widget,
                                       int z_order,
                                       OverlayTransform transform,
                                       const Rect& bounds_rect,
                                       const RectF& crop_rect) {
   return false;
 }
 
 }  // namespace gfx