Add GetScreenBoundsRect to ui::AXPlatformNodeDelegate

ui/accessibility/platform/ax_platform_node_win.cc

 // Copyright 2015 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 <atlbase.h>
 #include <atlcom.h>
 #include <limits.h>
 #include <oleacc.h>
 #include <stdint.h>
 
@@ skipping 243 matching lines @@
 HRESULT AXPlatformNodeWin::accDoDefaultAction(VARIANT var_id) {
   COM_OBJECT_VALIDATE_VAR_ID(var_id);
   delegate_->DoDefaultAction();
   return S_OK;
 }
 
 STDMETHODIMP AXPlatformNodeWin::accLocation(
     LONG* x_left, LONG* y_top, LONG* width, LONG* height, VARIANT var_id) {
   COM_OBJECT_VALIDATE_VAR_ID_4_ARGS(var_id, x_left, y_top, width, height);
   gfx::Rect bounds = gfx::ToEnclosingRect(GetData().location);
-  bounds += delegate_->GetGlobalCoordinateOffset();
-  *x_left = bounds.x();
-  *y_top  = bounds.y();
+  gfx::Rect screen_bounds = delegate_->GetScreenBoundsRect();

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

Same here - it should just call GetScreenBoundsRect and no longer check
location.

+
+  *x_left = bounds.x() + screen_bounds.x();
+  *y_top = bounds.y() + screen_bounds.y();
   *width  = bounds.width();
   *height = bounds.height();
 
   if (bounds.IsEmpty())
     return S_FALSE;
 
   return S_OK;
 }
 
 STDMETHODIMP AXPlatformNodeWin::accNavigate(
@@ skipping 915 matching lines @@
     ui::TextBoundaryDirection direction) {
   HandleSpecialTextOffset(&start_offset);
   ui::TextBoundaryType boundary = IA2TextBoundaryToTextBoundary(ia2_boundary);
   std::vector<int32_t> line_breaks;
   return static_cast<LONG>(ui::FindAccessibleTextBoundary(
       text, line_breaks, boundary, start_offset, direction,
       AX_TEXT_AFFINITY_DOWNSTREAM));
 }
 
 }  // namespace ui