16 bpp video stream capture, render and WebGL usage - Realsense R200 & SR300 support.

gpu/command_buffer/service/texture_manager.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 "gpu/command_buffer/service/texture_manager.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 148 matching lines @@
         {GL_RED, GL_RED, GL_FLOAT},
         {GL_RG, GL_RG, GL_FLOAT},
         {GL_RED, GL_RED, GL_HALF_FLOAT_OES},
         {GL_RG, GL_RG, GL_HALF_FLOAT_OES},
 
         // ES3.
         {GL_R8, GL_RED, GL_UNSIGNED_BYTE},
         {GL_R8_SNORM, GL_RED, GL_BYTE},
         {GL_R16F, GL_RED, GL_HALF_FLOAT},
         {GL_R16F, GL_RED, GL_FLOAT},
+        {GL_R16F, GL_R16F, GL_HALF_FLOAT},

[Ken Russell (switch to Gerrit), 2016/09/30 23:01:30]

These entries are flat-out wrong. Please remove them.

         {GL_R32F, GL_RED, GL_FLOAT},
+        {GL_R32F, GL_R32F, GL_FLOAT},
         {GL_R8UI, GL_RED_INTEGER, GL_UNSIGNED_BYTE},
         {GL_R8I, GL_RED_INTEGER, GL_BYTE},
         {GL_R16UI, GL_RED_INTEGER, GL_UNSIGNED_SHORT},
+        {GL_R16UI, GL_R16UI, GL_UNSIGNED_SHORT},
         {GL_R16I, GL_RED_INTEGER, GL_SHORT},
         {GL_R32UI, GL_RED_INTEGER, GL_UNSIGNED_INT},
         {GL_R32I, GL_RED_INTEGER, GL_INT},
         {GL_RG8, GL_RG, GL_UNSIGNED_BYTE},
         {GL_RG8_SNORM, GL_RG, GL_BYTE},
         {GL_RG16F, GL_RG, GL_HALF_FLOAT},
         {GL_RG16F, GL_RG, GL_FLOAT},
         {GL_RG32F, GL_RG, GL_FLOAT},
         {GL_RG8UI, GL_RG_INTEGER, GL_UNSIGNED_BYTE},
         {GL_RG8I, GL_RG_INTEGER, GL_BYTE},
@@ skipping 2715 matching lines @@
       return GL_RGB;
     if (format == GL_SRGB_ALPHA_EXT)
       return GL_RGBA;
   }
   if (feature_info->gl_version_info().is_desktop_core_profile) {
     const Texture::CompatibilitySwizzle* swizzle =
         GetCompatibilitySwizzle(format);
     if (swizzle)
       return swizzle->dest_format;
   }
+  switch (format) {

[Ken Russell (switch to Gerrit), 2016/09/30 23:01:30]

This mapping back of illegal formats to correct ones is not OK. Please figure
out some other solution that doesn't involve making this change.

+    case GL_R32F:
+    case GL_R16F:
+    case GL_R8:
+      return GL_RED;
+    case GL_RG8:
+      return GL_RG;
+    case GL_R16UI:
+      return GL_RED_INTEGER;
+    default:
+      break;
+  }
   return format;
 }
 
 void TextureManager::DoTexImage(
     DecoderTextureState* texture_state,
     ContextState* state,
     DecoderFramebufferState* framebuffer_state,
     const char* function_name,
     TextureRef* texture_ref,
     const DoTexImageArguments& args) {
@@ skipping 461 matching lines @@
 uint32_t TextureManager::GetServiceIdGeneration() const {
   return current_service_id_generation_;
 }
 
 void TextureManager::IncrementServiceIdGeneration() {
   current_service_id_generation_++;
 }
 
 }  // namespace gles2
 }  // namespace gpu