Automation API should use transforms for iframe coordinates.

chrome/renderer/extensions/automation_internal_custom_bindings.cc

 // Copyright 2014 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.
 
 #include "chrome/renderer/extensions/automation_internal_custom_bindings.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
@@ skipping 97 matching lines @@
     bounds.Union(child_bounds);
   }
 
   return bounds;
 }
 
 // Compute the bounding box of a node in global coordinates, walking up the
 // parent hierarchy to offset by frame offsets and scroll offsets.
 static gfx::Rect ComputeGlobalNodeBounds(TreeCache* cache, ui::AXNode* node) {
   gfx::Rect bounds = ComputeLocalNodeBounds(cache, node);
-  ui::AXNode* parent = node->parent();
-  bool need_to_offset_web_area = node->data().role == ui::AX_ROLE_WEB_AREA ||
-                                 node->data().role == ui::AX_ROLE_ROOT_WEB_AREA;
-  while (parent) {
-    if (bounds.IsEmpty()) {
-      bounds = parent->data().location;
-    } else if (need_to_offset_web_area && parent->data().location.width() > 0 &&
-               parent->data().location.height() > 0) {
-      bounds.Offset(parent->data().location.x(), parent->data().location.y());
-      need_to_offset_web_area = false;
+
+  ui::AXNode* root = cache->tree.root();
+  while (root) {
+    // Apply scroll offsets.
+    if (root != node) {
+      int sx = 0;
+      int sy = 0;
+      if (root->data().GetIntAttribute(ui::AX_ATTR_SCROLL_X, &sx) &&
+          root->data().GetIntAttribute(ui::AX_ATTR_SCROLL_Y, &sy)) {
+        bounds.Offset(-sx, -sy);
+      }
     }
 
-    if (parent->data().role == ui::AX_ROLE_WEB_AREA ||
-        parent->data().role == ui::AX_ROLE_ROOT_WEB_AREA) {
-      int sx = 0;
-      int sy = 0;
-      if (parent->data().GetIntAttribute(ui::AX_ATTR_SCROLL_X, &sx) &&
-          parent->data().GetIntAttribute(ui::AX_ATTR_SCROLL_Y, &sy)) {
-        bounds.Offset(-sx, -sy);
-      }
-      need_to_offset_web_area = true;
+    if (root->data().transform) {
+      gfx::RectF boundsf(bounds);
+      root->data().transform->TransformRect(&boundsf);
+      bounds = gfx::Rect(boundsf.x(), boundsf.y(), boundsf.width(),
+                         boundsf.height());
     }
-    parent = cache->owner->GetParent(parent, &cache);
+
+    ui::AXNode* parent = cache->owner->GetParent(root, &cache);
+    if (!parent)
+      break;
+
+    root = cache->tree.root();
   }
 
   return bounds;
 }
 
 ui::AXNode* FindNodeWithChildTreeId(ui::AXNode* node, int child_tree_id) {
   if (child_tree_id == node->data().GetIntAttribute(ui::AX_ATTR_CHILD_TREE_ID))
     return node;
 
   for (int i = 0; i < node->child_count(); ++i) {
@@ skipping 1102 matching lines @@
   v8::Local<v8::Array> args(v8::Array::New(GetIsolate(), 2U));
   args->Set(0U, v8::Integer::New(GetIsolate(), tree_id));
   v8::Local<v8::Array> nodes(v8::Array::New(GetIsolate(), ids.size()));
   args->Set(1U, nodes);
   for (size_t i = 0; i < ids.size(); ++i)
     nodes->Set(i, v8::Integer::New(GetIsolate(), ids[i]));
   context()->DispatchEvent("automationInternal.onNodesRemoved", args);
 }
 
 }  // namespace extensions