MCVD produces CodecImage backed VideoFrames. If the image is backed by an overlay, the image needs the ability to keep the overlay alive. It also needs the ability to call ScheduleLayout on the overlay when it's rendered.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe commit 4600a3db903b2692a4fb8a16ba3bfc375cafd3fe Author: liberato@chromium.org <liberato@chromium.org> Date: Fri Oct 06 22:26:16 2017 Position overlays for MCVD. This CL sends a callback to CodecImage that it can use to position the overlay in ScheduleOverlayPlane. Currently, the SurfaceBundle is held by the callback. SurfaceDestroyed is not handled properly. Bug: 771044 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I3c0cd9046950bf030dcfe2153c45f7ffeba764d4 Reviewed-on: https://chromium-review.googlesource.com/701254 Reviewed-by: Chris Watkins <watk@chromium.org> Commit-Queue: Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#507197} [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/android_video_decode_accelerator.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/android_video_decode_accelerator.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/avda_codec_image.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/codec_image.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/codec_image.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/codec_image_unittest.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/codec_wrapper.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/codec_wrapper.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/media_codec_video_decoder.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/media_codec_video_decoder.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/media_codec_video_decoder_unittest.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/promotion_hint_aggregator.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/promotion_hint_aggregator_impl_unittest.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/video_frame_factory.h [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/video_frame_factory_impl.cc [modify] https://crrev.com/4600a3db903b2692a4fb8a16ba3bfc375cafd3fe/media/gpu/android/video_frame_factory_impl.h
Comment 1 by w...@chromium.org
, Oct 3 2017Owner: liber...@chromium.org