Android WebView: flush input events during onVSync

content/browser/renderer_host/render_widget_host_view_android.h

 // Copyright (c) 2012 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 CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_
 
 #include <map>
 #include <queue>
 
@@ skipping 229 matching lines @@
 
   void SetOverlayVideoMode(bool enabled);
 
   typedef base::Callback<
       void(const base::string16& content, int start_offset, int end_offset)>
       TextSurroundingSelectionCallback;
   void SetTextSurroundingSelectionCallback(
       const TextSurroundingSelectionCallback& callback);
 
  private:
+  enum VSyncSubscriberType { INPUT = 1 << 0, BEGIN_FRAME = 1 << 1 };

[boliu, 2014/08/20 20:30:00]

nit: separate lines (unless this is what clang-format did..?)

+
   // TouchSelectionControllerClient implementation.
   virtual bool SupportsAnimation() const OVERRIDE;
   virtual void SetNeedsAnimate() OVERRIDE;
   virtual void MoveCaret(const gfx::PointF& position) OVERRIDE;
   virtual void SelectBetweenCoordinates(const gfx::PointF& start,
                                         const gfx::PointF& end) OVERRIDE;
   virtual void OnSelectionEvent(SelectionEventType event,
                                 const gfx::PointF& anchor_position) OVERRIDE;
   virtual scoped_ptr<TouchHandleDrawable> CreateDrawable() OVERRIDE;
 
@@ skipping 50 matching lines @@
   void InternalSwapCompositorFrame(uint32 output_surface_id,
                                    scoped_ptr<cc::CompositorFrame> frame);
 
   bool Animate(base::TimeTicks frame_time);
 
   void OnContentScrollingChange();
   bool IsContentScrolling() const;
 
   float GetDpiScale() const;
 
+  // type is a bit field containing one or more VSyncSubscriberType.
+  void SubscribeToVSync(uint32 type);
+  void UnsubscribeToVSync(uint32 type);
+  bool IsSubscribedToType(uint32 type);
+
   // The model object.
   RenderWidgetHostImpl* host_;
 
-  // Used to track whether this render widget needs a BeginFrame.
-  bool needs_begin_frame_;
-
   bool is_showing_;
 
   // ContentViewCoreImpl is our interface to the view system.
   ContentViewCoreImpl* content_view_core_;
 
   ImeAdapterAndroid ime_adapter_android_;
 
   // Body background color of the underlying document.
   SkColor cached_background_color_;
 
@@ skipping 27 matching lines @@
 
   // Handles gesture based text selection
   GestureTextSelector gesture_text_selector_;
 
   // Manages selection handle rendering and manipulation.
   // This will always be NULL if |content_view_core_| is NULL.
   scoped_ptr<TouchSelectionController> selection_controller_;
   bool touch_scrolling_;
   size_t potentially_active_fling_count_;
 
-  bool flush_input_requested_;
-
   int accelerated_surface_route_id_;
 
   // Size to use if we have no backing ContentViewCore
   gfx::Size default_size_;
 
   const bool using_synchronous_compositor_;
 
+  // Used to track how VSync should be dispatched. For example, if this bit
+  // mask has BEGIN_FRAME bit on, a new frame will be requested during OnVSync.
+  uint32 vsync_subscribers_bitmask_;
+
   scoped_ptr<DelegatedFrameEvictor> frame_evictor_;
 
   size_t locks_on_frame_count_;
-  bool observing_root_window_;
 
   struct LastFrameInfo {
     LastFrameInfo(uint32 output_id,
                   scoped_ptr<cc::CompositorFrame> output_frame);
     ~LastFrameInfo();
     uint32 output_surface_id;
     scoped_ptr<cc::CompositorFrame> frame;
   };
 
   scoped_ptr<LastFrameInfo> last_frame_info_;
 
   TextSurroundingSelectionCallback text_surrounding_selection_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewAndroid);
 };
 
 } // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_