GLImage support for Android zero-copy pixel buffers

ui/gl/gl_image_mac.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 "ui/gl/gl_image.h"
 
 #include "base/debug/trace_event.h"
 #include "ui/gl/gl_image_stub.h"
 #include "ui/gl/gl_implementation.h"
 
 namespace gfx {
 
 scoped_refptr<GLImage> GLImage::CreateGLImage(gfx::PluginWindowHandle window) {
   TRACE_EVENT0("gpu", "GLImage::CreateGLImage");
   switch (GetGLImplementation()) {
     case kGLImplementationOSMesaGL:
     case kGLImplementationDesktopGL:
     case kGLImplementationAppleGL: {
       return NULL;
     }
     case kGLImplementationMockGL:
       return new GLImageStub;
     default:
       NOTREACHED();
       return NULL;
   }
 }
 
+scoped_refptr<GLImage> GLImage::CreateGLImageForGraphicBuffer(
+    gfx::GraphicBufferHandle window, const gfx::Size& size) {
+  TRACE_EVENT0("gpu", "GLImage::CreateGLImageForGraphicBuffer");
+  switch (GetGLImplementation()) {

[reveman, 2013/04/05 00:13:24]

Please return NULL here for kGLImplementationOSMesaGL,
kGLImplementationDesktopGL, kGLImplementationAppleGL to be consistent with
GLImage::CreateGLImage above.

[kaanb, 2013/04/05 01:02:52]

Done.

+    case kGLImplementationMockGL:
+      return new GLImageStub;
+    default:
+      NOTREACHED();
+      return NULL;
+  }
+  return NULL;

[reveman, 2013/04/05 00:13:24]

nit: "return NULL" is redundant

[kaanb, 2013/04/05 01:02:52]

Done.

+}
+
 }  // namespace gfx