[chromedriver] Retry alert handling if HandleJavaScriptDialog fails.

chrome/test/chromedriver/chrome/javascript_dialog_manager.cc

Index: chrome/test/chromedriver/chrome/javascript_dialog_manager.cc
diff --git a/chrome/test/chromedriver/chrome/javascript_dialog_manager.cc b/chrome/test/chromedriver/chrome/javascript_dialog_manager.cc
index f95d5179f67a746f68054cf7e8cfd36b7a4b045e..c100f4a227125956e7774f2630f84c55f0ca9f1d 100644
--- a/chrome/test/chromedriver/chrome/javascript_dialog_manager.cc
+++ b/chrome/test/chromedriver/chrome/javascript_dialog_manager.cc
@@ -37,8 +37,11 @@ Status JavaScriptDialogManager::HandleDialog(bool accept,
   if (text)
     params.SetString("promptText", *text);
   Status status = client_->SendCommand("Page.handleJavaScriptDialog", params);
-  if (status.IsError())
-    return status;
+  if (status.IsError()) {

[samuong, 2016/11/16 22:53:34]

sorry, one more thing: can you add a comment here saying:

Retry once to work around
https://bugs.chromium.org/p/chromedriver/issues/detail?id=1500

[gmanikpure, 2016/11/16 23:03:42]

Done.

+    status = client_->SendCommand("Page.handleJavaScriptDialog", params);
+    if (status.IsError())
+      return status;
+  }
 
   // Remove a dialog from the queue. Need to check the queue is not empty here,
   // because it could have been cleared during waiting for the command