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

Unified Diff: ui/accessibility/platform/test_ax_node_wrapper.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: ui/accessibility/platform/test_ax_node_wrapper.cc
diff --git a/ui/accessibility/platform/test_ax_node_wrapper.cc b/ui/accessibility/platform/test_ax_node_wrapper.cc
index 22766403d3de3e1ece8557904a0fd7f89937c9cc..2657f8372b24b7bd1a617c2b54fd85d479bdb444 100644
--- a/ui/accessibility/platform/test_ax_node_wrapper.cc
+++ b/ui/accessibility/platform/test_ax_node_wrapper.cc
@@ -112,7 +112,11 @@ gfx::Rect TestAXNodeWrapper::GetScreenBoundsRect() const {
}
gfx::NativeViewAccessible TestAXNodeWrapper::HitTestSync(int x, int y) {
- return nullptr;
+ gfx::RectF bounds = GetData().location;
+ if (!bounds.Contains(x, y))
dmazzoni 2017/06/23 18:51:12 Let's have this iterate over children too, to do a
dougt 2017/06/25 21:16:32 SG. This impl isn't perfect and will not be able
+ return nullptr;
+
+ return ax_platform_node()->GetNativeViewAccessible();
}
gfx::NativeViewAccessible TestAXNodeWrapper::GetFocus() {

Powered by Google App Engine
This is Rietveld 408576698