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

This commit is contained in:
Vladislav Kaznacheev
2015-05-18 23:32:55 +00:00
committed by Android (Google) Code Review

View File

@ -82,17 +82,11 @@ public class ViewPropertyAnimator {
/**
* 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 ever set on
* this Animator, then we use the interpolator that it was set to.
* on the Animator and just use its default interpolator. If the interpolator is set to a
* non-null value on this Animator, then we use the interpolator that it was set to.
*/
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.
*/
@ -340,7 +334,6 @@ public class ViewPropertyAnimator {
* @return This object, allowing calls to methods in this class to be chained.
*/
public ViewPropertyAnimator setInterpolator(TimeInterpolator interpolator) {
mInterpolatorSet = true;
mInterpolator = interpolator;
return this;
}
@ -351,7 +344,7 @@ public class ViewPropertyAnimator {
* @return The timing interpolator for this animation.
*/
public TimeInterpolator getInterpolator() {
if (mInterpolatorSet) {
if (mInterpolator != null) {
return mInterpolator;
} else {
// Just return the default from ValueAnimator, since that's what we'd get if
@ -899,7 +892,7 @@ public class ViewPropertyAnimator {
if (mDurationSet) {
animator.setDuration(mDuration);
}
if (mInterpolatorSet) {
if (mInterpolator != null) {
animator.setInterpolator(mInterpolator);
}
animator.start();