[chromedriver] Support clicking an element in sub frames.

chrome/test/chromedriver/session.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_TEST_CHROMEDRIVER_SESSION_H_
 #define CHROME_TEST_CHROMEDRIVER_SESSION_H_
 
+#include <list>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/lock.h"
 #include "chrome/test/chromedriver/basic_types.h"
 
 namespace base {
 class DictionaryValue;
 }
 
 class Chrome;
 class Status;
 class WebView;
 
 struct Session {
   explicit Session(const std::string& id);
   Session(const std::string& id, scoped_ptr<Chrome> chrome);
   ~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;
   int script_timeout;
   std::string prompt_text;
   const scoped_ptr<base::DictionaryValue> capabilities;
 
  private:
   scoped_ptr<base::DictionaryValue> CreateCapabilities();
 };
@@ skipping 16 matching lines @@
  private:
   virtual ~SessionAccessorImpl();
 
   base::Lock session_lock_;
   scoped_ptr<Session> session_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionAccessorImpl);
 };
 
 #endif  // CHROME_TEST_CHROMEDRIVER_SESSION_H_