Prepare SyntheticPointerAction to handle touch actions for multiple fingers

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.h"
 #include "content/common/content_export.h"
 #include "content/common/input/synthetic_pointer_action_params.h"
 
 namespace content {
 
 class CONTENT_EXPORT SyntheticPointerAction : public SyntheticGesture {
  public:
   explicit SyntheticPointerAction(const SyntheticPointerActionParams& params);
-  SyntheticPointerAction(const SyntheticPointerActionParams& params,
-                         SyntheticPointer* synthetic_pointer);
+  SyntheticPointerAction(
+      const std::vector<SyntheticPointerActionParams>& param_list,
+      SyntheticPointer* synthetic_pointer,
+      int* index_map);
   ~SyntheticPointerAction() override;
 
   SyntheticGesture::Result ForwardInputEvents(
       const base::TimeTicks& timestamp,
       SyntheticGestureTarget* target) override;
 
   void ForwardTouchOrMouseInputEvents(const base::TimeTicks& timestamp,
                                       SyntheticGestureTarget* target);
 
+  void SetSyntheticPointer(SyntheticPointer* synthetic_pointer) {
+    synthetic_pointer_ = synthetic_pointer;
+  }
+
  private:
+  std::vector<SyntheticPointerActionParams> param_list_;
+  SyntheticPointer* synthetic_pointer_;

[tdresser, 2016/04/18 15:24:39]

What's the lifetime here? Should this be unique_ptr?

[lanwei, 2016/04/19 19:05:32]

synthetic_pointer_ and index_map_ are passed to this function as pointer,
because this function will change the value of these two variables, but
RenderWidgetHostImpl should keep the ownership of these two and manager their
create and destroy, so I use raw pointer.

+  int* index_map_;

[tdresser, 2016/04/18 15:24:39]

Should this be unique_ptr?

   SyntheticPointerActionParams params_;
-  SyntheticPointer* synthetic_pointer_;
 
   DISALLOW_COPY_AND_ASSIGN(SyntheticPointerAction);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_SYNTHETIC_POINTER_ACTION_H_