[Chromedriver] Chromedriver should handle unexpected alert automatically.

chrome/test/chromedriver/session.h

Index: chrome/test/chromedriver/session.h
diff --git a/chrome/test/chromedriver/session.h b/chrome/test/chromedriver/session.h
index b582212bf16a656cc5498a839271952591482629..cee525a89292eeb184c736b0beecebfce54472c9 100644
--- a/chrome/test/chromedriver/session.h
+++ b/chrome/test/chromedriver/session.h
@@ -86,6 +86,11 @@ struct Session {
   // |CommandListener|s might be |CommandListenerProxy|s that forward to
   // |DevToolsEventListener|s owned by |chrome|.
   ScopedVector<CommandListener> command_listeners;
+  std::string unexpected_alert_behaviour;
+  static const char kAccept[];

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

As discussed, adding values of unexpectedalertbehaviour as constants in
session.h/session.cc.

[samuong, 2016/11/22 19:52:16]

these don't need to be inside the Session class, you can put them at the top
level instead

[gmanikpure, 2016/11/23 18:34:26]

Done.

+  static const char kDismiss[];
+  static const char kIgnore[];
+  std::string GetUnexpectedAlertBehaviour() const;

[samuong, 2016/11/22 19:52:16]

this is a method, so group it with the other methods, rather than with the
variables

[gmanikpure, 2016/11/23 18:34:26]

Done.

 };
 
 Session* GetThreadLocalSession();