Make SyntheticPointerAction to flush the pointer action sequence

content/browser/renderer_host/input/synthetic_pointer_action.h

Index: content/browser/renderer_host/input/synthetic_pointer_action.h
diff --git a/content/browser/renderer_host/input/synthetic_pointer_action.h b/content/browser/renderer_host/input/synthetic_pointer_action.h
index e346e7d9dac66893420ff8e2507d954def98d56f..1a411abc1ef00d53bd12e87ded8e9bfc495ec751 100644
--- a/content/browser/renderer_host/input/synthetic_pointer_action.h
+++ b/content/browser/renderer_host/input/synthetic_pointer_action.h
@@ -10,16 +10,15 @@
 #include "content/browser/renderer_host/input/synthetic_gesture_target.h"
 #include "content/browser/renderer_host/input/synthetic_pointer_driver.h"
 #include "content/common/content_export.h"
+#include "content/common/input/synthetic_pointer_action_list_params.h"
 #include "content/common/input/synthetic_pointer_action_params.h"
 
-using blink::WebTouchEvent;
-
 namespace content {
 
 class CONTENT_EXPORT SyntheticPointerAction : public SyntheticGesture {
  public:
-  SyntheticPointerAction(std::vector<SyntheticPointerActionParams>* param_list,
-                         SyntheticPointerDriver* synthetic_pointer_driver);
+  explicit SyntheticPointerAction(
+      const SyntheticPointerActionListParams& params);
   ~SyntheticPointerAction() override;
 
   SyntheticGesture::Result ForwardInputEvents(
@@ -27,23 +26,18 @@ class CONTENT_EXPORT SyntheticPointerAction : public SyntheticGesture {
       SyntheticGestureTarget* target) override;
 
  private:
-  explicit SyntheticPointerAction(const SyntheticPointerActionParams& params);
-  SyntheticGesture::Result ForwardTouchOrMouseInputEvents(
-      const base::TimeTicks& timestamp,
-      SyntheticGestureTarget* target);
-
-  // SyntheticGestureController is responsible to create the
-  // SyntheticPointerActions and control when to forward them.
-
-  // These two objects will be owned by SyntheticGestureController, which
-  // will manage their lifetime by initiating them when it starts processing a
-  // pointer action sequence and resetting them when it finishes.
-  // param_list_ contains a list of pointer actions which will be dispatched
-  // together.
-  std::vector<SyntheticPointerActionParams>* param_list_;
-  SyntheticPointerDriver* synthetic_pointer_driver_;
-
-  SyntheticPointerActionParams params_;
+  enum GestureState { UNINITIALIZED, RUNNING, INVALID, DONE };
+
+  GestureState ForwardTouchOrMouseInputEvents(const base::TimeTicks& timestamp,
+                                              SyntheticGestureTarget* target);
+
+  // params_ contains a list of lists of pointer actions, that each list of
+  // pointer actions will be dispatched together.
+  SyntheticPointerActionListParams params_;
+  std::unique_ptr<SyntheticPointerDriver> synthetic_pointer_driver_;
+  SyntheticGestureParams::GestureSourceType gesture_source_type_;
+  GestureState state_;
+  size_t actions_dispatched_count_;

[samuong, 2016/12/21 00:02:27]

i was initially a bit confused by the meaning of this - what do you think of
"num_actions_dispatched_"?

[lanwei, 2016/12/21 20:01:14]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(SyntheticPointerAction);
 };