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

Issue 2299083002: Switch context_lost test to gpu_integration_test harness. (Closed)

Created:
4 years, 3 months ago by Ken Russell (switch to Gerrit)
Modified:
4 years, 3 months ago
Reviewers:
Zhenyao Mo
CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org, vmiura, eyaich1, nednguyen, ynovikov
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Switch context_lost test to gpu_integration_test harness. (Re-land after updating the test expectations for the new naming conventions, and marking some tests as Skip rather than Fail.) BUG=352807 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 Committed: https://crrev.com/338e4c1755baa7ff0705983495eda07b5cbc8285 Cr-Commit-Position: refs/heads/master@{#418415}

Patch Set 1 #

Patch Set 2 : Rebased. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+122 lines, -121 lines) Patch
M content/test/gpu/generate_buildbot_json.py View 1 2 chunks +7 lines, -7 lines 0 comments Download
M content/test/gpu/gpu_tests/context_lost_expectations.py View 3 chunks +12 lines, -12 lines 0 comments Download
M content/test/gpu/gpu_tests/context_lost_integration_test.py View 1 3 chunks +13 lines, -12 lines 0 comments Download
M testing/buildbot/chromium.gpu.json View 1 8 chunks +16 lines, -16 lines 0 comments Download
M testing/buildbot/chromium.gpu.fyi.json View 1 37 chunks +74 lines, -74 lines 0 comments Download

Messages

Total messages: 13 (6 generated)
Ken Russell (switch to Gerrit)
I'm going to hold off landing this until http://crbug.com/642984 has been addressed.
4 years, 3 months ago (2016-09-01 00:16:54 UTC) #2
Ken Russell (switch to Gerrit)
4 years, 3 months ago (2016-09-01 00:18:45 UTC) #4
Ken Russell (switch to Gerrit)
4 years, 3 months ago (2016-09-01 00:19:02 UTC) #5
Ken Russell (switch to Gerrit)
FYI. Proceeding with CQ'ing this again TBR'd.
4 years, 3 months ago (2016-09-13 22:23:28 UTC) #7
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/2299083002/20001
4 years, 3 months ago (2016-09-13 22:24:10 UTC) #9
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 3 months ago (2016-09-13 23:40:38 UTC) #11
commit-bot: I haz the power
4 years, 3 months ago (2016-09-13 23:44:07 UTC) #13
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/338e4c1755baa7ff0705983495eda07b5cbc8285
Cr-Commit-Position: refs/heads/master@{#418415}

Powered by Google App Engine
This is Rietveld 408576698