Expand the list of devices that require a GPU workaround.

Expand the list of devices that require a GPU workaround.

Old AMD devices are known to require a workaround for colormasks on multisample
renderbuffers. This CL expands the list of devices known to need this
workaround.

BUG=648466
CQ_INCLUDE_TRYBOTS=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

Committed: https://crrev.com/e9a527e058e65fcbfc37b817a5ef7e6693fbb209
Cr-Commit-Position: refs/heads/master@{#420756}

[erikchen, 2016-09-22 20:28:53]

Description was changed from

==========
Expand the list of devices that require a GPU workaround.

Old AMD devices are known to require a workaround for colormasks on multisample
renderbuffers. This CL expands the list of devices known to need this
workaround.

BUG=648466
==========

to

==========
Expand the list of devices that require a GPU workaround.

Old AMD devices are known to require a workaround for colormasks on multisample
renderbuffers. This CL expands the list of devices known to need this
workaround.

BUG=648466
CQ_INCLUDE_TRYBOTS=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
==========

[erikchen, 2016-09-22 20:29:09]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[erikchen, 2016-09-22 20:29:18]

erikchen@chromium.org changed reviewers:
+ kbr@chromium.org

[erikchen, 2016-09-22 20:29:19]

kbr: Please review.

[commit-bot: I haz the power, 2016-09-22 20:29:41]

Dry run: 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, 2016-09-22 22:49:21]

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

[commit-bot: I haz the power, 2016-09-22 22:49:22]

Dry run: This issue passed the CQ dry run.

[Ken Russell (switch to Gerrit), 2016-09-23 00:28:10]

Awesome, thank you. LGTM

Once this lands could you propose a merge to M54? I'd forgotten that this
control existed. Unfortunately, the merge will have to be done by hand because
of the need to update the version number on the branch.

https://codereview.chromium.org/2363833002/diff/1/gpu/config/gpu_driver_bug_l...
File gpu/config/gpu_driver_bug_list_json.cc (right):

https://codereview.chromium.org/2363833002/diff/1/gpu/config/gpu_driver_bug_l...
gpu/config/gpu_driver_bug_list_json.cc:22: "version": "8.99",
Please update the version.

[erikchen, 2016-09-23 00:58:40]

https://codereview.chromium.org/2363833002/diff/1/gpu/config/gpu_driver_bug_l...
File gpu/config/gpu_driver_bug_list_json.cc (right):

https://codereview.chromium.org/2363833002/diff/1/gpu/config/gpu_driver_bug_l...
gpu/config/gpu_driver_bug_list_json.cc:22: "version": "8.99",
On 2016/09/23 00:28:10, Ken Russell wrote:
> Please update the version.

Done.

[erikchen, 2016-09-23 00:58:45]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-09-23 00:58:45]

The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org
Link to the patchset: https://codereview.chromium.org/2363833002/#ps20001
(title: "Comments from kbr.")

[commit-bot: I haz the power, 2016-09-23 00:59:13]

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, 2016-09-23 03:07:28]

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

[commit-bot: I haz the power, 2016-09-23 03:07:28]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[erikchen, 2016-09-23 15:50:30]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-09-23 15:50:41]

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, 2016-09-23 15:57:30]

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

[commit-bot: I haz the power, 2016-09-23 15:57:31]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Ken Russell (switch to Gerrit), 2016-09-23 17:31:36]

On 2016/09/23 15:57:31, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

Gee, there have been a lot of rapid updates to this file recently. Sorry. Could
you rebase one more time?

[erikchen, 2016-09-23 17:37:11]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-09-23 17:37:12]

The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org
Link to the patchset: https://codereview.chromium.org/2363833002/#ps40001
(title: "REbase.")

[commit-bot: I haz the power, 2016-09-23 17:37:41]

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, 2016-09-23 17:57:37]

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

[commit-bot: I haz the power, 2016-09-23 17:57:38]

Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[erikchen, 2016-09-23 18:10:37]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-09-23 18:11:08]

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, 2016-09-23 19:21:01]

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

[commit-bot: I haz the power, 2016-09-23 19:21:02]

Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[Ken Russell (switch to Gerrit), 2016-09-23 19:35:35]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2016-09-23 19:36:28]

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

[Ken Russell (switch to Gerrit), 2016-09-23 19:37:00]

On 2016/09/23 19:21:02, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

Sorry about this flakiness. Reported it in http://crbug.com/626862 .

[commit-bot: I haz the power, 2016-09-23 22:46:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-23 22:50:53]

Description was changed from

==========
Expand the list of devices that require a GPU workaround.

Old AMD devices are known to require a workaround for colormasks on multisample
renderbuffers. This CL expands the list of devices known to need this
workaround.

BUG=648466
CQ_INCLUDE_TRYBOTS=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

==========
Expand the list of devices that require a GPU workaround.

Old AMD devices are known to require a workaround for colormasks on multisample
renderbuffers. This CL expands the list of devices known to need this
workaround.

BUG=648466
CQ_INCLUDE_TRYBOTS=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

Committed: https://crrev.com/e9a527e058e65fcbfc37b817a5ef7e6693fbb209
Cr-Commit-Position: refs/heads/master@{#420756}
==========

[commit-bot: I haz the power, 2016-09-23 22:50:54]

Patchset 3 (id:??) landed as
https://crrev.com/e9a527e058e65fcbfc37b817a5ef7e6693fbb209
Cr-Commit-Position: refs/heads/master@{#420756}