android: Migrate old content readback to use async readback (and delegated renderer)

content/public/browser/android/compositor.h

Index: content/public/browser/android/compositor.h
diff --git a/content/public/browser/android/compositor.h b/content/public/browser/android/compositor.h
index 0a96af555783a25f2c58dc31705976221cb564cd..7a0457f1733ac72846a2504a4cfa104cc5c09a70 100644
--- a/content/public/browser/android/compositor.h
+++ b/content/public/browser/android/compositor.h
@@ -72,38 +72,23 @@ class CONTENT_EXPORT Compositor {
   // Composite immediately. Used in single-threaded mode.
   virtual void Composite() = 0;
 
-  // Generates a UIResource and returns a UIResourceId.  May return 0.
+  // Generates a UIResource and returns a UIResourceId.  |is_transient|
+  // indicates whether or not to release the resource once the bitmap
+  // has been uploaded. May return 0.
   virtual cc::UIResourceId GenerateUIResource(
-      const cc::UIResourceBitmap& bitmap) = 0;
+      const cc::UIResourceBitmap& bitmap,

[aelias_OOO_until_Jul13, 2014/01/31 05:50:10]

Since GenerateCompressedUIResource takes a void* data, I think this should take
an SkBitmap for consistency.  (And maybe we can then avoid referencing
UIResourceBitmap from downstream entirely?)  Then what's currently in this
method should be in a private method.

[powei, 2014/02/01 01:30:49]

Done.  Removed UIResourceBitmap references downstream.

+      bool is_transient) = 0;
+
+  // Generates a compressed UIResource.  See above for |is_transient|.  May

[aelias_OOO_until_Jul13, 2014/01/31 05:50:10]

"ETC1 compressed"

[powei, 2014/02/01 01:30:49]

Done.

+  // return 0.
+  virtual cc::UIResourceId GenerateCompressedUIResource(gfx::Size& size,
+                                                        int data_size,
+                                                        void* data,
+                                                        bool is_transient) = 0;
 
   // Deletes a UIResource.
   virtual void DeleteUIResource(cc::UIResourceId resource_id) = 0;
 
-  // Generates an OpenGL texture and returns a texture handle.  May return 0
-  // if the current context is lost.
-  virtual blink::WebGLId GenerateTexture(gfx::JavaBitmap& bitmap) = 0;
-
-  // Generates an OpenGL compressed texture and returns a texture handle.  May
-  // return 0 if the current context is lost.
-  virtual blink::WebGLId GenerateCompressedTexture(gfx::Size& size,
-                                                    int data_size,
-                                                    void* data) = 0;
-
-  // Deletes an OpenGL texture.
-  virtual void DeleteTexture(blink::WebGLId texture_id) = 0;
-
-  // Grabs a copy of |texture_id| and saves it into |bitmap|.  No scaling is
-  // done.  It is assumed that the texture size matches that of the bitmap.
-  virtual bool CopyTextureToBitmap(blink::WebGLId texture_id,
-                                   gfx::JavaBitmap& bitmap) = 0;
-
-  // Grabs a copy of |texture_id| and saves it into |bitmap|.  No scaling is
-  // done. |src_rect| allows the caller to specify which rect of |texture_id|
-  // to copy to |bitmap|.  It needs to match the size of |bitmap|.  Returns
-  // true if the |texture_id| was copied into |bitmap|, false if not.
-  virtual bool CopyTextureToBitmap(blink::WebGLId texture_id,
-                                   const gfx::Rect& src_rect,
-                                   gfx::JavaBitmap& bitmap) = 0;
  protected:
   Compositor() {}
 };