Reland of Extend CopyTextureCHROMIUM to more ES 3.0 texture formats.

gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h

Index: gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h
diff --git a/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h b/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h
index dd705323badecb1833e85a8da0353fb8b1dbd7f3..22f4a0af0dd600b75a1e7c110d3ca71275634229 100644
--- a/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h
+++ b/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h
@@ -18,6 +18,13 @@ namespace gles2 {
 
 class GLES2Decoder;
 
+enum SupportedCopyMethodByFormat {

[Ken Russell (switch to Gerrit), 2016/12/07 06:38:01]

Could we choose another name for this enum? Perhaps CopyTextureMethod?

[qiankun, 2016/12/10 00:12:40]

Done.

+  DIRECT_COPY,

[Ken Russell (switch to Gerrit), 2016/12/07 06:38:02]

Could you please add a one-line comment documenting each of these? Mentioning
that this uses CopyTex{Sub}Image2D to copy from the source to the destination,
for example?

[qiankun, 2016/12/10 00:12:40]

Done.

+  DIRECT_DRAW,

[Ken Russell (switch to Gerrit), 2016/12/07 06:38:02]

Could you mention that this draws from the source to the destination texture?

[qiankun, 2016/12/10 00:12:40]

Done.

+  DRAW_AND_COPY,

[Ken Russell (switch to Gerrit), 2016/12/07 06:38:01]

Could you mention that this draws to an intermediate texture, and the copies to
the destination texture?

[qiankun, 2016/12/10 00:12:40]

Done.

+  NONE_COPY

[Ken Russell (switch to Gerrit), 2016/12/07 06:38:02]

Perhaps NOT_COPYABLE?

[qiankun, 2016/12/10 00:12:40]

Done.

+};
+
 // This class encapsulates the resources required to implement the
 // GL_CHROMIUM_copy_texture extension.  The copy operation is performed
 // via glCopyTexImage2D() or a blit to a framebuffer object.
@@ -42,7 +49,8 @@ class GPU_EXPORT CopyTextureCHROMIUMResourceManager {
                      GLsizei height,
                      bool flip_y,
                      bool premultiply_alpha,
-                     bool unpremultiply_alpha);
+                     bool unpremultiply_alpha,
+                     SupportedCopyMethodByFormat method);
 
   void DoCopySubTexture(const gles2::GLES2Decoder* decoder,
                         GLenum source_target,
@@ -63,7 +71,8 @@ class GPU_EXPORT CopyTextureCHROMIUMResourceManager {
                         GLsizei source_height,
                         bool flip_y,
                         bool premultiply_alpha,
-                        bool unpremultiply_alpha);
+                        bool unpremultiply_alpha,
+                        SupportedCopyMethodByFormat method);
 
   void DoCopySubTextureWithTransform(const gles2::GLES2Decoder* decoder,
                                      GLenum source_target,
@@ -94,8 +103,10 @@ class GPU_EXPORT CopyTextureCHROMIUMResourceManager {
   void DoCopyTextureWithTransform(const gles2::GLES2Decoder* decoder,
                                   GLenum source_target,
                                   GLuint source_id,
+                                  GLenum source_format,
                                   GLenum dest_target,
                                   GLuint dest_id,
+                                  GLenum dest_format,
                                   GLsizei width,
                                   GLsizei height,
                                   bool flip_y,
@@ -136,8 +147,10 @@ class GPU_EXPORT CopyTextureCHROMIUMResourceManager {
   void DoCopyTextureInternal(const gles2::GLES2Decoder* decoder,
                              GLenum source_target,
                              GLuint source_id,
+                             GLenum source_format,
                              GLenum dest_target,
                              GLuint dest_id,
+                             GLenum dest_format,
                              GLint xoffset,
                              GLint yoffset,
                              GLint x,