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

Issue 760827 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 660942



Sign in to add a comment

Refactor ContentVideoViewOverlayAllocator to be usable by MCVD

Project Member Reported by w...@chromium.org, Aug 31 2017

Issue description

The main issue here is for sync SurfaceDestroyed. We need to carefully consider how to post the message to the thread on which the MCVD owning the surface is running. Presumably this will require making the surface allocation record threadsafe, and also recording a TaskRunner that surfaceDestoryed should arrive on. While avoiding issues due to MCVD thread teardown racing with surfacedestroyed. 
 

Comment 1 by w...@chromium.org, Aug 31 2017

liberato@: What's the timeline for removing CVV? It would be far preferable to be able to skip these complications.
Status: WontFix (was: Assigned)
cvv is currently unused in M62.  +1 not bothering with it for MCVD.

Comment 3 by w...@chromium.org, Aug 31 2017

\o/

Sign in to add a comment