[Chromedriver] Chromedriver should handle unexpected alert automatically.

chrome/test/chromedriver/window_commands.cc

 // Copyright (c) 2013 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 "chrome/test/chromedriver/window_commands.h"
 
 #include <stddef.h>
 
 #include <list>
 #include <string>
@@ skipping 202 matching lines @@
   status = web_view->ConnectIfNecessary();
   if (status.IsError())
     return status;
 
   status = web_view->HandleReceivedEvents();
   if (status.IsError())
     return status;
 
   if (web_view->GetJavaScriptDialogManager()->IsDialogOpen()) {
     std::string alert_text;
+    std::string alert_behaviour = session->chrome->UnexpectedAlertBehaviour();

[samuong, 2016/11/02 22:48:48]

nit: do this just before the if statement on line 231

[gmanikpure, 2016/11/04 00:23:24]

Done.

     status =
         web_view->GetJavaScriptDialogManager()->GetDialogMessage(&alert_text);
     if (status.IsError())
       return status;
+
+    // Close the dialog depending on the unexpectedalert behaviour set by user
+    // before returning an error, so that subsequent commands do not fail.
+    if (alert_behaviour == "accept")
+      status = web_view->GetJavaScriptDialogManager()
+          ->HandleDialog(true, session->prompt_text.get());
+    else if (alert_behaviour == "dismiss")
+      status = web_view->GetJavaScriptDialogManager()
+          ->HandleDialog(false, session->prompt_text.get());

[samuong, 2016/11/02 22:48:48]

does "ignore" mean "don't throw an error", or should we still be throwing an
unexpected alert error?

[gmanikpure, 2016/11/04 00:23:24]

As discussed, we will be throwing unexpected alert error for ignore value.

+    if (status.IsError())
+      return status;
+
     return Status(kUnexpectedAlertOpen, "{Alert text : " + alert_text + "}");
   }
 
   Status nav_status(kOk);
   for (int attempt = 0; attempt < 3; attempt++) {
     if (attempt == 2) {
       // Switch to main frame and retry command if subframe no longer exists.
       session->SwitchToTopFrame();
     }
 
@@ skipping 827 matching lines @@
   return status;
 }
 
 Status ExecuteTakeHeapSnapshot(Session* session,
                                WebView* web_view,
                                const base::DictionaryValue& params,
                                std::unique_ptr<base::Value>* value,
                                Timeout* timeout) {
   return web_view->TakeHeapSnapshot(value);
 }