Forward BrowserAccessibility::accHitTest to AXPlatformNode.

content/browser/accessibility/browser_accessibility_com_win.cc

 // Copyright (c) 2017 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 "content/browser/accessibility/browser_accessibility_com_win.h"
 
 #include <UIAutomationClient.h>
 #include <UIAutomationCoreApi.h>
 
 #include <algorithm>
@@ skipping 423 matching lines @@
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::accHitTest(LONG x_left,
                                                     LONG y_top,
                                                     VARIANT* child) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_ACC_HIT_TEST);
 
   if (!owner())
     return E_FAIL;
 
-  auto* manager = Manager();
-  if (!manager)
-    return E_FAIL;
-
-  if (!child)
-    return E_INVALIDARG;
-
-  gfx::Point point(x_left, y_top);
-  if (!owner()->GetScreenBoundsRect().Contains(point)) {

[dmazzoni, 2017/06/23 18:51:12]

Where did this check go? It looks like it was lost

[dougt, 2017/06/25 21:16:31]

Done.

-    // Return S_FALSE and VT_EMPTY when outside the object's boundaries.
-    child->vt = VT_EMPTY;
-    return S_FALSE;
-  }
-
-  BrowserAccessibility* result = manager->CachingAsyncHitTest(point);
-  if (result == owner()) {
-    // Point is within this object.
-    child->vt = VT_I4;
-    child->lVal = CHILDID_SELF;
-  } else {
-    child->vt = VT_DISPATCH;
-    child->pdispVal = ToBrowserAccessibilityComWin(result)->NewReference();
-  }
-  return S_OK;
+  return AXPlatformNodeWin::accHitTest(x_left, y_top, child);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::accLocation(LONG* x_left,
                                                      LONG* y_top,
                                                      LONG* width,
                                                      LONG* height,
                                                      VARIANT var_id) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_ACC_LOCATION);
   if (!owner())
     return E_FAIL;
@@ skipping 5105 matching lines @@
 
 BrowserAccessibilityComWin* ToBrowserAccessibilityComWin(
     BrowserAccessibility* obj) {
   if (!obj || !obj->IsNative())
     return nullptr;
   auto* result = static_cast<BrowserAccessibilityWin*>(obj)->GetCOM();
   return result;
 }
 
 }  // namespace content