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

Issue 2254393002: Reorganize WebGL conformance expectations. (Closed)

Created:
4 years, 4 months ago by Ken Russell (switch to Gerrit)
Modified:
4 years, 4 months ago
Reviewers:
Zhenyao Mo, cblume
CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org, ynovikov
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reorganize WebGL conformance expectations. A lot of the new Android test expectations went into the "WebGL extension" part of the expectations file, leading to two sections for multiple Android devices. Carefully merge these together. Also collapse some texture-related expectations using wildcards to improve maintainability and save space. BUG=490984 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 TBR=zmo@chromium.org,cblume@chromium.org Committed: https://crrev.com/4b445375f64a95a41234aba3a84a44f9cc2b3f57 Cr-Commit-Position: refs/heads/master@{#413590}

Patch Set 1 #

Patch Set 2 : Redid from scratch, merging similar entries better. #

Patch Set 3 : Minor cleanups. #

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

Messages

Total messages: 17 (10 generated)
Ken Russell (switch to Gerrit)
FYI. CQ'ing TBR'd.
4 years, 4 months ago (2016-08-19 02:02:12 UTC) #3
Ken Russell (switch to Gerrit)
CC'ing Yuly as well. Added android_optional_gpu_tests_rel manually.
4 years, 4 months ago (2016-08-19 02:03:28 UTC) #6
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/2254393002/1
4 years, 4 months ago (2016-08-19 02:03:56 UTC) #8
commit-bot: I haz the power
Exceeded global retry quota
4 years, 4 months ago (2016-08-19 03:45:40 UTC) #10
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/2254393002/40001
4 years, 4 months ago (2016-08-22 22:30:17 UTC) #13
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years, 4 months ago (2016-08-23 00:07:59 UTC) #15
commit-bot: I haz the power
4 years, 4 months ago (2016-08-23 00:10:52 UTC) #17
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/4b445375f64a95a41234aba3a84a44f9cc2b3f57
Cr-Commit-Position: refs/heads/master@{#413590}

Powered by Google App Engine
This is Rietveld 408576698