Reland of Extend CopyTextureCHROMIUM to more ES 3.0 texture formats.

gpu/command_buffer/service/feature_info.h

 // 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.
 
 #ifndef GPU_COMMAND_BUFFER_SERVICE_FEATURE_INFO_H_
 #define GPU_COMMAND_BUFFER_SERVICE_FEATURE_INFO_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 151 matching lines @@
   void EnableOESTextureHalfFloatLinear();
 
   bool ext_color_buffer_float_available() const {
     return ext_color_buffer_float_available_;
   }
 
   bool oes_texture_float_linear_available() const {
     return oes_texture_float_linear_available_;
   }
 
+  bool texture_format_bgra8888_available() const {
+    return texture_format_bgra8888_available_;
+  }
+
+  bool ext_srgb_available() const { return ext_srgb_available_; }
+
  private:
   friend class base::RefCounted<FeatureInfo>;
   friend class BufferManagerClientSideArraysTest;
 
   ~FeatureInfo();
 
   void AddExtensionString(const char* s);
   void InitializeBasicState(const base::CommandLine* command_line);
   void InitializeFeatures();
 
   Validators validators_;
 
   DisallowedFeatures disallowed_features_;
 
   ContextType context_type_;
 
   // The extensions string returned by glGetString(GL_EXTENSIONS);
   std::string extensions_;
 
   // Flags for some features
   FeatureFlags feature_flags_;
 
   // Flags for Workarounds.
   const GpuDriverBugWorkarounds workarounds_;
 
   bool ext_color_buffer_float_available_;
   bool oes_texture_float_linear_available_;
   bool oes_texture_half_float_linear_available_;
+  bool texture_format_bgra8888_available_;

[Zhenyao Mo, 2016/11/19 00:42:32]

This is the wrong place for these two flags.

The above three indicate that they are available but *NOT* enabled (so WebGL can
dynamically enable them)

What you need is two flags to be part of feature_flags_ (and I think you already
have one, so only need to add the ext_srgb

[qiankun, 2016/11/21 16:01:50]

Yes. Thanks for pointing this. Follow your suggestions.

+  bool ext_srgb_available_;
 
   bool disable_shader_translator_;
   std::unique_ptr<gl::GLVersionInfo> gl_version_info_;
 
   DISALLOW_COPY_AND_ASSIGN(FeatureInfo);
 };
 
 }  // namespace gles2
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_FEATURE_INFO_H_