Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(404)

Issue 2356773002: WebGL CTS: add back expectations for blitframebuffer srgb (Closed)

Created:
4 years, 3 months ago by Corentin Wallez
Modified:
4 years, 3 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam, piman+watch_chromium.org, ccameron, ynovikov
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

WebGL CTS: add back expectations for blitframebuffer srgb The expectations were eagerly removed because the test passed on Linux Intel, but it still doesn't pass on Linux AMD and Linux Intel ANGLE TBR=zmo@chromium.org BUG=634525 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;master.tryserver.chromium.android:android_optional_gpu_tests_rel NOTRY=true Committed: https://crrev.com/aa1fae069018c6a9f0410a8259ee73d9d5ab0e84 Cr-Commit-Position: refs/heads/master@{#419920}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M content/test/gpu/gpu_tests/webgl2_conformance_expectations.py View 2 chunks +4 lines, -0 lines 0 comments Download

Messages

Total messages: 24 (13 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2356773002/1
4 years, 3 months ago (2016-09-20 14:39:26 UTC) #5
Corentin Wallez
Chris, Yuly FYI as wranglers
4 years, 3 months ago (2016-09-20 14:39:44 UTC) #6
commit-bot: I haz the power
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_tests_rel/builds/3830)
4 years, 3 months ago (2016-09-20 18:38:48 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2356773002/1
4 years, 3 months ago (2016-09-20 19:13:38 UTC) #10
commit-bot: I haz the power
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_tests_rel/builds/3845)
4 years, 3 months ago (2016-09-21 00:06:00 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2356773002/1
4 years, 3 months ago (2016-09-21 00:10:35 UTC) #14
Ken Russell (switch to Gerrit)
On 2016/09/21 00:10:35, commit-bot: I haz the power wrote: > CQ is trying da patch. ...
4 years, 3 months ago (2016-09-21 00:31:42 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2356773002/1
4 years, 3 months ago (2016-09-21 00:33:05 UTC) #19
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-21 00:39:31 UTC) #21
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/aa1fae069018c6a9f0410a8259ee73d9d5ab0e84 Cr-Commit-Position: refs/heads/master@{#419920}
4 years, 3 months ago (2016-09-21 00:41:39 UTC) #23
Corentin Wallez
4 years, 3 months ago (2016-09-21 13:17:52 UTC) #24
Message was sent while issue was closed.
On 2016/09/21 at 00:41:39, commit-bot wrote:
> Patchset 1 (id:??) landed as
https://crrev.com/aa1fae069018c6a9f0410a8259ee73d9d5ab0e84
> Cr-Commit-Position: refs/heads/master@{#419920}

Thanks for pushing this Ken

Powered by Google App Engine
This is Rietveld 408576698