Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(856)

Unified Diff: content/browser/accessibility/browser_accessibility.cc

Issue 2948513002: Forward BrowserAccessibility::accHitTest to AXPlatformNode. (Closed)
Patch Set: Adjust the test to match the spec Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/accessibility/browser_accessibility.cc
diff --git a/content/browser/accessibility/browser_accessibility.cc b/content/browser/accessibility/browser_accessibility.cc
index a90d107f32d0658e18bcae416008ec3b43ecbd8b..7cad16dd6e5dbd0f5092ebf88c32e29ecfa7377d 100644
--- a/content/browser/accessibility/browser_accessibility.cc
+++ b/content/browser/accessibility/browser_accessibility.cc
@@ -1130,8 +1130,11 @@ gfx::Rect BrowserAccessibility::GetScreenBoundsRect() const {
}
gfx::NativeViewAccessible BrowserAccessibility::HitTestSync(int x, int y) {
- NOTREACHED();
- return nullptr;
+ auto* accessible = ApproximateHitTest(gfx::Point(x, y));
dmazzoni 2017/06/23 18:51:12 This patch would replace CachingAsyncHitTest (whic
dougt 2017/06/25 21:16:31 Done.
+ if (!accessible)
+ return nullptr;
+
+ return accessible->GetNativeViewAccessible();
}
gfx::NativeViewAccessible BrowserAccessibility::GetFocus() {

Powered by Google App Engine
This is Rietveld 408576698