Make SyntheticPointerAction to flush the pointer action sequence

content/browser/renderer_host/input/synthetic_mouse_driver.cc

Index: content/browser/renderer_host/input/synthetic_mouse_driver.cc
diff --git a/content/browser/renderer_host/input/synthetic_mouse_driver.cc b/content/browser/renderer_host/input/synthetic_mouse_driver.cc
index 82dcbdd8a413df55d939127dc47de124f90d9822..2541388d63b25a097571488efc2975c4e8b22b6c 100644
--- a/content/browser/renderer_host/input/synthetic_mouse_driver.cc
+++ b/content/browser/renderer_host/input/synthetic_mouse_driver.cc
@@ -18,11 +18,10 @@ void SyntheticMouseDriver::DispatchEvent(SyntheticGestureTarget* target,
   target->DispatchInputEventToPlatform(mouse_event_);
 }
 
-int SyntheticMouseDriver::Press(float x, float y) {
+void SyntheticMouseDriver::Press(float x, float y, int index) {
   mouse_event_ = SyntheticWebMouseEventBuilder::Build(

[tdresser, 2016/12/12 15:32:49]

Why did we switch from returning the index to passing it in? Doesn't that make
the API more difficult to use?

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

Acknowledged.

       blink::WebInputEvent::MouseDown, x, y, 0);
   mouse_event_.clickCount = 1;
-  return 0;
 }
 
 void SyntheticMouseDriver::Move(float x, float y, int index) {
@@ -44,7 +43,7 @@ void SyntheticMouseDriver::Release(int index) {
 
 bool SyntheticMouseDriver::UserInputCheck(
     const SyntheticPointerActionParams& params) const {
-  if (params.gesture_source_type != SyntheticGestureParams::MOUSE_INPUT)
+  if (params.gesture_source_type() != SyntheticGestureParams::MOUSE_INPUT)
     return false;
 
   if (params.pointer_action_type() ==