New issue
Advanced search Search tips

Issue 879424 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

VideoFrameSubmitter refactor

Project Member Reported by lethalantidote@chromium.org, Aug 31

Issue description

Now that churn should be mostly done for this code, please refactor it to be cleaner. Write a design doc explaining (at least) these 3 components:

1) Pulling out cc::Surface functionality out of both media players, placing it in a centralized location to prevent out of sync updates.

2) Simplifying and renaming VideoFrameSubmitter

3) Making the relationship between VideoFrameSubmitter and VideoLayer cleaner, as VideoLayer isn't going away quickly due to lack of support on certain Android platforms.

Beyond this, the code could use more documentation (design doc & in code comments).
 
Status: Started (was: Assigned)

Sign in to add a comment