GitLab

Original CL was reverted because it wrongly removed the line
  va_buffers_.erase(buffer_id);

Reland changes in
  crrev.com/c/2363833/1..2/media/gpu/vaapi/vaapi_wrapper.cc


Original CL 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: Id377ebb44b6add14eb9227d1450c68a24eaee42b
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-Original-Commit-Position: refs/heads/master@{#799410}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363833
Cr-Commit-Position: refs/heads/master@{#799891}