[chromedriver] Support clicking an element in sub frames.

chrome/test/chromedriver/basic_types.h

Index: chrome/test/chromedriver/basic_types.h
diff --git a/chrome/test/chromedriver/basic_types.h b/chrome/test/chromedriver/basic_types.h
index 8a623c481d3b944f3001b1a620defe9712170ee0..b217f05f6188b35280f4d35cec70cef21946e671 100644
--- a/chrome/test/chromedriver/basic_types.h
+++ b/chrome/test/chromedriver/basic_types.h
@@ -5,12 +5,14 @@
 #ifndef CHROME_TEST_CHROMEDRIVER_BASIC_TYPES_H_
 #define CHROME_TEST_CHROMEDRIVER_BASIC_TYPES_H_
 
+#include <string>
+
 struct WebPoint {
   WebPoint();
   WebPoint(int x, int y);
   ~WebPoint();
 
-  void offset(int x_, int y_);
+  void Offset(int x_, int y_);
 
   int x;
   int y;
@@ -31,13 +33,18 @@ struct WebRect {
   WebRect(const WebPoint& origin, const WebSize& size);
   ~WebRect();
 
-  int x();
-  int y();
-  int width();
-  int height();
+  int width() const;
+  int height() const;
 
   WebPoint origin;
   WebSize size;
 };
 
+struct FrameInfo {

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

i don't think i'd call this a basic type. how about move it to session.h?

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

can you think of a better name?

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

Done.

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

FrameElement.

+  FrameInfo(const std::string& parent_id, const std::string& xpath);
+
+  std::string parent_id;
+  std::string xpath;

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

at first read, it is a bit unclear what this refers to. can you think of a
better name

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

chromedriver_frame_id, holding the random value of attribute cd_frame_id_.

+};
+
 #endif  // CHROME_TEST_CHROMEDRIVER_BASIC_TYPES_H_