Add Vector2d classes that represent offsets, instead of using Point.

chrome/browser/automation/testing_automation_provider_gtk.cc

Index: chrome/browser/automation/testing_automation_provider_gtk.cc
diff --git a/chrome/browser/automation/testing_automation_provider_gtk.cc b/chrome/browser/automation/testing_automation_provider_gtk.cc
index 83f98433276d36270208a0dbaa80a368cd020f80..6851c972658efdb6a24857d0ce3aa342b85e4707 100644
--- a/chrome/browser/automation/testing_automation_provider_gtk.cc
+++ b/chrome/browser/automation/testing_automation_provider_gtk.cc
@@ -37,9 +37,9 @@ void TestingAutomationProvider::WindowGetViewBounds(int handle,
     *bounds = gfx::Rect(allocation.width, allocation.height);
     gint x, y;
     if (screen_coordinates) {
-      gfx::Point point = ui::GetWidgetScreenPosition(widget);
-      x = point.x();
-      y = point.y();
+      gfx::Vector2d offset = ui::GetWidgetScreenOffset(widget);
+      x = offset.x();
+      y = offset.y();

[Peter Kasting, 2012/10/27 01:01:39]

Nit: This causes us to translate back and forth from gint.  How about:

    if (screen_coordinates) {
      bounds->set_origin(gfx::Point() + ui::GetWidgetScreenOffset(widget));
    } else {
      gint x, y;
      gtk_widget_translate_coordinates(widget, GTK_WIDGET(window), 0, 0, &x,
                                       &y);
      bounds->set_origin(gfx::Point(x, y));
    }

[danakj, 2012/10/29 19:17:20]

Done.

     } else {
       gtk_widget_translate_coordinates(widget, GTK_WIDGET(window),
                                        0, 0, &x, &y);