Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Revert "Make VPA.setInterpolator(null) unset the interpolator."" into lmp-mr1-ub-dev

Browse Source
This commit is contained in:
Yigit Boyar
2015-05-19 18:40:36 +00:00
committed by Android (Google) Code Review
parent e7e9d17715 21cdbd298d
commit 9b7167937a
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
core/java/android/view/ViewPropertyAnimator.java
View File

@@ -82,11 +82,17 @@ public class ViewPropertyAnimator {
/** /**
* The interpolator of the underlying Animator object. By default, we don't set the interpolator * The interpolator of the underlying Animator object. By default, we don't set the interpolator
* on the Animator and just use its default interpolator. If the interpolator is set to a * on the Animator and just use its default interpolator. If the interpolator is ever set on
* non-null value on this Animator, then we use the interpolator that it was set to. * this Animator, then we use the interpolator that it was set to.
*/ */
private TimeInterpolator mInterpolator; private TimeInterpolator mInterpolator;
/**
* A flag indicating whether the interpolator has been set on this object. If not, we don't set
* the interpolator on the underlying Animator, but instead just use its default interpolator.
*/
private boolean mInterpolatorSet = false;
/** /**
* Listener for the lifecycle events of the underlying ValueAnimator object. * Listener for the lifecycle events of the underlying ValueAnimator object.
*/ */
@@ -334,6 +340,7 @@ public class ViewPropertyAnimator {
* @return This object, allowing calls to methods in this class to be chained. * @return This object, allowing calls to methods in this class to be chained.
*/ */
public ViewPropertyAnimator setInterpolator(TimeInterpolator interpolator) { public ViewPropertyAnimator setInterpolator(TimeInterpolator interpolator) {
mInterpolatorSet = true;
mInterpolator = interpolator; mInterpolator = interpolator;
return this; return this;
} }
@@ -344,7 +351,7 @@ public class ViewPropertyAnimator {
* @return The timing interpolator for this animation. * @return The timing interpolator for this animation.
*/ */
public TimeInterpolator getInterpolator() { public TimeInterpolator getInterpolator() {
if (mInterpolator != null) { if (mInterpolatorSet) {
return mInterpolator; return mInterpolator;
} else { } else {
// Just return the default from ValueAnimator, since that's what we'd get if // Just return the default from ValueAnimator, since that's what we'd get if
@@ -892,7 +899,7 @@ public class ViewPropertyAnimator {
if (mDurationSet) { if (mDurationSet) {
animator.setDuration(mDuration); animator.setDuration(mDuration);
} }
if (mInterpolator != null) { if (mInterpolatorSet) {
animator.setInterpolator(mInterpolator); animator.setInterpolator(mInterpolator);
} }
animator.start(); animator.start();