Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
NOTRY=true
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2733953007
Cr-Commit-Position: refs/heads/master@{#455634}
Committed: https://chromium.googlesource.com/chromium/src/+/62643d9574be80279c2d94868216a8cf9706c11c

[Zhenyao Mo, 2017-03-08 18:50:20]

Description was changed from

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
TBR=kbr@chromium.org
==========

to

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Zhenyao Mo, 2017-03-08 18:50:46]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2017-03-08 18:51:21]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-08 21:03:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-08 21:03:58]

Failed to apply patch for gpu/config/gpu_driver_bug_list_json.cc:
While running git apply --index -p1;
  error: patch failed: gpu/config/gpu_driver_bug_list_json.cc:19
  error: gpu/config/gpu_driver_bug_list_json.cc: patch does not apply

Patch:       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
850b3eb896905f76cd4fee1819d64fa0c489b05b..afe65de0c24604482b1f75d80ceb1f91bc6aab9c
100644
--- a/gpu/config/gpu_driver_bug_list_json.cc
+++ b/gpu/config/gpu_driver_bug_list_json.cc
@@ -19,7 +19,7 @@ const char kGpuDriverBugListJson[] = LONG_STRING_CONST(
 {
   "name": "gpu driver bug list",
   // Please update the version number whenever you change this file.
-  "version": "9.32",
+  "version": "9.33",
   "entries": [
     {
       "id": 1,
@@ -1899,7 +1899,11 @@ LONG_STRING_CONST(
       "description": "glTexStorage* are buggy when base mipmap level is not 0",
       "cr_bugs": [640506],
       "os": {
-        "type": "macosx"
+        "type": "macosx",
+        "version": {
+          "op": "<",
+          "value": "10.12.4"
+        }
       },
       "features": [
         "reset_base_mipmap_level_before_texstorage"

[Zhenyao Mo, 2017-03-08 22:35:17]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2017-03-08 22:36:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-08 23:38:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-08 23:38:23]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Zhenyao Mo, 2017-03-08 23:44:43]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2017-03-08 23:45:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-09 00:41:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-09 00:41:54]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ken Russell (switch to Gerrit), 2017-03-09 00:49:38]

LGTM

There must surely already be a bug filed about the unrelated
linux_chromium_chromeos_ozone_rel_ng failures, so please comment on it and point
to this CL as one of surely many affected. Thanks.

[Zhenyao Mo, 2017-03-09 01:42:31]

On 2017/03/09 00:49:38, Ken Russell OOO-till-Mar-13 wrote:
> LGTM
> 
> There must surely already be a bug filed about the unrelated
> linux_chromium_chromeos_ozone_rel_ng failures, so please comment on it and
point
> to this CL as one of surely many affected. Thanks.

Not really.  I filed crbug.com/699832.  Can you help finding an owner?

[Zhenyao Mo, 2017-03-09 01:47:02]

Description was changed from

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
NOTRY=true
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Zhenyao Mo, 2017-03-09 01:47:27]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2017-03-09 01:48:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-09 01:52:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489024047166340,
"parent_rev": "1372de15eab0d9365f7b3b0d006eb1a52d47edf6", "commit_rev":
"62643d9574be80279c2d94868216a8cf9706c11c"}

[commit-bot: I haz the power, 2017-03-09 01:53:40]

Description was changed from

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
NOTRY=true
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Limit reset_base_mipmap_level_before_texstorage driver bug workaround.

To OSX older than 10.12.4.

BUG=640506
TEST=
NOTRY=true
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2733953007
Cr-Commit-Position: refs/heads/master@{#455634}
Committed:
https://chromium.googlesource.com/chromium/src/+/62643d9574be80279c2d94868216...
==========

[commit-bot: I haz the power, 2017-03-09 01:53:41]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/62643d9574be80279c2d94868216...