From 9a7444cec53a51bfc374cef430d39d4dfdd2c784 Mon Sep 17 00:00:00 2001 From: Robert Horvath Date: Wed, 23 Feb 2022 19:06:34 +0100 Subject: [PATCH] Keep custom keep-clear Rects if preferKeepClear is set Since ViewParents can be set to not clip their children, custom keep clear areas can extend beyond the bounds of its hosting View. If preferKeepClear is set, add the View's bounds as a keep clear area, but also keep the custom Rects. Bug: 221073574 Test: atest KeepClearRectsTests Change-Id: I85a4e4a223fbba787c5afebf2689a031243808d4 --- core/java/android/view/View.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/core/java/android/view/View.java b/core/java/android/view/View.java index 8b3a29a7aa6f4..b49e075cb710d 100644 --- a/core/java/android/view/View.java +++ b/core/java/android/view/View.java @@ -11807,8 +11807,8 @@ public class View implements Drawable.Callback, KeyEvent.Callback, *

* The system will try to respect this preference, but when not possible will ignore it. *

- * Note: while this is set to {@code true}, the system will ignore the {@code Rect}s provided - * through {@link #setPreferKeepClearRects} (but not clear them). + * Note: This is independent from {@link #setPreferKeepClearRects}. If both are set, both will + * be taken into account. *

* @see #setPreferKeepClearRects * @see #isPreferKeepClear @@ -11842,8 +11842,8 @@ public class View implements Drawable.Callback, KeyEvent.Callback, *

* The system will try to respect this preference, but when not possible will ignore it. *

- * Note: While {@link #isPreferKeepClear} is {@code true}, the {@code Rect}s set here are - * ignored. + * Note: This is independent from {@link #setPreferKeepClear}. If both are set, both will be + * taken into account. *

* @see #setPreferKeepClear * @see #getPreferKeepClearRects @@ -11939,7 +11939,8 @@ public class View implements Drawable.Callback, KeyEvent.Callback, final List list = new ArrayList(); if (info.mPreferKeepClear) { list.add(new Rect(0, 0, getWidth(), getHeight())); - } else if (info.mKeepClearRects != null) { + } + if (info.mKeepClearRects != null) { list.addAll(info.mKeepClearRects); } return list;