• Miguel Casas's avatar
    vaapi: Don't use = {} to clear base::queues · 83bd7656
    Miguel Casas authored
    This CL removes the use of = {} to clear base::queues, because, for classes,
    zero initialization simply clears the (non-static) data members, which might
    leave the objects in an incorrect meta state because ctors are not called:
    this should not be an issue for std::queue, but it is for base::queue, which
    is implemented as a base::circular_deque.
    
    Nothing is crashing now, but a sibling unlanded sister CL (crrev.com/c/947341)
    had some issues with a similar idiom, so I'm removing it just in case.
    
    [1] http://en.cppreference.com/w/cpp/language/zero_initialization
    
    Bug: 717265
    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;master.tryserver.chromium.win:win_optional_gpu_tests_rel
    Change-Id: I96af3e6475c0596f3316219bd51732e085f5e098
    Reviewed-on: https://chromium-review.googlesource.com/1005917Reviewed-by: default avatarDaniele Castagna <dcastagna@chromium.org>
    Commit-Queue: Miguel Casas <mcasas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#550162}
    83bd7656
vaapi_video_decode_accelerator.cc 29.5 KB