Add SyntheticPointerActionParams used in Chromedriver extension

content/common/input/synthetic_pointer_action_params.h

+// Copyright 2016 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_COMMON_INPUT_SYNTHETIC_POINTER_ACTION_PARAMS_H_
+#define CONTENT_COMMON_INPUT_SYNTHETIC_POINTER_ACTION_PARAMS_H_
+
+#include "base/logging.h"
+#include "content/common/content_export.h"
+#include "content/common/input/input_param_traits.h"
+#include "content/common/input/synthetic_gesture_params.h"
+#include "ui/gfx/geometry/point_f.h"
+
+namespace content {
+
+struct CONTENT_EXPORT SyntheticPointerActionParams
+    : public SyntheticGestureParams {
+ public:
+  // When we send actions one by one, once we receive a PROCESS action, we will
+  // start to dispatch an event with all the pending actions.
+  // TODO(lanwei): we will pre-process the delay action in the

[tdresser, 2016/03/22 14:38:10]

As the plan of record never involves putting the delay here, remove this TODO.

Maybe reword to something like:

"Actions are queued up until we receive a PROCESS action, at which point we'll
dispatch all queued events."

[lanwei, 2016/03/23 14:01:52]

Done.

+  // chromedrive_extension, here we do nothing for the delay. If the design
+  // changes, I will add a type for DELAY, see https://crbug.com/525187.
+  enum class PointerActionType {
+    NOT_INITIALIZED,
+    PRESS,
+    MOVE,
+    RELEASE,
+    PROCESS,
+    POINTER_ACTION_TYPE_MAX = PROCESS
+  };
+
+  SyntheticPointerActionParams();
+  SyntheticPointerActionParams(PointerActionType type);
+  SyntheticPointerActionParams(const SyntheticPointerActionParams& other);
+  ~SyntheticPointerActionParams() override;
+
+  GestureType GetGestureType() const override;
+
+  static const SyntheticPointerActionParams* Cast(
+      const SyntheticGestureParams* gesture_params);
+
+  void set_pointer_action_type(PointerActionType pointer_action_type) {
+    pointer_action_type_arg = pointer_action_type;
+  }
+
+  void set_index(int index) {
+    DCHECK(pointer_action_type_arg != PointerActionType::PROCESS);
+    index_arg = index;
+  }
+
+  void set_position(const gfx::PointF& position) {
+    DCHECK(pointer_action_type_arg == PointerActionType::PRESS ||
+           pointer_action_type_arg == PointerActionType::MOVE);
+    position_arg = position;
+  }
+
+  PointerActionType pointer_action_type() const {
+    return pointer_action_type_arg;
+  }
+
+  int index() const {
+    DCHECK(pointer_action_type_arg != PointerActionType::PROCESS);
+    return index_arg;
+  }
+
+  gfx::PointF position() const {
+    DCHECK(pointer_action_type_arg == PointerActionType::PRESS ||
+           pointer_action_type_arg == PointerActionType::MOVE);
+    return position_arg;
+  }
+
+ private:
+  friend struct IPC::ParamTraits<content::SyntheticPointerActionParams>;
+
+  PointerActionType pointer_action_type_arg;

[tdresser, 2016/03/22 14:38:10]

I'm not sure the |_arg| suffix is adding much value here.

These variables need to end in _, as they're private.

[lanwei, 2016/03/23 14:01:52]

I feel maybe it is confusing to have variable's name end with _ in the
input_message.h, but I guess it should be fine.

+  // Pass a position value when sending a press or move action.
+  gfx::PointF position_arg;
+  // Pass an index value except the pointer_action_type_arg is PROCESS.
+  int index_arg;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_INPUT_SYNTHETIC_POINTER_ACTION_PARAMS_H_