Use high precision texture format during srgb conversion emulation for blitFramebuffer

gpu/command_buffer/service/gles2_cmd_srgb_converter.cc

Index: gpu/command_buffer/service/gles2_cmd_srgb_converter.cc
diff --git a/gpu/command_buffer/service/gles2_cmd_srgb_converter.cc b/gpu/command_buffer/service/gles2_cmd_srgb_converter.cc
index 5ffbb47de31545874f7b9c2a2b5ac981c4a40b7d..1aa3fd802310f3909dac9e9d614a673310aad566 100644
--- a/gpu/command_buffer/service/gles2_cmd_srgb_converter.cc
+++ b/gpu/command_buffer/service/gles2_cmd_srgb_converter.cc
@@ -198,6 +198,11 @@ void SRGBConverter::Blit(
   // If we need to blit from linear to srgb or vice versa, some steps will be
   // skipped.
   DCHECK(srgb_converter_initialized_);
+  // Use RGBA32F as the temp texture's internalformat to prevent precision
+  // loss during conversion. But it is not color-renderable and
+  // texture-filterable in ES context.
+  DCHECK(feature_info_->context_type() != CONTEXT_TYPE_OPENGLES2 &&

[Zhenyao Mo, 2016/10/18 02:20:35]

This is incorrect. If context_type() is ES2 or ES3() in command buffer, it might
still be OK.  It's the underlying driver can't be ES.

featureInof_->gl_driver_info().is_es

[yunchao, 2016/10/18 07:42:52]

Done.

+      feature_info_->context_type() != CONTEXT_TYPE_OPENGLES3);
 
   // Set the states
   glActiveTexture(GL_TEXTURE0);
@@ -239,7 +244,7 @@ void SRGBConverter::Blit(
     glBindBuffer(GL_PIXEL_UNPACK_BUFFER, 0);
 
     glBindTexture(GL_TEXTURE_2D, srgb_converter_textures_[1]);
-    glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA,
+    glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA32F,
                  c.width(), c.height(),
                  0, GL_RGBA, GL_UNSIGNED_BYTE, nullptr);
     glBindFramebufferEXT(GL_FRAMEBUFFER, srgb_decoder_fbo_);
@@ -272,7 +277,7 @@ void SRGBConverter::Blit(
     height_draw = dstY1 > dstY0 ? dstY1 - dstY0 : dstY0 - dstY1;
     glBindBuffer(GL_PIXEL_UNPACK_BUFFER, 0);
     glTexImage2D(
-        GL_TEXTURE_2D, 0, decode ? GL_RGBA : src_framebuffer_internal_format,
+        GL_TEXTURE_2D, 0, decode ? GL_RGBA32F : src_framebuffer_internal_format,
         width_draw, height_draw, 0,
         decode ? GL_RGBA : src_framebuffer_format,
         decode ? GL_UNSIGNED_BYTE : src_framebuffer_type,