[chromedriver] Support clicking an element in sub frames.

chrome/test/chromedriver/basic_types.h

 // Copyright (c) 2013 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_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;
 };
 
 struct WebSize {
   WebSize();
   WebSize(int width, int height);
   ~WebSize();
 
   int width;
   int height;
 };
 
 struct WebRect {
   WebRect();
   WebRect(int x, int y, int width, int height);
   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_