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

Issue 752651 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug

Blocking:
issue 618368



Sign in to add a comment

must-promote overlays for L1/3

Project Member Reported by liber...@chromium.org, Aug 4 2017

Issue description

since AVDA will now use overlays for L1 / L3, make the compositor promote them even if it thinks that it's not a good idea.

the easiest way is probably to switch "is_backed_by_surface_texture" to be "is_backed_by_surface_view".  then, always promote it ALLOW_OVERLAY and it's backed by a surface view.  or, maybe we can just keep it that way, since it's android only.

also check if the "must promote" overlays for windows are easily used here.
 
Cc: yini...@chromium.org
one symptom of this bug is that when rotate L1 video content in fullscreen mode, there is a moment video becomes all black.
repro steps:
1. navigate to https://ats.sandbox.google.com/static/chrome_test_app/index.html
2. load L1_video.mp4.mpd
3. switch to fullscreen mode
4. rotate between portrait and landscape mode

expect:
video playback is always present even at rotate time.

actual:
for on moment after rotate, the video playback is all black, then video resumes.
notes to myself about c#2:

- L3 and clear both show android animation
- i suspect that it's caused by compositor feedback during the transition.  L3 cancels the overlay, L1 doesn't.  could verify by setting is_required for L3, but didn't do that yet.
- turning off the orientation change blackout didn't seem to affect it.

the black screen behavior also happens when switch video tab to background then bring it back to foreground.
Cc: liber...@chromium.org
Owner: ----
Status: Available (was: Assigned)
Labels: -Pri-1 Pri-2

Sign in to add a comment