New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 596236 link

Starred by 1 user
Status: Fixed
Owner:
dalecur...@chromium.org
Closed: Apr 2016
Cc:
█ ti...@chromium.org
xhw...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Trying to initialize AVDA w/ known-unaccelerated codecs that it will reject is slow. Don't do this.

Project Member Reported by dalecur...@chromium.org, Mar 19 2016 
This can cost up to 300ms on slower devices -- instead we should provider the known-unaccelerated status to the GpuVideoDecoder through supported profiles such that these can be rejected on the renderer side.

CL out here: https://codereview.chromium.org/1820553002

Might be worth merging later.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 19 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9

commit 4708098a8d03f9d0e7b5e6931f9c52c092e78fa9
Author: dalecurtis <dalecurtis@chromium.org>
Date: Sat Mar 19 04:54:11 2016

Expose encrypted_only attribute on VDA supported profiles.

This allows us to reduce video playback startup for software codecs
from ~50-300ms (N5..Android One) to ~microseconds.

BUG= 596236 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Review URL: https://codereview.chromium.org/1820553002

Cr-Commit-Position: refs/heads/master@{#382185}

[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/content/common/gpu/gpu_param_traits_macros.h
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/content/common/gpu/media/android_video_decode_accelerator.cc
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/content/common/gpu/media/gpu_video_accelerator_util.cc
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/gpu/config/gpu_info.cc
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/gpu/config/gpu_info.h
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/media/filters/gpu_video_decoder.cc
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/media/filters/gpu_video_decoder.h
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/media/video/video_decode_accelerator.cc
[modify] https://crrev.com/4708098a8d03f9d0e7b5e6931f9c52c092e78fa9/media/video/video_decode_accelerator.h

Comment 2 by dalecur...@chromium.org, Apr 7 2016

Status: Fixed (was: Started)
Not merging.
►

Sign in to add a comment