diff options
| author | posciak@chromium.org <posciak@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-05 20:06:22 +0000 |
|---|---|---|
| committer | posciak@chromium.org <posciak@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-05 20:06:22 +0000 |
| commit | 39a964df7f3d131ae17c087baa1adc8dd7ecf18c (patch) | |
| tree | d28aa3dbf5a6c2c588b4ba9345ad87181b50b6a1 | |
| parent | fc142ae086f902f1647049ce45115b5a08e80242 (diff) | |
| download | chromium_src-39a964df7f3d131ae17c087baa1adc8dd7ecf18c.zip chromium_src-39a964df7f3d131ae17c087baa1adc8dd7ecf18c.tar.gz chromium_src-39a964df7f3d131ae17c087baa1adc8dd7ecf18c.tar.bz2 | |
VAVDA: Restore check for frame_mbs_only_flag.
This check was removed due to misinterpretation of the spec, but it
seems it's needed for some streams.
BUG=225787
TEST=manual playback
Review URL: https://chromiumcodereview.appspot.com/13726007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192620 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | content/common/gpu/media/vaapi_h264_decoder.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/content/common/gpu/media/vaapi_h264_decoder.cc b/content/common/gpu/media/vaapi_h264_decoder.cc index 65fd5cd..3b53dae 100644 --- a/content/common/gpu/media/vaapi_h264_decoder.cc +++ b/content/common/gpu/media/vaapi_h264_decoder.cc @@ -2070,6 +2070,12 @@ bool VaapiH264Decoder::ProcessSPS(int sps_id) { const H264SPS* sps = parser_.GetSPS(sps_id); DCHECK(sps); + if (sps->frame_mbs_only_flag == 0) { + // Fields/interlaced video not supported. + DVLOG(1) << "frame_mbs_only_flag != 1 not supported"; + return false; + } + if (sps->gaps_in_frame_num_value_allowed_flag) { DVLOG(1) << "Gaps in frame numbers not supported"; return false; |