Relimit Skia to ES2 to address performance regression

Relimit Skia to ES2 to address performance regression

BUG=719572
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/2895153004
Cr-Commit-Position: refs/heads/master@{#473666}
Committed: https://chromium.googlesource.com/chromium/src/+/8d52fb787ae3f370ba1bf7be4b15ae01287879ac

[bsalomon, 2017-05-22 18:14:58]

Description was changed from

==========
Limit Skia to ES2 to address performance regression

BUG=719572
==========

to

==========
Limit Skia to ES2 to address performance regression

BUG=719572
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
==========

[bsalomon, 2017-05-22 18:17:03]

The CQ bit was checked by bsalomon@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 18:17:32]

Dry run: CQ is trying da patch.

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

[bsalomon, 2017-05-22 18:18:11]

Description was changed from

==========
Limit Skia to ES2 to address performance regression

BUG=719572
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

==========
Relimit Skia to ES2 to address performance regression

BUG=719572
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
==========

[commit-bot: I haz the power, 2017-05-22 19:23:32]

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

[commit-bot: I haz the power, 2017-05-22 19:23:33]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2017-05-22 19:42:27]

bsalomon@google.com changed reviewers:
+ zmo@chromium.org

[bsalomon, 2017-05-22 19:42:27]

Mo, turns out you were right to caution against performance regressions. After
my change we still were using ES2 for GPU raster but not in all other code paths
that use Ganesh. I want to understand why we had a regression but haven't
managed to repro so far. With the branch coming up I'd like to punt for right
now.

[Zhenyao Mo, 2017-05-22 19:44:22]

lgtm
Hope we can get to the bottom of this perf issue and move on soon

[bsalomon, 2017-05-22 19:52:31]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2017-05-22 19:53:09]

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-05-22 19:58:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495482751106220, "parent_rev":
"0adcad330b0c78ec30e4f95e527e8ceff5ca749d", "commit_rev":
"bd79821c3d30cf87db47c905169ebd99666f393f"}

[commit-bot: I haz the power, 2017-05-22 19:58:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495482751106220, "parent_rev":
"428678ac532ca3213c7eaceeeb0453d5fae8f7a0", "commit_rev":
"8d52fb787ae3f370ba1bf7be4b15ae01287879ac"}

[commit-bot: I haz the power, 2017-05-22 19:58:55]

Description was changed from

==========
Relimit Skia to ES2 to address performance regression

BUG=719572
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

==========
Relimit Skia to ES2 to address performance regression

BUG=719572
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/2895153004
Cr-Commit-Position: refs/heads/master@{#473666}
Committed:
https://chromium.googlesource.com/chromium/src/+/8d52fb787ae3f370ba1bf7be4b15...
==========

[commit-bot: I haz the power, 2017-05-22 19:58:56]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8d52fb787ae3f370ba1bf7be4b15...