[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..a8f03efafddd3f499ebfca909fed2cfc1988ce77 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"
@@ -21,6 +22,16 @@ class Chrome;
 class Status;
 class WebView;
 
+struct FrameElement {

[kkania, 2013/03/13 00:06:20]

actually, i think FrameInfo is better than FrameElement

[chrisgao (Use stgao instead), 2013/03/13 00:36:48]

Ok, take FrameInfo.

+  FrameElement(const std::string& parent_frame_id,
+               const std::string& frame_id,
+               const std::string& chromedriver_frame_id);
+
+  std::string parent_frame_id;
+  std::string frame_id;
+  std::string chromedriver_frame_id;
+};
+
 struct Session {
   explicit Session(const std::string& id);
   Session(const std::string& id, scoped_ptr<Chrome> chrome);
@@ -28,10 +39,18 @@ struct Session {
 
   Status GetTargetWindow(WebView** web_view);
 
+  void SwitchToTopFrame();
+  void SwitchToSubFrame(const std::string& frame_id,
+                        const std::string& chromedriver_frame_id);
+  std::string GetCurrentFrameId() const;
+
   const std::string id;
   scoped_ptr<Chrome> chrome;
   std::string window;
-  std::string frame;
+  // List of |FrameElement|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<FrameElement> frame_elements;
   WebPoint mouse_position;
   int implicit_wait;
   int page_load_timeout;