Clean up outdated gpu driver bug workarounds for Mac 10.9

gpu/config/gpu_driver_bug_list_json.cc

Index: gpu/config/gpu_driver_bug_list_json.cc
diff --git a/gpu/config/gpu_driver_bug_list_json.cc b/gpu/config/gpu_driver_bug_list_json.cc
index b6e1441067d6ab8c640651e5c705b08f6d2724e5..f7ba5562619cf4b745aa8e2ef67df6a008021c48 100644
--- a/gpu/config/gpu_driver_bug_list_json.cc
+++ b/gpu/config/gpu_driver_bug_list_json.cc
@@ -52,6 +52,18 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     {
       "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"
       ]
@@ -59,6 +71,18 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     {
       "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"
       ]
@@ -66,7 +90,19 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     {
       "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"
       ]
@@ -105,8 +141,13 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     {
       "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": [
@@ -699,8 +740,19 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     },
     {
       "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"
       ]
@@ -777,8 +829,8 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
     },
     {
       "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": {
@@ -786,50 +838,21 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
           "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],
+      "id": 63,
+      "description": "Shaders with sampler arrays indexing for-loop indices cause the GLSL compiler to crash on OS X",
+	"cr_bugs": [348198, 349137],
       "os": {
         "type": "macosx",
         "version": {
           "op": "<",
-          "value": "10.8.3"
+          "value": "10.9"
         }
       },
-      "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],
-      "os": {
-        "type": "macosx"
-      },
       "features": [
         "unroll_for_loop_with_sampler_array_index"
       ]