From a1ac6a09252a8a615230f08767ab6153bd23a165 Mon Sep 17 00:00:00 2001 From: Svetoslav Date: Fri, 23 Jan 2015 16:06:43 -0800 Subject: Accessibility: Ignore overlapping siblings when computing a click location To click a view we were computing a click location by ignoring overlapping views that are actionable. However, detection whether a view is actionable is not always possible as the view may handle touch events directly. This leads to unhandled edge cases. We are taking a conservative approach and ignore all overlapping siblings regardless if clickable. This is also has limitations but hopefully less frequent edge cases. bug:18889611 Change-Id: Icea0b7b3e2d4ed53e50e01cb6a99b880be560b14 --- .../java/com/android/internal/widget/ActionBarContainer.java | 12 ------------ 1 file changed, 12 deletions(-) (limited to 'core/java/com') diff --git a/core/java/com/android/internal/widget/ActionBarContainer.java b/core/java/com/android/internal/widget/ActionBarContainer.java index e8e2c8d..7937a95 100644 --- a/core/java/com/android/internal/widget/ActionBarContainer.java +++ b/core/java/com/android/internal/widget/ActionBarContainer.java @@ -227,18 +227,6 @@ public class ActionBarContainer extends FrameLayout { return true; } - /** - * @hide - */ - @Override - public void addClickableRectsForAccessibility(List outRects) { - // This class always consumes touch events, therefore if it - // covers a view we do not want to send a click over it. - RectF bounds = new RectF(); - bounds.set(0, 0, getWidth(), getHeight()); - outRects.add(bounds); - } - @Override public boolean onHoverEvent(MotionEvent ev) { super.onHoverEvent(ev); -- cgit v1.1