Update GL_IMPLEMENTATION_COLOR_READ_FORMAT for BGRA

content/common/gpu/client/gl_helper_readback_support.cc

 // Copyright 2014 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.
 
 #include "content/common/gpu/client/gl_helper_readback_support.h"
 #include "base/logging.h"
 #include "gpu/GLES2/gl2extchromium.h"
 #include "third_party/skia/include/core/SkImageInfo.h"
 
 namespace content {
@@ skipping 86 matching lines @@
   struct FormatCacheEntry entry = { format, type, *format_out, *type_out };
   format_cache_.push_back(entry);
 }
 
 bool GLHelperReadbackSupport::SupportsFormat(GLenum format, GLenum type) {
   // GLES2.0 Specification says this pairing is always supported
   // with additional format from GL_IMPLEMENTATION_COLOR_READ_FORMAT/TYPE
   if (format == GL_RGBA && type == GL_UNSIGNED_BYTE)
     return true;
 
-  if (format == GL_BGRA_EXT && type == GL_UNSIGNED_BYTE) {

[ericrk, 2016/03/02 17:40:30]

The presence of GL_EXT_read_format_bgra doesn't guarantee that BGRA readback is
supported, it just indicates that it may be returned from
GL_IMPLEMENTATION_COLOR_READ_FORMAT. We should always be calling that function
in all cases.

We used to avoid doing so to work around an issue where the driver would never
report BGRA from GL_IMPLEMENTATION_COLOR_READ_FORMAT, but that issue is resolved
with this CL.

-    const GLubyte* tmp = gl_->GetString(GL_EXTENSIONS);
-    if (tmp) {
-      std::string extensions =
-          " " + std::string(reinterpret_cast<const char*>(tmp)) + " ";
-      if (extensions.find(" GL_EXT_read_format_bgra ") != std::string::npos) {
-        return true;
-      }
-    }
-  }
-
   bool supports_format = false;
   GLenum ext_format = 0, ext_type = 0;
   GetAdditionalFormat(format, type, &ext_format, &ext_type);
   if ((ext_format == format) && (ext_type == type)) {
     supports_format = true;
   }
   return supports_format;
 }
 
 GLHelperReadbackSupport::FormatSupport
@@ skipping 46 matching lines @@
       return GLHelperReadbackSupport::NOT_SUPPORTED;
     default:
       NOTREACHED();
       break;
   }
 
   return GLHelperReadbackSupport::NOT_SUPPORTED;
 }
 
 }  // namespace content