微信越滑越卡
这篇文章是我的朋友小二哥写的,他解决了一个问题并提交了一个补丁给google合入,当然了,这也是他第一次提交补丁被google采纳。我觉得这才是做技术的样子,二哥也很喜欢篮球,,为二哥吹一波。哈哈。
关于这次补丁的提交如下文,喜欢的同学可以好好看看。
背景
在一个已经加载完成很长的微信聊天记录中,持续不断的滑动,慢慢的微信会越滑越卡。
一、卡顿的原因分析
Choreographer#doFrame的animation中会堆积大量的Callback-AbsListView#FlingRunnable
从而导致了最后这一帧的绘制超时,导致了卡顿。
二、FlingRunnable堆积的原因
一次滑动会触发一个Down事件,多个Move事件,一个Up事件。
从下图可以发现,这次滑动,导致animation的FlingRunnable从3个增加到了4个
看看这4个是怎么来的:
3个是来自于之前的FlingRunnable,新增的一个来自于Up事件触发的。
三、代码分析
3.1 onTouchDown
Touch Down事件会触发mFlingRunnable.flywheelTouch()
private void onTouchDown(MotionEvent ev) {
...
if (mTouchMode == TOUCH_MODE_OVERFLING) {
...
} else {
...
if (!mDataChanged) { //ListView的数据没有更新
if (mTouchMode == TOUCH_MODE_FLING) {//ListView处于Fling的状态
// Stopped a fling. It is a scroll.
createScrollingCache();
mTouchMode = TOUCH_MODE_SCROLL;
mMotionCorrection = 0;
motionPosition = findMotionRow(y);
mFlingRunnable.flywheelTouch();//跳转到3.1.1
...
3.1.1 mFlingRunnable.flywheelTouch
flywheelTouch会postdelay一个mCheckFlywheel延迟40ms。
当mCheckFlywheel被执行的时候,会去判断ListView当前的滑动速度。
如果Math.abs(yvel) >= mMinimumVelocity,将会再次postdelay一个mCheckFlywheel,让ListView继续滑动一段时间。
如果Math.abs(yvel) < mMinimumVelocity,将会endFling(),这就是为什么ListView滑动之后慢慢停止的逻辑。
endFling中将会removeCallbacks(this)和removeCallbacks(mCheckFlywheel)
private static final int FLYWHEEL_TIMEOUT = 40; // milliseconds
void flywheelTouch() {
postDelayed(mCheckFlywheel, FLYWHEEL_TIMEOUT);
}
private final Runnable mCheckFlywheel = new Runnable() {
@Override
public void run() {
...
if (Math.abs(yvel) >= mMinimumVelocity
&& scroller.isScrollingInDirection(0, yvel)) {
// Keep the fling alive a little longer
//yvel > mMinimumVelocity继续滑动,将mCheckFlywheel在推迟40ms
postDelayed(this, FLYWHEEL_TIMEOUT);
} else {
endFling();
mTouchMode = TOUCH_MODE_SCROLL;
reportScrollStateChange(OnScrollListener.SCROLL_STATE_TOUCH_SCROLL);
}
}
};
void endFling() {
...
removeCallbacks(this);
removeCallbacks(mCheckFlywheel);
...
}
3.2 onTouchUp
在onTouchUp中将会执行mFlingRunnable.start(-initialVelocity),从而postOnAnimation(this);
private void onTouchUp(MotionEvent ev) {
switch (mTouchMode) {
...
case TOUCH_MODE_SCROLL:
...
if (!dispatchNestedPreFling(0, -initialVelocity)) {
if (mFlingRunnable == null) {
mFlingRunnable = new FlingRunnable();
}
reportScrollStateChange(OnScrollListener.SCROLL_STATE_FLING);
mFlingRunnable.start(-initialVelocity);//跳到下面的start方法
dispatchNestedFling(0, -initialVelocity, true);
} else {
mTouchMode = TOUCH_MODE_REST;
reportScrollStateChange(OnScrollListener.SCROLL_STATE_IDLE);
}
...
}
void start(int initialVelocity) {
...
postOnAnimation(this);
...
}
3.3 FlingRunnable#run
FlingRunnable的run方法,如果ListView处于TOUCH_MODE_SCROLL或者TOUCH_MODE_FLING的状态,并且还有更多的内容,就会继续postOnAnimation(this)
@Override
public void run() {
switch (mTouchMode) {
default:
endFling();
return;
case TOUCH_MODE_SCROLL:
if (mScroller.isFinished()) {
return;
}
// Fall through
case TOUCH_MODE_FLING: {
...
if (more && !atEnd) {
if (atEdge) invalidate();
mLastFlingY = y;
postOnAnimation(this);
} else {
...
}
break;
}
...
}
}
小结:
onTouchDown会postdelay 40ms一个mCheckFlywheel,mCheckFlywheel将会检查ListView是否应该停止
onTouchUp会postOnAnimation(FlingRunnable),让ListView开始Fling起来。
每一个FlingRunnable又会再次触发一个postOnAnimation(FlingRunnable)。
四、对比分析
4.1 为什么Google Pixel不存在这个BUG
原来Google Pixel每次滑动Down和Move事件的间隔绝大多数情况下大于40ms,从而导致mCheckFlywheel中endFling可以在持续的滑动中被有效的执行,这样子就不会导致FlingRunnable的堆积
4.2 为什么我们的手机会存在这个BUG
原来我们的手机TP采样率比较高,接近180hz,Down和Move的时间间隔竟然在9ms左右,从而导致了mCheckFlywheel永远被postdelay,无法有效的执行endFling,这样子就导致了FlingRunnable的堆积
五、解决方案
在FlingRunnable.start中调用postOnAnimation之前removeCallbacks(this),避免FlingRunnable的堆积
这个方案已经被merge进了Android官方主分支中:
https://android-review.googlesource.com/c/platform/frameworks/base/+/1645426
void start(int initialVelocity) {
int initialY = initialVelocity < 0 ? Integer.MAX_VALUE : 0;
mLastFlingY = initialY;
mScroller.setInterpolator(null);
mScroller.fling(0, initialY, 0, initialVelocity,
0, Integer.MAX_VALUE, 0, Integer.MAX_VALUE);
mTouchMode = TOUCH_MODE_FLING;
mSuppressIdleStateChangeCall = false;
removeCallbacks(this);//修复的patch
postOnAnimation(this);
if (PROFILE_FLINGING) {
if (!mFlingProfilingStarted) {
Debug.startMethodTracing("AbsListViewFling");
mFlingProfilingStarted = true;
}
}
if (mFlingStrictSpan == null) {
mFlingStrictSpan = StrictMode.enterCriticalSpan("AbsListView-fling");
}
}
总结
这是我作为android工程师第一次成功提交代码到Android官方主分支,还是值得纪念的,可惜提交的账户不是我自己的,而是公司账户,因为自己的账户很有可能Google工程师不会review你的提交。有了一次就会有第二次,期待我下次继续为Android开源代码贡献代码。