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).
Comment 1 by lethalantidote@chromium.org
, Sep 18