cc: Move video upload to VideoResourceUpdater.

The VideoLayerImpl currently does upload of video frames from media::VideoFrame
to hardware/software resources, and displays these resources through quads. This
code is unfriendly to ubercomp as it requires holding onto the hardware resources
when using hardware decode, and because it reuses the same textures every frame
even though they need to be sent to the parent compositor.

This CL introduces the VideoResourceUpdater class, and moves all logic around
media::VideoFrame to that class. The VideoResourceUpdater class takes as input
a VideoFrame, and produces a set of TextureMailboxes for the video layer to
consume.

In the software case, the VideoResourceUpdater sets itself up as the release
callback so it can delete the backing texture when the browser compositor is
done with it (it could recycle it in the future).

In the hardware case, the VideoResourceUpdater takes a callback as input, so
it can have the WebMediaPlayer notified when the texture is given back from
the browser, and the hardware decoder can use it again for writing.

This CL also prepares us better for software uber-compositing video. The video
layer deals now only with abstract "resources", except for the mailboxes for
ubercompositor. The TextureMailbox construct needs to be abstracted in order to
hold a hardware or software backing for ubercompositor. Then the video layer's
special-case code for software can be entirely removed and it will be fully
software-uber-compositor-ready.

Currently, the VideoLayerImpl just makes use of the VideoResourceUpdater class,
making this a refactor without any functional change. This will make video
playback in ubercompositor work correctly for software-decoded video. In order
to address hardware-decoded video, a followup CL will have the WebMediaPlayer
hook up a callback through the VideoResourceUpdater to ensure it doesn't reuse
a texture for decode until it is returned from the parent compositor.

R=enne,jamesr
BUG=179729


Review URL: https://chromiumcodereview.appspot.com/13445009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193323 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 7 deletions

diff --git a/cc/quads/io_surface_draw_quad.cc b/cc/quads/io_surface_draw_quad.cc
index eb4aa20..1bbf8a4 100644
--- a/cc/quads/io_surface_draw_quad.cc
+++ b/cc/quads/io_surface_draw_quad.cc
@@ -9,7 +9,7 @@
 namespace cc {
 
 IOSurfaceDrawQuad::IOSurfaceDrawQuad()
-    : io_surface_texture_id(0),
+    : io_surface_resource_id(0),
       orientation(FLIPPED) {
 }
 
@@ -21,14 +21,14 @@ void IOSurfaceDrawQuad::SetNew(const SharedQuadState* shared_quad_state,
                                gfx::Rect rect,
                                gfx::Rect opaque_rect,
                                gfx::Size io_surface_size,
-                               unsigned io_surface_texture_id,
+                               unsigned io_surface_resource_id,
                                Orientation orientation) {
   gfx::Rect visible_rect = rect;
   bool needs_blending = false;
   DrawQuad::SetAll(shared_quad_state, DrawQuad::IO_SURFACE_CONTENT, rect,
                    opaque_rect, visible_rect, needs_blending);
   this->io_surface_size = io_surface_size;
-  this->io_surface_texture_id = io_surface_texture_id;
+  this->io_surface_resource_id = io_surface_resource_id;
   this->orientation = orientation;
 }
 
@@ -38,19 +38,18 @@ void IOSurfaceDrawQuad::SetAll(const SharedQuadState* shared_quad_state,
                                gfx::Rect visible_rect,
                                bool needs_blending,
                                gfx::Size io_surface_size,
-                               unsigned io_surface_texture_id,
+                               unsigned io_surface_resource_id,
                                Orientation orientation) {
   DrawQuad::SetAll(shared_quad_state, DrawQuad::IO_SURFACE_CONTENT, rect,
                    opaque_rect, visible_rect, needs_blending);
   this->io_surface_size = io_surface_size;
-  this->io_surface_texture_id = io_surface_texture_id;
+  this->io_surface_resource_id = io_surface_resource_id;
   this->orientation = orientation;
 }
 
 void IOSurfaceDrawQuad::IterateResources(
     const ResourceIteratorCallback& callback) {
-  // TODO(danakj): Convert to TextureDrawQuad?
-  NOTIMPLEMENTED();
+  io_surface_resource_id = callback.Run(io_surface_resource_id);
 }
 
 const IOSurfaceDrawQuad* IOSurfaceDrawQuad::MaterialCast(