drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

Browse Source
This commit is contained in:
Vladislav Kaznacheev
2015-05-18 23:32:55 +00:00
committed by Android (Google) Code Review
parent 905bc335a5 58ae164100
commit d2368d07a7
1 changed files with 4 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@ -82,17 +82,11 @@ 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 ever set on * on the Animator and just use its default interpolator. If the interpolator is set to a
* this Animator, then we use the interpolator that it was set to. * non-null value on 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.
*/ */
@ -340,7 +334,6 @@ 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;
} }
@ -351,7 +344,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 (mInterpolatorSet) { if (mInterpolator != null) {
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
@ -899,7 +892,7 @@ public class ViewPropertyAnimator {
if (mDurationSet) { if (mDurationSet) {
animator.setDuration(mDuration); animator.setDuration(mDuration);
} }
if (mInterpolatorSet) { if (mInterpolator != null) {
animator.setInterpolator(mInterpolator); animator.setInterpolator(mInterpolator);
} }
animator.start(); animator.start();