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

gpu/command_buffer/common/gles2_cmd_utils.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.
 
 // This file is here so other GLES2 related files can have a common set of
 // includes where appropriate.
 
 #include "gpu/command_buffer/common/gles2_cmd_utils.h"
 
 #include <GLES2/gl2.h>
@@ skipping 1073 matching lines @@
       return GL_SHORT;
     case GL_R32I:
     case GL_RG32I:
     case GL_RGBA32I:
       return GL_INT;
     case GL_R32F:
     case GL_RG32F:
     case GL_RGB32F:
     case GL_RGBA32F:
     case GL_R11F_G11F_B10F:
-      return GL_UNSIGNED_BYTE;
+      return GL_FLOAT;

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

I think this change is actually wrong. For these floating-point formats the
default supported ReadPixels type is GL_FLOAT. This function gives the
"alternate", or "implementation", read type, and the command buffer typically
chooses to make this GL_UNSIGNED_BYTE. Hopefully this change would have caused
some tests to fail (at least some in the WebGL conformance suite). If not then
more tests should be added.

[aleksandar.stojiljkovic, 2016/10/03 10:56:52]

Done. Wasn't important. GL_IMPLEMENTATION_COLOR_READ_XXXX behavior on OSX
(driver returns UNSIGNED_BYTE) differed from Windows&Linux (on those I believe
driver returned FLOAT).

     case GL_R16F:
     case GL_RG16F:
     case GL_RGB16F:
     case GL_RGBA16F:
       return GL_HALF_FLOAT;
     default:
       // Unsized internal format.
       return texture_type;
   }
 }
@@ skipping 771 matching lines @@
   }
 
   return true;
 }
 
 #include "gpu/command_buffer/common/gles2_cmd_utils_implementation_autogen.h"
 
 }  // namespace gles2
 }  // namespace gpu