OnDrawHardware() implementation with async messages

android_webview/browser/browser_view_renderer.h

Index: android_webview/browser/browser_view_renderer.h
diff --git a/android_webview/browser/browser_view_renderer.h b/android_webview/browser/browser_view_renderer.h
index ac70535f4bef0c4c4dc983843b06608f4910b8c7..c55ba0db160e52654b3379c53a825cac022b6fe2 100644
--- a/android_webview/browser/browser_view_renderer.h
+++ b/android_webview/browser/browser_view_renderer.h
@@ -129,6 +129,8 @@ class BrowserViewRenderer : public content::SynchronousCompositorClient,
                      const gfx::Vector2dF& accumulated_overscroll,
                      const gfx::Vector2dF& latest_overscroll_delta,
                      const gfx::Vector2dF& current_fling_velocity) override;
+  void OnDrawHardwareProcessFrame(
+      content::SynchronousCompositor::Frame frame) override;
 
   // CompositorFrameProducer overrides
   void OnParentDrawConstraintsUpdated(
@@ -221,6 +223,13 @@ class BrowserViewRenderer : public content::SynchronousCompositorClient,
 
   ParentCompositorDrawConstraints external_draw_constraints_;
 
+  gfx::Transform transform_for_tile_priority_;
+
+  gfx::Rect viewport_rect_for_tile_priority_;

[boliu, 2016/08/23 02:39:52]

there is no need to save these two because they are already saved, transform is
in external_draw_constraints, and viewport can be recomputed

fwiw, remember this code will be deleted again once the futures thing works. and
this will be passed to render through the futures object

[ojars, 2016/08/23 21:58:43]

Done.

+
+  bool async_frame_messages_;

[boliu, 2016/08/23 02:39:52]

const, and move it up next to ui_task_runner_ ie the other const variables

maybe rename this async_on_draw_hardware_? "message" doesn't make sense here

[ojars, 2016/08/23 21:58:42]

Done.

+  bool frame_produced_;
+
   DISALLOW_COPY_AND_ASSIGN(BrowserViewRenderer);
 };