Replace mouse actions in pointer event tests with pointerActionSequence

content/renderer/gpu/actions_parser.cc

 // Copyright (c) 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.
 
 #include "content/renderer/gpu/actions_parser.h"
 
 #include "base/format_macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
@@ skipping 172 matching lines @@
                                         action_index_);
     return false;
   }
 
   if (action.HasKey("y") && !action.GetDouble("y", &position_y)) {
     error_message_ = base::StringPrintf("actions[%d].actions.y is not a number",
                                         action_index_);
     return false;
   }
 
+  int button = 1;
+  if (action.HasKey("button") && !action.GetInteger("button", &button)) {
+    error_message_ = base::StringPrintf(
+        "actions[%d].actions.button is not a number", action_index_);
+    return false;
+  }
+
   SyntheticPointerActionParams action_param(pointer_action_type);
   action_param.set_index(action_index_);
-  if (pointer_action_type ==
-          SyntheticPointerActionParams::PointerActionType::PRESS ||
-      pointer_action_type ==
-          SyntheticPointerActionParams::PointerActionType::MOVE) {
-    action_param.set_position(gfx::PointF(position_x, position_y));
+  switch (pointer_action_type) {
+    case SyntheticPointerActionParams::PointerActionType::PRESS:
+      action_param.set_position(gfx::PointF(position_x, position_y));
+      action_param.set_button(button);
+      break;
+    case SyntheticPointerActionParams::PointerActionType::MOVE:
+      action_param.set_position(gfx::PointF(position_x, position_y));
+      break;
+    case SyntheticPointerActionParams::PointerActionType::RELEASE:
+      action_param.set_button(button);

[Navid Zolghadr, 2017/01/12 18:23:46]

Is there any reason we don't set the x/y here similar to press?

[lanwei, 2017/01/13 17:29:30]

For SyntheticTouchDriver::Release and SyntheticMouseDriver::Release, we do not
need x/y values to dispatch a release event, and we do not request user to put
x/y for RELEASE action.

+      break;
+    case SyntheticPointerActionParams::PointerActionType::IDLE:
+    case SyntheticPointerActionParams::PointerActionType::NOT_INITIALIZED:
+      break;
   }
   param_list.push_back(action_param);
   return true;
 }
 
 }  // namespace content