Commit 0108ae61 authored by Andres Calderon Jaramillo's avatar Andres Calderon Jaramillo Committed by Chromium LUCI CQ

VaapiTest.LowQualityEncodingSetting: Increase size

This CL increases the size passed to VaapiWrapper::CreateContext() from
64x64 to 128x128. It seems that there is a minimum size that is not
obvious through vainfo -a. However, when doing vadumpcaps, the
VP9Profile0/EncSliceLP lists min_width: 128, min_height: 96 for the
associated surface formats. Therefore, 128x128 should allow the context
to be created and the test to continue.

Bug: b:174486634
Test: graphics.VAAPIUnittest.common on volteer2 and drawlat.
Change-Id: I5fe06e2aa3e5e5247fd14b3a016645fad948400c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2566253
Auto-Submit: Andres Calderon Jaramillo <andrescj@chromium.org>
Reviewed-by: default avatarMiguel Casas <mcasas@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#832388}
parent 9115d3f9
...@@ -306,7 +306,7 @@ TEST_F(VaapiTest, LowQualityEncodingSetting) { ...@@ -306,7 +306,7 @@ TEST_F(VaapiTest, LowQualityEncodingSetting) {
// number (it could ignore it), so instead just make sure there's a // number (it could ignore it), so instead just make sure there's a
// |pending_va_buffers_| that, when mapped, looks correct. That buffer // |pending_va_buffers_| that, when mapped, looks correct. That buffer
// should be created by CreateContext(). // should be created by CreateContext().
ASSERT_TRUE(wrapper->CreateContext(gfx::Size(64, 64))); ASSERT_TRUE(wrapper->CreateContext(gfx::Size(128, 128)));
ASSERT_EQ(wrapper->pending_va_buffers_.size(), 1u); ASSERT_EQ(wrapper->pending_va_buffers_.size(), 1u);
{ {
base::AutoLock auto_lock(*wrapper->va_lock_); base::AutoLock auto_lock(*wrapper->va_lock_);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment