Commit 06aed1c8 authored by Chih-Yu Huang's avatar Chih-Yu Huang Committed by Commit Bot

media/gpu/chromeos/V4L2SVD: Skip setting resolution at Initialize().

The client of video decoder would pass the initial resolution when
calling VideoDecoder::Initialize(). However, V4L2SVD could still
get the resolution by AcceleratedVideoDecoder parsing the header.
Therefore, V4L2SVD don't need to set the resolution at Initialize().
Moreover, at the next CL we will implement an adapter from VDA to VD.
Setting the resolution at Initialize() cannot be fit with VDA api.
This CL removes V4L2SVD setting resolution at Initialize().

Bug: b:136716838
Test: pass vda_tests on Kevin

Change-Id: I7957bf23574ad47d3666f39b05371182f2b37301
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1923877Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
Commit-Queue: Chih-Yu Huang <akahuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#718910}
parent 3fb8f1d5
......@@ -192,19 +192,6 @@ void V4L2SliceVideoDecoder::Initialize(const VideoDecoderConfig& config,
return;
}
if (!SetCodedSizeOnInputQueue(config.coded_size())) {
VLOGF(1) << "Failed to set coded size on input queue";
std::move(init_cb).Run(false);
return;
}
// Setup output format.
if (!SetupOutputFormat(config.coded_size(), config.visible_rect())) {
VLOGF(1) << "Failed to setup output format.";
std::move(init_cb).Run(false);
return;
}
if (input_queue_->AllocateBuffers(kNumInputBuffers, V4L2_MEMORY_MMAP) == 0) {
VLOGF(1) << "Failed to allocate input buffer.";
std::move(init_cb).Run(false);
......
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