GLImage support for Android zero-copy pixel buffers

ui/gl/gl_image_linux.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_glx.h"
 #include "ui/gl/gl_image_stub.h"
 #include "ui/gl/gl_implementation.h"
@@ skipping 17 matching lines @@
       return NULL;
     }
     case kGLImplementationMockGL:
       return new GLImageStub;
     default:
       NOTREACHED();
       return NULL;
   }
 }
 
+scoped_refptr<GLImage> GLImage::CreateGLImageForPixelBuffer(
+    gfx::PixelBufferHandle window, const gfx::Size& size) {
+  TRACE_EVENT0("gpu", "GLImage::CreateGLImageForPixelBuffer");
+  switch (GetGLImplementation()) {

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

Please return NULL here for kGLImplementationOSMesaGL,
kGLImplementationDesktopGL, kGLImplementationEGLGLES2 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