commit | d4dd0edbdb408cb45002531d00fc96b318fd34e9 | [log] [tgz] |
---|---|---|
author | Hirokazu Honda <hiroh@chromium.org> | Fri Dec 14 03:02:01 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Dec 14 03:02:01 2018 |
tree | 36d9bf15a2c5cdf9dcf6c632de28738a58375a18 | |
parent | 682c9f02d71b59ef95cfbcec28a57edbe0ba203b [diff] |
Revert "Vaapi decode: split |decoder_|s GetRequiredNumOfPictures()" This reverts commit 30fbb99c379e3aa5cd9db30649fdf370681ae334. Reason for revert: Breaks ARC++ video playback with YouTube app. Original change's description: > Vaapi decode: split |decoder_|s GetRequiredNumOfPictures() > > This CL reduces the amount of PictureBuffers requested to be allocated > by the |client_| when we are not |decode_using_client_picture_buffers_|. > Instead, it "splits" the requested allocations into > - the actual needed PictureBuffers (A) > - the codec's requested reference frames (B) (a new method > GetNumReferenceFrames() is added to AcceleratedVideoDecoder for this). > > This splitting saves a lot of memory, since we allocate A+B buffers > instead of 2*(A+B). (B is 5 and A is 4-VP8, 4-12 H264/VP9) > > Test: crosvideo changing resolutions for each codec, v_d_a_unittest > on nocturne (KBL) and caroline (SKL). > > Bug: 909926 > Change-Id: I1fce92863beddce0e8b221861cf156b9c3ad0c0f > Reviewed-on: https://chromium-review.googlesource.com/c/1363807 > Reviewed-by: Hirokazu Honda <hiroh@chromium.org> > Commit-Queue: Miguel Casas <mcasas@chromium.org> > Cr-Commit-Position: refs/heads/master@{#615571} TBR=mcasas@chromium.org,dcastagna@chromium.org,hiroh@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 909926, 912295 Change-Id: Iac9941bdcaef1a7638518f835329ffb8be82233e Reviewed-on: https://chromium-review.googlesource.com/c/1377480 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by: Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#616569}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .