summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Daniel Kim <danieljkim@google.com> 2023-11-06 07:52:32 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2023-11-06 07:52:32 +0000
commitc737d35e0d6bc7f09007db6ecf4aecd5d10e690b (patch)
tree508e364d760b8a49a42f6e9726e7de1f5b2ee479
parent201a936f338f72b3f5b6c2003a059fd58ff36b90 (diff)
parentfc00db0af3504552e6069dd534234e53225e0e47 (diff)
Merge "Revert "Include invisible parent layouts in assistStructure"" into main
-rw-r--r--core/java/android/view/ViewGroup.java10
1 files changed, 1 insertions, 9 deletions
diff --git a/core/java/android/view/ViewGroup.java b/core/java/android/view/ViewGroup.java
index 5720fc01abce..287c7b29813f 100644
--- a/core/java/android/view/ViewGroup.java
+++ b/core/java/android/view/ViewGroup.java
@@ -3752,15 +3752,7 @@ public abstract class ViewGroup extends View implements ViewParent, ViewManager
&& !child.isActivityDeniedForAutofillForUnimportantView())
|| (shouldIncludeAllChildrenViewWithAutofillTypeNotNone(afm)
&& child.getAutofillType() != AUTOFILL_TYPE_NONE)
- || shouldIncludeAllChildrenViews(afm)
- || (child instanceof ViewGroup && child.getVisibility() != View.VISIBLE)) {
- // If the child is a ViewGroup object and its visibility is not visible, include
- // it as part of the assist structure. The children of these invisible ViewGroup
- // objects are parsed and included in the assist structure. When the Autofill
- // Provider determines the visibility of these children, it looks at their
- // visibility as well as their parent's visibility. Omitting invisible parents
- // will lead to the Autofill Provider incorrectly assuming that these children
- // of invisible parents are actually visible.
+ || shouldIncludeAllChildrenViews(afm)){
list.add(child);
} else if (child instanceof ViewGroup) {
((ViewGroup) child).populateChildrenForAutofill(list, flags);