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

Issue 2797843002: Re-enable pixel video tests. (Closed)

Created:
3 years, 8 months ago by jbauman
Modified:
3 years, 8 months ago
CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Re-enable pixel video tests. The tests have been resized to work on the Nexus 5, and new windows 10 baselines were generated to handle its different video processing system. VP9 tests are still disabled on linux and android NVIDIA cards because we were seeing hangs there. BUG=704389 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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 Review-Url: https://codereview.chromium.org/2797843002 Cr-Commit-Position: refs/heads/master@{#461895} Committed: https://chromium.googlesource.com/chromium/src/+/45c4581da9985f74c34aef5456786d5cc1773790

Patch Set 1 #

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

Messages

Total messages: 14 (9 generated)
jbauman
3 years, 8 months ago (2017-04-04 22:41:47 UTC) #5
Ken Russell (switch to Gerrit)
lgtm
3 years, 8 months ago (2017-04-04 23:04:40 UTC) #6
Ken Russell (switch to Gerrit)
Note, feel free to TBR= test expectation changes like this to me to keep things ...
3 years, 8 months ago (2017-04-04 23:05:15 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/2797843002/1
3 years, 8 months ago (2017-04-04 23:12:40 UTC) #11
commit-bot: I haz the power
3 years, 8 months ago (2017-04-04 23:31:49 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/45c4581da9985f74c34aef545678...

Powered by Google App Engine
This is Rietveld 408576698