Move VideoFrame* out of platform/graphics and into separate directory. |
||||
Issue descriptionCurrently, VideoFrameSubmitter and VideoFrameResourceProvider both live in platform/graphics. This doesn't make much sense since these classes represent an isolated check of work dealing solely with the video stack. These files should be moved to their own platform/media directory.
,
Jan 31 2018
My hope here is that these two classes don't need to live in Blink. They're used by content/renderer/media and by media/blink. Is there a common directory that both of these can access that it would make sense to move this code to?
,
Feb 1 2018
Didn't they have to live in Blink to access the OffscreenSurface stuff? Or has something changed such that dependency is no longer valid?
,
Feb 1 2018
OffscreenCanvasProvider isn't really very blink-y either, other than just using a few blink base types. I could see moving it to some common place that both this media code and offscreen canvas could use. +junov,fsamuel who might have opinions on where this code should all live. Do y'all think all of this code should continue to live in Blink? Or maybe should go elsewhere?
,
Jul 25
,
Sep 18
pinging this to see if there are any opinions
,
Sep 19
|
||||
►
Sign in to add a comment |
||||
Comment 1 by lethalantidote@chromium.org
, Jan 31 2018