Add new gpu driver bug workaround DISABLE_TRANSPARENT_VISUALS

gpu/config/gpu_driver_bug_workaround_type.h

 // Copyright (c) 2013 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_CONFIG_GPU_DRIVER_BUG_WORKAROUND_TYPE_H_
 #define GPU_CONFIG_GPU_DRIVER_BUG_WORKAROUND_TYPE_H_
 
 #include <string>
 
 #include "gpu/gpu_export.h"
@@ skipping 47 matching lines @@
   GPU_OP(DISABLE_POST_SUB_BUFFERS_FOR_ONSCREEN_SURFACES,     \
          disable_post_sub_buffers_for_onscreen_surfaces)     \
   GPU_OP(DISABLE_PROGRAM_CACHE,                              \
          disable_program_cache)                              \
   GPU_OP(DISABLE_TEXTURE_CUBE_MAP_SEAMLESS,                  \
          disable_texture_cube_map_seamless)                  \
   GPU_OP(DISABLE_TEXTURE_STORAGE,                            \
          disable_texture_storage)                            \
   GPU_OP(DISABLE_TIMESTAMP_QUERIES,                          \
          disable_timestamp_queries)                          \
+  GPU_OP(DISABLE_TRANSPARENT_VISUALS,                        \
+         disable_transparent_visuals)                        \

[Ken Russell (switch to Gerrit), 2016/07/07 22:23:40]

As Corentin pointed out elsewhere, changing the sense of this driver bug
workaround to "use_transparent_visuals" would make it more clear on which
platforms it's enabled. The "exceptions" field wouldn't be needed in
gpu_driver_bug_list_json.cc.

[Julien Isorce Samsung, 2016/07/08 09:52:53]

My understanding was that Corentin wanted to have it changed only in the
function, so I did it in Patch Set 6:
https://codereview.chromium.org/2124633002/diff2/80001:100001/ui/base/x/x11_u...

For the workaround name I would like to keep the current name since it is
coherent with other workaround names. For example, using transparent visuals is
the expected working usage, but if there is a bug then it disables it.

[Ken Russell (switch to Gerrit), 2016/07/12 22:46:26]

OK. I'm not 100% convinced that the future direction is to always use
transparent visuals, but either way is fine. Note that there is precedence in
this file for the "use_" naming convention.

   GPU_OP(DISABLE_WEBGL_MULTISAMPLING_COLOR_MASK_USAGE,       \
          disable_webgl_multisampling_color_mask_usage)       \
   GPU_OP(DISABLE_WEBGL_RGB_MULTISAMPLING_USAGE,              \
          disable_webgl_rgb_multisampling_usage)              \
   GPU_OP(ETC1_POWER_OF_TWO_ONLY,                             \
          etc1_power_of_two_only)                             \
   GPU_OP(EXIT_ON_CONTEXT_LOST,                               \
          exit_on_context_lost)                               \
   GPU_OP(FORCE_CUBE_COMPLETE,                                \
          force_cube_complete)                                \
@@ skipping 98 matching lines @@
 #undef GPU_OP
   NUMBER_OF_GPU_DRIVER_BUG_WORKAROUND_TYPES
 };
 
 GPU_EXPORT std::string GpuDriverBugWorkaroundTypeToString(
     GpuDriverBugWorkaroundType type);
 
 }  // namespace gpu
 
 #endif  // GPU_CONFIG_GPU_DRIVER_BUG_WORKAROUND_TYPE_H_