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

Issue 2683953003: Add new md5sums for Intel Platforms

Created:
3 years, 10 months ago by seanvk
Modified:
3 years, 10 months ago
Reviewers:
jrummell, Pawel Osciak
CC:
chromium-reviews, feature-media-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add new md5sums for Intel Platforms Changing scaling property to VA_FILTER_SCALING_HQ in media/gpu/vaapi_wrapper.c requires an update to checksums due to the differeing filter properties AVS Scaling options. BUG=685432 TEST=TEST=Thumbnail.* test from video_decode_accelerator_unittest for h264, vp8, vp9 R=posciak@chromium.org,jrummell Signed-off-by: Sean V Kelley <seanvk@posteo.de>;

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M media/test/data/test-25fps.h264.md5 View 1 chunk +2 lines, -0 lines 0 comments Download
M media/test/data/test-25fps.vp8.md5 View 1 chunk +2 lines, -0 lines 0 comments Download
M media/test/data/test-25fps.vp9.md5 View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
jrummell
rs lgtm
3 years, 10 months ago (2017-02-08 22:33:16 UTC) #1
Pawel Osciak
lgtm % nit: Please correct the CL message to say these are for VA_FILTER_SCALING_DEFAULT, not ...
3 years, 10 months ago (2017-02-17 02:00:12 UTC) #2
Pawel Osciak
3 years, 10 months ago (2017-02-17 02:02:11 UTC) #3
Actually, could you combine this into one CL with
https://codereview.chromium.org/2656073003/ please? This would make
reverts/tracking dependencies easier. Thank you.

Powered by Google App Engine
This is Rietveld 408576698