sync compositor: Signal async frame on IO thread

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 112 matching lines @@
                             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;
   ui::TouchHandleDrawable* CreateDrawable() override;
-  void OnDrawHardwareProcessFrameFuture(
-      const scoped_refptr<content::SynchronousCompositor::FrameFuture>&
-          frame_future) 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 51 matching lines @@
   bool hardware_enabled_;
   float dip_scale_;
   float page_scale_factor_;
   float min_page_scale_factor_;
   float max_page_scale_factor_;
   bool on_new_picture_enable_;
   bool clear_view_;
 
   bool offscreen_pre_raster_;
 
+  // Must do a synchronous draw first to ensure GL bindings are intialized.

[dcheng, 2016/10/21 22:20:12]

Nit: initialized, though that will push this line over 80 chars =/

[boliu, 2016/10/24 15:57:23]

Nah, this line is only 75 chars :p

Done.

+  // TODO(boliu): Wait on render thread and remove this.
+  bool allow_async_draw_;
+
   gfx::Vector2d last_on_draw_scroll_offset_;
   gfx::Rect last_on_draw_global_visible_rect_;
 
   gfx::Size size_;
 
   gfx::SizeF scrollable_size_dip_;
 
   // Current scroll offset in CSS pixels.
   // TODO(miletus): Make scroll_offset_dip_ a gfx::ScrollOffset.
   gfx::Vector2dF scroll_offset_dip_;
 
   // Max scroll offset in CSS pixels.
   // TODO(miletus): Make max_scroll_offset_dip_ a gfx::ScrollOffset.
   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_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserViewRenderer);
 };
 
 }  // namespace android_webview
 
 #endif  // ANDROID_WEBVIEW_BROWSER_BROWSER_VIEW_RENDERER_H_