Add GetScreenBoundsRect to ui::AXPlatformNodeDelegate

ui/accessibility/platform/ax_platform_node_base.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 "ui/accessibility/platform/ax_platform_node_base.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/accessibility/ax_action_data.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/accessibility/platform/ax_platform_node_delegate.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 
 namespace ui {
 
 void AXPlatformNodeBase::Init(AXPlatformNodeDelegate* delegate) {
   delegate_ = delegate;
 }
 
 const AXNodeData& AXPlatformNodeBase::GetData() const {
   CHECK(delegate_);
   return delegate_->GetData();
 }
 
 gfx::Rect AXPlatformNodeBase::GetBoundsInScreen() const {
   CHECK(delegate_);
   gfx::Rect bounds = gfx::ToEnclosingRect(GetData().location);
-  bounds.Offset(delegate_->GetGlobalCoordinateOffset());
+  gfx::Rect screen_bounds = delegate_->GetScreenBoundsRect();
+  bounds.Offset(screen_bounds.x(), screen_bounds.y());

[dmazzoni, 2017/04/04 15:05:19]

I don't think this should be here.

It should just call delegate_->GetScreenBoundsRect(), nothing more should be
needed.

That iframe transform example would fail here - the width and height would be
incorrect at minimum.

   return bounds;
 }
 
 gfx::NativeViewAccessible AXPlatformNodeBase::GetParent() {
   CHECK(delegate_);
   return delegate_->GetParent();
 }
 
 int AXPlatformNodeBase::GetChildCount() {
   CHECK(delegate_);
@@ skipping 171 matching lines @@
   ui::AXActionData action_data;
   action_data.action = ui::AX_ACTION_SET_SELECTION;
   action_data.anchor_node_id = action_data.focus_node_id = GetData().id;
   action_data.anchor_offset = start_offset;
   action_data.focus_offset = end_offset;
   DCHECK(delegate_);
   return delegate_->AccessibilityPerformAction(action_data);
 }
 
 }  // namespace ui