GitLab

This reverts commit 9881e6cd.

Reason for revert: Needs a follow up crrev.com/c/2362068, but
the code is still faulty because it'd hit a DCHECK. Let's revert
this one and land a correct CL instead.

Original change's description:
> media/gpu/vaapi: use new |va_buffer_for_vpp_| for Vpp/BlitSurface()
> 
> ToT VaapiWrapper uses |va_buffers_| for internal Vpp uses and also for
> external VEA/JEA use. This CL starts refactoring these VABufferIDs
> lifetime, by splitting the Vpp VABufferID out of |va_buffers_| and
> into a ScopedID |va_buffer_for_vpp_|.
> 
> Bug: b:162962069
> Change-Id: Ic54d0d26f5d17ea49fe896cf7ca5e1c931d07e0f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2352522
> Commit-Queue: Miguel Casas <mcasas@chromium.org>
> Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#799410}

TBR=mcasas@chromium.org,hiroh@chromium.org

Change-Id: I2b13e096fb7fce00c4dbbaed64bd4d231de34d15
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: b:162962069
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363798Reviewed-by: Miguel Casas <mcasas@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799455}