Make SyntheticPointerAction to flush the pointer action sequence

content/browser/renderer_host/input/synthetic_pointer_action.h

 // Copyright 2015 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_INPUT_SYNTHETIC_POINTER_ACTION_H_
 #define CONTENT_BROWSER_RENDERER_HOST_INPUT_SYNTHETIC_POINTER_ACTION_H_
 
 #include "base/macros.h"
 #include "content/browser/renderer_host/input/synthetic_gesture.h"
 #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(
       const base::TimeTicks& timestamp,
       SyntheticGestureTarget* target) override;
 
  private:
-  explicit SyntheticPointerAction(const SyntheticPointerActionParams& params);
-  SyntheticGesture::Result ForwardTouchOrMouseInputEvents(
-      const base::TimeTicks& timestamp,
-      SyntheticGestureTarget* target);
+  enum GestureState { SETUP, RUNNING, INVALID, DONE };

[tdresser, 2016/12/13 14:37:43]

SETUP -> UNINITIALIZED?

The rest of these, you could say:
"Foo is GestureState"
as in 
"Foo is RUNNING"
"Foo is INVALID"
"Foo is DONE"

But SETUP is different. If you say 
"Foo is SETUP", it means something completely different!

[lanwei, 2016/12/18 17:35:55]

Acknowledged.

 
-  // SyntheticGestureController is responsible to create the
-  // SyntheticPointerActions and control when to forward them.
+  GestureState ForwardTouchOrMouseInputEvents(const base::TimeTicks& timestamp,
+                                              SyntheticGestureTarget* target);
 
-  // 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_;
+  // 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 count_;

[tdresser, 2016/12/13 14:37:43]

Rename something like |params_dispatched_count_|

[lanwei, 2016/12/18 17:35:55]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(SyntheticPointerAction);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_SYNTHETIC_POINTER_ACTION_H_