android: add Java-side support for browser compositor async readback

content/browser/android/content_readback_handler.h

Index: content/browser/android/content_readback_handler.h
diff --git a/content/browser/android/content_readback_handler.h b/content/browser/android/content_readback_handler.h
index 11d5aa3ebeea895167619f9948836872763bb70b..f5080acbff4a0775bbe6af144a40589e0cc7788c 100644
--- a/content/browser/android/content_readback_handler.h
+++ b/content/browser/android/content_readback_handler.h
@@ -13,6 +13,10 @@
 
 class SkBitmap;
 
+namespace cc {
+class CopyOutputResult;
+}
+
 namespace content {
 
 // Native side of the ContentReadbackHandler.java, which issues content
@@ -35,13 +39,23 @@ class ContentReadbackHandler {
                         jfloat width,
                         jfloat height,
                         jobject content_view_core);
+  void GetCompositorBitmap(JNIEnv* env,
+                           jobject obj,
+                           jint readback_id,
+                           jlong native_window_android);
 
  private:
+  typedef base::Callback<void(bool, const SkBitmap&)> ResultCallback;
+
   virtual ~ContentReadbackHandler();
 
-  void OnFinishContentReadback(int readback_id,
-                               bool success,
-                               const SkBitmap& bitmap);
+  void OnFinishReadback(int readback_id,
+                        bool success,
+                        const SkBitmap& bitmap);
+
+  static void OnFinishCopyOutputRequest(

[Ted C, 2014/05/21 17:56:29]

maybe this should just be in an empty namespace to avoid the header declaration?

[powei, 2014/05/23 01:15:09]

Done.

+      const ResultCallback& result_callback,
+      scoped_ptr<cc::CopyOutputResult> copy_output_result);
 
   base::android::ScopedJavaGlobalRef<jobject> java_obj_;
   base::WeakPtrFactory<ContentReadbackHandler> weak_factory_;