ozone: dri: Composite to intermediate surface & copy

ui/gfx/ozone/surface_ozone_canvas.h

Index: ui/gfx/ozone/surface_ozone_canvas.h
diff --git a/ui/gfx/ozone/surface_ozone_canvas.h b/ui/gfx/ozone/surface_ozone_canvas.h
index 7b09b230bfa5db640c2662e780648320c6e50d05..0d93974ba2e019cc02c44e41fb58e70c8843982c 100644
--- a/ui/gfx/ozone/surface_ozone_canvas.h
+++ b/ui/gfx/ozone/surface_ozone_canvas.h
@@ -28,15 +28,14 @@ class GFX_EXPORT SurfaceOzoneCanvas {
   // Returns an SkCanvas for drawing on the window.
   virtual skia::RefPtr<SkCanvas> GetCanvas() = 0;
 
-  // Attempts to resize the canvas to match the viewport size. Returns true if
-  // resizing was successful, otherwise false (platforms may require a fixed
-  // size canvas). After resizing, the compositor must call GetCanvas() to get
-  // the next canvas.
-  virtual bool ResizeCanvas(const gfx::Size& viewport_size) = 0;
-
-  // Present the current canvas. After presenting, the compositor must call
-  // GetCanvas() to get the next canvas.
-  virtual bool PresentCanvas() = 0;
+  // Attempts to resize the canvas to match the viewport size. After
+  // resizing, the compositor must call GetCanvas() to get the next
+  // canvas.
+  virtual void ResizeCanvas(const gfx::Size& viewport_size) = 0;

[rjkroege, 2014/04/23 22:29:32]

what was your reasoning for removing the bool return?

+
+  // Present the current canvas. After presenting, the compositor must
+  // call GetCanvas() to get the next canvas.

[rjkroege, 2014/04/23 22:29:32]

say what damage is and its semantics please? Can I call PresentCanvas > 1 time
per canvas?

+  virtual void PresentCanvas(const gfx::Rect& damage) = 0;
 
   // Returns a gfx::VsyncProvider for this surface. Note that this may be
   // called after we have entered the sandbox so if there are operations (e.g.