commit d4dd0edbdb408cb45002531d00fc96b318fd34e9
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

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}