[chromedriver] Support clicking an element in sub frames.

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 f7bc2e2cb1c3b401b0041fe906ca0788f4fe19c6..5c5ba6d57e6817d0d340dd9108c1ad0734a66350 100644
--- a/chrome/test/chromedriver/session.h
+++ b/chrome/test/chromedriver/session.h
@@ -5,6 +5,7 @@
 #ifndef CHROME_TEST_CHROMEDRIVER_SESSION_H_
 #define CHROME_TEST_CHROMEDRIVER_SESSION_H_
 
+#include <list>
 #include <string>
 
 #include "base/basictypes.h"
@@ -28,10 +29,16 @@ struct Session {
 
   Status GetTargetWindow(WebView** web_view);
 
+  void SetToFrame(const std::string& new_frame, const std::string& xpath);
+
   const std::string id;
   scoped_ptr<Chrome> chrome;
   std::string window;
   std::string frame;

[kkania, 2013/03/12 18:34:25]

this is a bit messy now that the client must remember to keep frame_elements and
frame in sync. Fix this.

[chrisgao (Use stgao instead), 2013/03/12 23:17:41]

Remove frame, and only maintain frame_elements.

+  // Vector of |FrameInfo|s for each frame to the current target frame from the
+  // first frame element in the root document. If target frame is window.top,
+  // this list will be empty.
+  std::list<FrameInfo> frame_elements;
   WebPoint mouse_position;
   int implicit_wait;
   int page_load_timeout;