 Chromium Code Reviews
 Chromium Code Reviews Issue 2795543002:
  Add GetScreenBoundsRect to ui::AXPlatformNodeDelegate  (Closed)
    
  
    Issue 2795543002:
  Add GetScreenBoundsRect to ui::AXPlatformNodeDelegate  (Closed) 
  | Index: ui/accessibility/platform/ax_platform_node_win.cc | 
| diff --git a/ui/accessibility/platform/ax_platform_node_win.cc b/ui/accessibility/platform/ax_platform_node_win.cc | 
| index d811182f2f17fceaac6f803e3c159df170953c28..ae49c4cd2d84dfbe52e2b53bc788d0efdda3bdae 100644 | 
| --- a/ui/accessibility/platform/ax_platform_node_win.cc | 
| +++ b/ui/accessibility/platform/ax_platform_node_win.cc | 
| @@ -261,9 +261,10 @@ 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 GetScreenBoundsRec
 | 
| + | 
| + *x_left = bounds.x() + screen_bounds.x(); | 
| + *y_top = bounds.y() + screen_bounds.y(); | 
| *width = bounds.width(); | 
| *height = bounds.height(); |