Simplify TestRunnerBindings::SimulateWebNotificationClick param handling

components/test_runner/test_runner.cc

 // Copyright 2014 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 "components/test_runner/test_runner.h"
 
 #include <stddef.h>
 #include <limits>
 #include <utility>
 
@@ skipping 277 matching lines @@
   void SetDatabaseQuota(int quota);
   void SetAlwaysAcceptCookies(bool accept);
   void SetWindowIsKey(bool value);
   std::string PathToLocalResource(const std::string& path);
   void SetBackingScaleFactor(double value, v8::Local<v8::Function> callback);
   void EnableUseZoomForDSF(v8::Local<v8::Function> callback);
   void SetColorProfile(const std::string& name,
                        v8::Local<v8::Function> callback);
   void SetPOSIXLocale(const std::string& locale);
   void SetMIDIAccessorResult(bool result);
-  void SimulateWebNotificationClick(gin::Arguments* args);
+  void SimulateWebNotificationClick(const std::string& title, int action_index);
   void AddMockSpeechRecognitionResult(const std::string& transcript,
                                       double confidence);
   void SetMockSpeechRecognitionError(const std::string& error,
                                      const std::string& message);
   bool WasMockSpeechRecognitionAborted();
   void AddMockCredentialManagerResponse(const std::string& id,
                                         const std::string& name,
                                         const std::string& avatar,
                                         const std::string& password);
   void AddWebPageOverlay();
@@ skipping 1097 matching lines @@
 void TestRunnerBindings::SetPOSIXLocale(const std::string& locale) {
   if (runner_)
     runner_->SetPOSIXLocale(locale);
 }
 
 void TestRunnerBindings::SetMIDIAccessorResult(bool result) {
   if (runner_)
     runner_->SetMIDIAccessorResult(result);
 }
 
-void TestRunnerBindings::SimulateWebNotificationClick(gin::Arguments* args) {
+void TestRunnerBindings::SimulateWebNotificationClick(const std::string& title,
+                                                      int action_index) {
   if (!runner_)
     return;
-  std::string title;
-  int action_index = -1;
-  if (args->GetNext(&title) &&
-      (args->PeekNext().IsEmpty() || args->GetNext(&action_index)))
-    runner_->SimulateWebNotificationClick(title, action_index);
-  else
-    args->ThrowError();
+  runner_->SimulateWebNotificationClick(title, action_index);
 }
 
 void TestRunnerBindings::AddMockSpeechRecognitionResult(
     const std::string& transcript, double confidence) {
   if (runner_)
     runner_->AddMockSpeechRecognitionResult(transcript, confidence);
 }
 
 void TestRunnerBindings::SetMockSpeechRecognitionError(
     const std::string& error, const std::string& message) {
@@ skipping 1782 matching lines @@
 }
 
 void TestRunner::DidLosePointerLockInternal() {
   bool was_locked = pointer_locked_;
   pointer_locked_ = false;
   if (was_locked)
     web_view_->didLosePointerLock();
 }
 
 }  // namespace test_runner