• Miguel Casas's avatar
    Revert "vaapi: cleanup a few comments and var names" · 2edc7ca9
    Miguel Casas authored
    This reverts commit c813adf6.
    
    Reason for revert: Needed to revert other CLs that caused a
    regression in video_VideoSeek: https://crbug.com/834146
    Bug: 834146
    
    Original change's description:
    > vaapi: cleanup a few comments and var names
    >
    > This CL cleans up a few things that I realized while
    > debugging other CLs in the area:
    > - |available_va_surfaces_| is a std::list on ToT, but has
    >  queue semantics, so it's changed int his CL.
    > -  s/output_buffers_/available_va_surfaces_/ because that's
    >  what they are.
    > - s/TryOutputSurface/TryOutputPicture/ because that's what
    >  the method does.
    > - s/pictures_/picture_map_/ to better define what it is. After
    >  dcastagna@ suggestion, it's made a base::small_map in PS7
    >
    > Comments updated throughout these variables.
    >
    > No change in behaviour intended, but tested nonetheless with
    > vp8/9-h264 crosvideo playback.
    >
    > Bug: 822346
    > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel
    > Change-Id: I2117ceac29f7d6ed23d698c4abd5c72e0140ae47
    > Reviewed-on: https://chromium-review.googlesource.com/988512
    > Commit-Queue: Miguel Casas <mcasas@chromium.org>
    > Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#547552}
    
    TBR=mcasas@chromium.org,dcastagna@chromium.org,hoegsberg@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 822346
    Change-Id: I7d242ad92a68596766f6cfe0ad2be37da3c4fb20
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel
    Reviewed-on: https://chromium-review.googlesource.com/1020005
    Commit-Queue: Miguel Casas <mcasas@chromium.org>
    Reviewed-by: default avatarMiguel Casas <mcasas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#552166}
    2edc7ca9
vaapi_video_decode_accelerator.cc 29 KB