Set correct internalformat info for TexStorageEXT for WebGL1 context

gpu/command_buffer/service/gles2_cmd_decoder.cc

 // 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.
 
 #include "gpu/command_buffer/service/gles2_cmd_decoder.h"
 
 #include <limits.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <stdio.h>
@@ skipping 16729 matching lines @@
   }
   if (workarounds().reset_base_mipmap_level_before_texstorage &&
       texture->base_level() > 0)
     glTexParameteri(target, GL_TEXTURE_BASE_LEVEL, texture->base_level());
 
   {
     GLsizei level_width = width;
     GLsizei level_height = height;
     GLsizei level_depth = depth;
 
-    GLenum adjusted_internal_format =
-        feature_info_->context_type() == CONTEXT_TYPE_OPENGLES2 ?
-        format : internal_format;
+    const bool is_es2 =
+        feature_info_->context_type() == CONTEXT_TYPE_OPENGLES2 ||
+        feature_info_->context_type() == CONTEXT_TYPE_WEBGL1;

[qiankun, 2016/12/02 10:10:38]

You can use feature_info_->IsWebGL1OrES2Context() here.

[dshwang, 2016/12/02 17:54:20]

Good point! Done.

+    GLenum adjusted_internal_format = is_es2 ? format : internal_format;
     for (int ii = 0; ii < levels; ++ii) {
       if (target == GL_TEXTURE_CUBE_MAP) {
         for (int jj = 0; jj < 6; ++jj) {
           GLenum face = GL_TEXTURE_CUBE_MAP_POSITIVE_X + jj;
           texture_manager()->SetLevelInfo(
               texture_ref, face, ii, adjusted_internal_format, level_width,
               level_height, 1, 0, format, type, gfx::Rect());
         }
       } else {
         texture_manager()->SetLevelInfo(
@@ skipping 2109 matching lines @@
 }
 
 // Include the auto-generated part of this file. We split this because it means
 // we can easily edit the non-auto generated parts right here in this file
 // instead of having to edit some template or the code generator.
 #include "base/macros.h"
 #include "gpu/command_buffer/service/gles2_cmd_decoder_autogen.h"
 
 }  // namespace gles2
 }  // namespace gpu