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

Issue 2388803002: Remove failed entries and skip Android tests for gpu pixel expectations (Closed)

Created:
4 years, 2 months ago by xlai (Olivia)
Modified:
4 years, 2 months ago
CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove failed entries and skip Android tests for gpu pixel expectations The two unaccelerated 2d gpu integration tests are working fine in Win,Linux and Mac. So I'm removing their failed entries. Because software compositing is not supported on Android. These two tests should not be running on Android platforms. So I mark them as Skipped. TBR=kbr@chromium.org BUG=563852 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 Committed: https://crrev.com/5280614b9452793b73489cb103cd740ea24f638f Cr-Commit-Position: refs/heads/master@{#423184}

Patch Set 1 #

Patch Set 2 : Still mark android bots as failed #

Patch Set 3 : Skip android bots #

Patch Set 4 : rebase #

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

Messages

Total messages: 28 (15 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/2388803002/1
4 years, 2 months ago (2016-10-03 14:44:07 UTC) #3
commit-bot: I haz the power
Try jobs failed on following builders: android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/41086)
4 years, 2 months ago (2016-10-03 15:50:55 UTC) #5
xlai (Olivia)
4 years, 2 months ago (2016-10-04 18:36:27 UTC) #8
xlai (Olivia)
Changing title and description.
4 years, 2 months ago (2016-10-04 19:12:45 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/2388803002/40001
4 years, 2 months ago (2016-10-04 20:28:53 UTC) #13
commit-bot: I haz the power
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_android_rel_ng/builds/153927) win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED, ...
4 years, 2 months ago (2016-10-04 22:07:13 UTC) #15
Ken Russell (switch to Gerrit)
LGTM Let's try again. It would be good if you'd file bugs about the flaky ...
4 years, 2 months ago (2016-10-04 23:12:15 UTC) #16
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/2388803002/40001
4 years, 2 months ago (2016-10-04 23:12:47 UTC) #18
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/273662)
4 years, 2 months ago (2016-10-04 23:17:58 UTC) #20
Ken Russell (switch to Gerrit)
On 2016/10/04 23:17:58, commit-bot: I haz the power wrote: > Try jobs failed on following ...
4 years, 2 months ago (2016-10-04 23:33:36 UTC) #21
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/2388803002/60001
4 years, 2 months ago (2016-10-05 14:40:58 UTC) #24
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 2 months ago (2016-10-05 16:34:26 UTC) #26
commit-bot: I haz the power
4 years, 2 months ago (2016-10-05 16:37:50 UTC) #28
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/5280614b9452793b73489cb103cd740ea24f638f
Cr-Commit-Position: refs/heads/master@{#423184}

Powered by Google App Engine
This is Rietveld 408576698