Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fa58a8da authored by Adam Powell's avatar Adam Powell
Browse files

Dispatch View#onVisibilityAggregated to views becoming invisible 

When a view becomes newly invisible/gone via setFlags, its visibility
state will have already changed by the time we
dispatchVisibilityAggregated. Since we already do visibility filtering
in the ViewGroup override when we traverse to child views, permit the
normal View implementation to dispatch onVisibilityAggregated if we're
not visible and dispatching visibility false.

Bug 28123146

Change-Id: I528dffe95d2057ef938508b9fb4219c5338b060c
parent f763ea3b

core/java/android/view/View.java

+2 −1
Original line number Diff line number Diff line
@@ -10278,7 +10278,8 @@ public class View implements Drawable.Callback, KeyEvent.Callback, 
     */

    @Visibility boolean dispatchVisibilityAggregated(boolean isVisible) {

        final boolean thisVisible = getVisibility() == VISIBLE;

        if (thisVisible) {

        // If we're not visible but something is telling us we are, ignore it.

        if (thisVisible || !isVisible) {

            onVisibilityAggregated(isVisible);

        }

        return thisVisible && isVisible;