ozone: OZONE_NATIVE_BUFFER requires GL_TEXTURE_EXTERNAL_OES.

content/browser/gpu/browser_gpu_channel_host_factory.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 "content/browser/gpu/browser_gpu_channel_host_factory.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/profiler/scoped_tracker.h"
@@ skipping 244 matching lines @@
 
   std::vector<gfx::GpuMemoryBufferType> supported_types;
   GpuMemoryBufferFactory::GetSupportedTypes(&supported_types);
   DCHECK(!supported_types.empty());
 
   // The GPU service will always use the preferred type.
   gfx::GpuMemoryBufferType type = supported_types[0];
 
   switch (type) {
     case gfx::SURFACE_TEXTURE_BUFFER:
-      // Surface texture backed GPU memory buffers require
-      // TEXTURE_EXTERNAL_OES.
+    case gfx::OZONE_NATIVE_BUFFER:
+      // EGLImage backed GPU memory buffers require TEXTURE_EXTERNAL_OES.

[reveman, 2015/04/16 19:39:20]

nit: These buffers are not technically backed by EGLImages. EGLImages are just
used as a mechanism to share these buffers with the GL. Maybe "GPU memory
buffers that are shared with the GL using EGLImages require
TEXTURE_EXTERNAL_OES."

       return GL_TEXTURE_EXTERNAL_OES;
     case gfx::IO_SURFACE_BUFFER:
       // IOSurface backed images require GL_TEXTURE_RECTANGLE_ARB.
       return GL_TEXTURE_RECTANGLE_ARB;
     default:
       return GL_TEXTURE_2D;
   }
 }
 
 BrowserGpuChannelHostFactory::BrowserGpuChannelHostFactory()
@@ skipping 276 matching lines @@
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   CreateGpuMemoryBufferCallbackMap::iterator iter =
       create_gpu_memory_buffer_requests_.find(request_id);
   DCHECK(iter != create_gpu_memory_buffer_requests_.end());
   iter->second.Run(handle);
   create_gpu_memory_buffer_requests_.erase(iter);
 }
 
 }  // namespace content