Work around CopyTexImage2D issue on Intel Mac OSX 10.12

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 13296 matching lines @@
             GetBoundReadFramebufferServiceId(),
             GetBoundReadFramebufferInternalFormat());
       } else {
         glCopyTexSubImage2D(target, level, destX, destY, copyX, copyY,
                             copyWidth, copyHeight);
       }
     }
   } else {
     GLenum final_internal_format = TextureManager::AdjustTexInternalFormat(
         feature_info_.get(), internal_format);
+    if (workarounds().init_two_cube_map_levels_before_copyteximage &&
+        texture->target() == GL_TEXTURE_CUBE_MAP &&
+        target != GL_TEXTURE_CUBE_MAP_POSITIVE_X) {
+      for (int i = 0; i < 2; ++i) {
+        TextureManager::DoTexImageArguments args = {
+          target, i, final_internal_format, 1, 1, 1, border,
+          format, type, nullptr, 1, 0,
+          TextureManager::DoTexImageArguments::kTexImage2D };
+        texture_manager()->WorkaroundCopyTexImageCubeMap(&texture_state_,
+            &state_, &framebuffer_state_, texture_ref, func_name, args);
+      }
+    }
 
     // The service id and target of the texture attached to READ_FRAMEBUFFER.
     GLuint source_texture_service_id = 0;
     GLenum source_texture_target = 0;
     uint32_t channels_exist = GLES2Util::GetChannelsForFormat(read_format);
     bool use_workaround = NeedsCopyTextureImageWorkaround(
         final_internal_format, channels_exist, &source_texture_service_id,
         &source_texture_target);
     if (requires_luma_blit) {
       copy_tex_image_blit_->DoCopyTexImage2DToLUMACompatibilityTexture(

[Zhenyao Mo, 2016/09/30 16:24:20]

Here is the luminance/alpha format emulation.  Of course the
WorkaroundCopyTexImageCubeMap() will fail because it calls TexImage*() functions
if the formats are luminance/alpha, which isn't supported on Mac Core profile.

[qiankun, 2016/09/30 17:44:42]

Won't line 13315 do LUMINANCE/ALPHA mapping?

[Zhenyao Mo, 2016/09/30 18:02:11]

That's not enough.  The format will still cause an GL error.

[qiankun, 2016/10/01 01:25:06]

Mac OSX should be use core profile now. But I didn't see the GL errors with this
CL for luminance cases. Do you know what situation will trigger the errors?

           this, texture->service_id(), texture->target(), target, format,
           type, level, internal_format, copyX, copyY, copyWidth, copyHeight,
           GetBoundReadFramebufferServiceId(),
           GetBoundReadFramebufferInternalFormat());
     } else if (use_workaround) {
       GLenum dest_texture_target = target;
       GLenum framebuffer_target = features().chromium_framebuffer_multisample
                                       ? GL_READ_FRAMEBUFFER_EXT
                                       : GL_FRAMEBUFFER;
 
@@ skipping 25 matching lines @@
       glCopyTexImage2D(dest_texture_target, level, final_internal_format, 0, 0,
                        copyWidth, copyHeight, 0);
 
       // Rebind source texture.
       glFramebufferTexture2DEXT(framebuffer_target, GL_COLOR_ATTACHMENT0,
                                 source_texture_target,
                                 source_texture_service_id, 0);
 
       glDeleteTextures(1, &temp_texture);
     } else {
-      if (workarounds().do_teximage_before_copyteximage_to_cube_map &&
+      if (workarounds().init_one_cube_map_level_before_copyteximage &&
           texture->target() == GL_TEXTURE_CUBE_MAP &&
           target != GL_TEXTURE_CUBE_MAP_POSITIVE_X) {
         TextureManager::DoTexImageArguments args = {
           target, level, final_internal_format, width, height, 1, border,
           format, type, nullptr, pixels_size, 0,
           TextureManager::DoTexImageArguments::kTexImage2D };
         texture_manager()->WorkaroundCopyTexImageCubeMap(&texture_state_,
             &state_, &framebuffer_state_, texture_ref, func_name, args);
       }
       glCopyTexImage2D(target, level, final_internal_format, copyX, copyY,
@@ skipping 4700 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