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

Commit 0653f259 authored by Todd Volkert's avatar Todd Volkert
Browse files

Increase visibility of Transition.areValuesChanged() to protected to allow 

subclasses outside the android.transition package to override it.

Change-Id: Ib6a5502827e734de7494958b510afc9ddb80dffe
parent 1781b7ed

api/current.txt

+1 −0
Original line number Diff line number Diff line
@@ -31807,6 +31807,7 @@ package android.transition { 
    method public android.transition.Transition addTarget(java.lang.String);

    method public android.transition.Transition addTarget(java.lang.Class);

    method public android.transition.Transition addTarget(android.view.View);

    method protected boolean areValuesChanged(android.transition.TransitionValues, android.transition.TransitionValues);

    method public boolean canRemoveViews();

    method public abstract void captureEndValues(android.transition.TransitionValues);

    method public abstract void captureStartValues(android.transition.TransitionValues);

api/system-current.txt

+1 −0
Original line number Diff line number Diff line
@@ -33966,6 +33966,7 @@ package android.transition { 
    method public android.transition.Transition addTarget(java.lang.String);

    method public android.transition.Transition addTarget(java.lang.Class);

    method public android.transition.Transition addTarget(android.view.View);

    method protected boolean areValuesChanged(android.transition.TransitionValues, android.transition.TransitionValues);

    method public boolean canRemoveViews();

    method public abstract void captureEndValues(android.transition.TransitionValues);

    method public abstract void captureStartValues(android.transition.TransitionValues);

core/java/android/transition/Transition.java

+11 −1
Original line number Diff line number Diff line
@@ -1762,7 +1762,17 @@ public abstract class Transition implements Cloneable { 
        runAnimators();

    }



    boolean areValuesChanged(TransitionValues oldValues, TransitionValues newValues) {

    /**

     * Returns whether transition values have changed between the start scene and the end scene

     * (thus determining whether animation is required). The default implementation compares the

     * property values returned from {@link #getTransitionProperties()}, or all property values if

     * {@code getTransitionProperties()} returns null. Subclasses may override this method to

     * provide logic more specific to their transition implementation.

     *

     * @param oldValues the first set of values, may be {@code null}

     * @param newValues the second set of values, may be {@code null}

     */

    protected boolean areValuesChanged(TransitionValues oldValues, TransitionValues newValues) {

        boolean valuesChanged = false;

        // if oldValues null, then transition didn't care to stash values,

        // and won't get canceled

core/java/android/transition/Visibility.java

+2 −2
Original line number Diff line number Diff line
@@ -182,7 +182,7 @@ public abstract class Visibility extends Transition { 
        return visibility == View.VISIBLE && parent != null;

    }



    private VisibilityInfo getVisibilityChangeInfo(TransitionValues startValues,

    private static VisibilityInfo getVisibilityChangeInfo(TransitionValues startValues,

            TransitionValues endValues) {

        final VisibilityInfo visInfo = new VisibilityInfo();

        visInfo.visibilityChange = false;
@@ -484,7 +484,7 @@ public abstract class Visibility extends Transition { 
    }



    @Override

    boolean areValuesChanged(TransitionValues oldValues, TransitionValues newValues) {

    protected boolean areValuesChanged(TransitionValues oldValues, TransitionValues newValues) {

        if (oldValues == null && newValues == null) {

            return false;

        }