EventSender::GestureFlingStart requires a source device.

This is the third of three patches.

The first allowed passing a source device to EventSender:
https://codereview.chromium.org/902173003

The second passed a source device to EventSender:
https://codereview.chromium.org/901323004

This patch requires passing a source device to event sender.

BUG=456136

Review URL: https://codereview.chromium.org/934883002

Cr-Commit-Position: refs/heads/master@{#316812}

1 files changed, 3 insertions, 10 deletions

diff --git a/content/shell/renderer/test_runner/event_sender.cc b/content/shell/renderer/test_runner/event_sender.cc
index 4352db3..1960be9 100644
--- a/content/shell/renderer/test_runner/event_sender.cc
+++ b/content/shell/renderer/test_runner/event_sender.cc
@@ -1661,16 +1661,9 @@ void EventSender::GestureFlingStart(float x,
   WebGestureEvent event;
   event.type = WebInputEvent::GestureFlingStart;
 
-  // TODO(tdresser): Once we've migrated all calls to GestureFlingStart
-  // to pass the device string, throw an error if args is empty. See
-  // crbug.com/456136 for details.
-  std::string device_string = kSourceDeviceStringTouchpad;
-  if (!args->PeekNext().IsEmpty() && args->PeekNext()->IsString()) {
-    if (!args->GetNext(&device_string)) {
-      args->ThrowError();
-      return;
-    }
-  }
+  std::string device_string;
+  if (!args->PeekNext().IsEmpty() && args->PeekNext()->IsString())
+    args->GetNext(&device_string);
 
   if (device_string == kSourceDeviceStringTouchpad) {
     event.sourceDevice = blink::WebGestureDeviceTouchpad;