Clean up outdated gpu driver bug workarounds for Mac 10.9

gpu/config/gpu_driver_bug_list_json.cc

 // 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.
 
 // Determines whether a certain driver bug exists in the current system.
 // The format of a valid gpu_driver_bug_list.json file is defined in
 // <gpu/config/gpu_control_list_format.txt>.
 // The supported "features" can be found in
 // <gpu/config/gpu_driver_bug_workaround_type.h>.
 
@@ skipping 34 matching lines @@
         "op": "beginwith",
         "value": "ARM"
       },
       "features": [
         "use_client_side_arrays_for_stream_buffers"
       ]
     },
     {
       "id": 3,
       "description": "glGenerateMipmap leaks vram without setting texture filters on some drivers",
+      "cr_bugs": [349137],

[Ken Russell (switch to Gerrit), 2014/03/07 23:34:13]

Is this really the right bug to reference, given that we'll only be skipping
this workaround on OS X >= 10.9, and applying it everywhere else? The same
comment applies to all the new exceptions.

[Zhenyao Mo, 2014/03/07 23:40:24]

It is a little bit off, but I can't find the original bug (if there is one) to
reference.

I plan to clean up the list on Windows and Linux also, at least this bug will
let people know when the workaround is removed.

+      "exceptions": [
+        {
+          "os": {
+            "type": "macosx",
+            "version": {
+              "op": ">=",
+              "value": "10.9"
+            }
+          }
+        }
+      ],
       "features": [
         "set_texture_filter_before_generating_mipmap"
       ]
     },
     {
       "id": 4,
       "description": "glReadPixels incorrectly sets alpha to 0 on some drivers from a drawing buffer without alpha channel",
+      "cr_bugs": [349137],
+      "exceptions": [
+        {
+          "os": {
+            "type": "macosx",
+            "version": {
+              "op": ">=",
+              "value": "10.9"
+            }
+          }
+        }
+      ],
       "features": [
         "clear_alpha_in_readpixels"
       ]
     },
     {
       "id": 5,
       "description": "Always call glUseProgram after a successful link to avoid a driver bug",
+      "cr_bugs": [349137],
       "vendor_id": "0x10de",
+      "exceptions": [
+        {
+          "os": {
+            "type": "macosx",
+            "version": {
+              "op": ">=",
+              "value": "10.9"
+            }
+          }
+        }
+      ],
       "features": [
         "use_current_program_after_successful_link"
       ]
     },
     {
       "id": 6,
       "description": "Restore scissor on FBO change with Qualcomm GPUs on older versions of Android",
       "cr_bugs": [165493, 222018],
       "os": {
         "type": "android",
@@ skipping 18 matching lines @@
         "type": "macosx"
       },
       "vendor_id": "0x10de",
       "features": [
         "needs_offscreen_buffer_workaround"
       ]
     },
     {
       "id": 8,
       "description": "A few built-in glsl functions on Mac behave incorrectly",
+      "cr_bugs": [349137],
       "os": {
-        "type": "macosx"
+        "type": "macosx",
+        "version": {
+          "op": "<",
+          "value": "10.9"
+        }
       },
       "vendor_id": "0x1002",
       "features": [
         "needs_glsl_built_in_function_emulation"
       ]
     },
     {
       "id": 9,
       "description": "AMD drivers get gl_PointCoord backward on OS X 10.8 or earlier",
       "cr_bugs": [256349],
@@ skipping 572 matching lines @@
       "gl_vendor": {
         "op": "beginwith",
         "value": "Qualcomm"
       },
       "features": [
         "disable_multisampling"
       ]
     },
     {
       "id": 54,
-      "cr_bugs": [124764],
+      "cr_bugs": [124764, 349137],
       "description": "Clear uniforms before first program use on all platforms",
+      "exceptions": [
+        {
+          "os": {
+            "type": "macosx",
+            "version": {
+              "op": ">=",
+              "value": "10.9"
+            }
+          }
+        }
+      ],
       "features": [
         "clear_uniforms_before_first_program_use"
       ]
     },
     {
       "id": 55,
       "cr_bugs": [333885],
       "description": "Mesa drivers in Linux handle varyings without static use incorrectly",
       "os": {
         "type": "linux"
@@ skipping 56 matching lines @@
         "type": "linux"
       },
       "vendor_id": "0x8086",
       "device_id": ["0x0152", "0x0156", "0x015a", "0x0162", "0x0166"],
       "features": [
         "disable_multisampling"
       ]
     },
     {
       "id": 60,
-      "description": "Multisampling is buggy on Mac with NVIDIA gpu prior to 10.8.3",
-      "cr_bugs": [137303],
+      "description": "Multisampling is buggy on Mac OS X prior to 10.8.3",
+      "cr_bugs": [137303, 162466],
       "os": {
         "type": "macosx",
         "version": {
           "op": "<",
           "value": "10.8.3"
         }
       },
-      "vendor_id": "0x10de",
-      "features": [
-        "disable_multisampling"
-      ]
-    },
-    {
-      "id": 61,
-      "description": "Multisampling is buggy on Mac with Intel gpu prior to 10.8.3",
-      "cr_bugs": [137303],
-      "os": {
-        "type": "macosx",
-        "version": {
-          "op": "<",
-          "value": "10.8.3"
-        }
-      },
-      "vendor_id": "0x8086",
-      "features": [
-        "disable_multisampling"
-      ]
-    },
-    {
-      "id": 62,
-      "description": "Multisampling is buggy on Mac with AMD gpu prior to 10.8.3",
-      "cr_bugs": [162466],
-      "os": {
-        "type": "macosx",
-        "version": {
-          "op": "<",
-          "value": "10.8.3"
-        }
-      },
-      "vendor_id": "0x1002",
       "features": [
         "disable_multisampling"
       ]
     },
     {
       "id": 63,
       "description": "Shaders with sampler arrays indexing for-loop indices cause the GLSL compiler to crash on OS X",
-      "cr_bugs": [348198],
+        "cr_bugs": [348198, 349137],
       "os": {
-        "type": "macosx"
+        "type": "macosx",
+        "version": {
+          "op": "<",
+          "value": "10.9"
+        }
       },
       "features": [
         "unroll_for_loop_with_sampler_array_index"
       ]
     },
     {
       "id": 64,
       "description": "Linux AMD drivers incorrectly return initial value of 1 for TEXTURE_MAX_ANISOTROPY",
       "cr_bugs": [348237],
       "os": {
         "type": "linux"
       },
       "vendor_id": "0x1002",
       "features": [
         "init_texture_max_anisotropy"
       ]
     }
   ]
 }
 
 );  // LONG_STRING_CONST macro
 
 }  // namespace gpu