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

Issue 2170153002: Suppress two GLES2 conformance failures on Mac NVIDIA. (Closed)

Created:
4 years, 5 months ago by Ken Russell (switch to Gerrit)
Modified:
4 years, 5 months ago
Reviewers:
Zhenyao Mo, piman
CC:
chromium-reviews, piman+watch_chromium.org, Corentin Wallez, Jamie Madill, Geoff Lang, eyaich1
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Suppress two GLES2 conformance failures on Mac NVIDIA. BUG=630452 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 R=zmo@chromium.org TBR=piman@chromium.org NOTRY=true Committed: https://crrev.com/73bf375290c8ac2a8ebb834319c515756a667ae9 Cr-Commit-Position: refs/heads/master@{#406997}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M gpu/gles2_conform_support/gles2_conform_test_expectations.txt View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 22 (10 generated)
Ken Russell (switch to Gerrit)
Mo: please review. Thanks. Others: FYI.
4 years, 5 months ago (2016-07-21 23:29:51 UTC) #3
Ken Russell (switch to Gerrit)
Note: tested locally on Linux. The conformance expectations still parse correctly there.
4 years, 5 months ago (2016-07-21 23:30:14 UTC) #4
Zhenyao Mo
lgtm
4 years, 5 months ago (2016-07-21 23:30:40 UTC) #5
Ken Russell (switch to Gerrit)
CQ'ing TBR'd and NOTRY to get the bots greener sooner.
4 years, 5 months ago (2016-07-21 23:31:46 UTC) #6
Ken Russell (switch to Gerrit)
On 2016/07/21 23:31:46, Ken Russell wrote: > CQ'ing TBR'd and NOTRY to get the bots ...
4 years, 5 months ago (2016-07-21 23:32:11 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/2170153002/1
4 years, 5 months ago (2016-07-21 23:33:28 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/222859)
4 years, 5 months ago (2016-07-21 23:40:43 UTC) #12
Ken Russell (switch to Gerrit)
TBR= to piman
4 years, 5 months ago (2016-07-21 23:44:10 UTC) #15
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/2170153002/1
4 years, 5 months ago (2016-07-21 23:44:43 UTC) #17
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-21 23:50:43 UTC) #19
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/73bf375290c8ac2a8ebb834319c515756a667ae9 Cr-Commit-Position: refs/heads/master@{#406997}
4 years, 5 months ago (2016-07-21 23:53:54 UTC) #21
piman
4 years, 5 months ago (2016-07-22 00:36:13 UTC) #22
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698