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:
> Reviewed-by: Hirokazu Honda <>
> Commit-Queue: Miguel Casas <>
> Cr-Commit-Position: refs/heads/master@{#615571},,

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 909926, 912295
Change-Id: Iac9941bdcaef1a7638518f835329ffb8be82233e
Commit-Queue: Hirokazu Honda <>
Reviewed-by: Hirokazu Honda <>
Cr-Commit-Position: refs/heads/master@{#616569}
10 files changed