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

gpu/command_buffer/service/feature_info.cc

Index: gpu/command_buffer/service/feature_info.cc
diff --git a/gpu/command_buffer/service/feature_info.cc b/gpu/command_buffer/service/feature_info.cc
index 58b1820159af29365e3ccb96b3d4d6eb122f26e6..d0cd0cd6f8085baef2252cf43ecc8f7c1930b3f5 100644
--- a/gpu/command_buffer/service/feature_info.cc
+++ b/gpu/command_buffer/service/feature_info.cc
@@ -1281,6 +1281,16 @@ void FeatureInfo::InitializeFeatures() {
   }
   UMA_HISTOGRAM_BOOLEAN("GPU.TextureRG", feature_flags_.ext_texture_rg);
 
+  if (enable_texture_storage && enable_texture_float &&
+      feature_flags_.ext_texture_rg) {
+    validators_.texture_internal_format.AddValue(GL_R32F_EXT);
+  }
+
+  if (enable_texture_storage && enable_texture_half_float &&
+      feature_flags_.ext_texture_rg) {
+    validators_.texture_internal_format.AddValue(GL_R16F_EXT);
+  }

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

Instead of adding these values in separate if-statements, could you move the
block above (regarding  feature_flags_.ext_texture_rg) above the block higher
starting with "if (enable_texture_storage)"?

That way these could just be folded into the existing tests during the enabling
of the texture_storage extension (where they ostensibly belong).

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

tOn 2016/09/30 23:01:29, Ken Russell wrote:
Done. For future reference (if submitting patch related to this), the code is
wrong - texture_internal_format_storage validator should be modified, not
texture_internal_format.

+
   bool has_opengl_dual_source_blending =
       gl_version_info_->IsAtLeastGL(3, 3) ||
       (gl_version_info_->IsAtLeastGL(3, 2) &&