OnDrawHardware() implementation with async messages

android_webview/browser/browser_view_renderer.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ANDROID_WEBVIEW_BROWSER_BROWSER_VIEW_RENDERER_H_
 #define ANDROID_WEBVIEW_BROWSER_BROWSER_VIEW_RENDERER_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 111 matching lines @@
                             const gfx::Vector2dF& total_scroll_offset_dip,
                             const gfx::Vector2dF& max_scroll_offset_dip,
                             const gfx::SizeF& scrollable_size_dip,
                             float page_scale_factor,
                             float min_page_scale_factor,
                             float max_page_scale_factor) override;
   void DidOverscroll(content::SynchronousCompositor* compositor,
                      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(
       CompositorFrameConsumer* compositor_frame_consumer) override;
   void RemoveCompositorFrameConsumer(
       CompositorFrameConsumer* compositor_frame_consumer) override;
 
   void SetActiveCompositorID(const CompositorID& compositor_id);
 
   // Visible for testing.
@@ skipping 72 matching lines @@
   gfx::Vector2dF max_scroll_offset_dip_;
 
   // Used to prevent rounding errors from accumulating enough to generate
   // visible skew (especially noticeable when scrolling up and down in the same
   // spot over a period of time).
   // TODO(miletus): Make overscroll_rounding_error_ a gfx::ScrollOffset.
   gfx::Vector2dF overscroll_rounding_error_;
 
   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);
 };
 
 }  // namespace android_webview
 
 #endif  // ANDROID_WEBVIEW_BROWSER_BROWSER_VIEW_RENDERER_H_