记一个焦点获取与事件传递相关BUG

字数 799阅读 246

问题重现

页面只有一个ListView 。 ListView 设置了Header 和若干元素,类型均是TextView。
所有TextView 均设置:

android:textIsSelectable="true"

结果发现第一次长按文本,只会设置selection,而不会弹出菜单(8.0系统稳定复现,其他版本不一定):


bug截图.png

第一次操作失败之后,后续操作正常:


正常截图.png

追查原因

首先猜测应该是up事件丢失, 简单的包装了下TextView和ListView,重写了其内事件分发相关的方法。发现长按过程事件分发过程如下:


日志截图.png

得亏用的是前任亲儿子 Nexus6P(8.1), 可以直接跟TextView和ListView的代码。发现在长按过程中ViewGroup的dispatchTouchEvent事件会突然把move事件转换为canceled。
具体代码是:


ViewGroup.dispatchTouchEvent

进一步分析是ListView的 flag被标记为了取消下一个up事件:


ViewGroup.resetCancelNextUpFlag

更进一步

进一步搜索ListView,TextView相关源码中所有对 PFLAG_CANCEL_NEXT_UP_EVENT 的操作,发现在view.onStartTemporaryDetach() 的时候会修改这个flag:


View.onStartTemporaryDetach

进一步设置断点,发现是长按过程中触发了onLayout,onLayout的过程中调用了onStartTemporaryDetach。

第一次长按的过程中为什么会触发Layout呢?而之后再长按却不会触发呢?

再进一步

查阅了下源码,没在长按相关的事件里找到调用invalid() 或者requestLayout()相关的代码,所以直接通过Method trace对长按事件做了监控。

            Debug.startMethodTracing("trace file path");  //开始
            Debug.stopMethodTracing();  //结束

发现唯一触发requestLayout的地方是focusChange的时候:


requestLayout方法.png

问题最终原因

最终发现是长按文本时,会触发TextView的requestFocus方法。 (因为TextView isTextSelectable = true, 所以focusable\focusable in touch mode 等都是true)
而View的requestFocus最终会调用parent的ViewGroup.requestChildFocus()


View源码

此时会尝试清除父View的焦点:


ViewGroup源码

最终会调用View的onFocusChanged回调:


View.clearFocusInternal()

onFocusChanged 回调里面会调用view.invalidate() 方法来重新绘制.

@CallSuper
protected void onFocusChanged(boolean gainFocus, @FocusDirection int direction,
            @Nullable Rect previouslyFocusedRect) {
        ...
        invalidate(true);
        ...
}

所以问题的根本原因是:长按TextView 触发Selection的过程会让TextView获得焦点,ListView在失去焦点时会重新Layout,重新layout的过程中会重置view对应的flag,导致当前view的flag异常,move被处理成cancel操作。textview收到Cancel操作后,事件终止不会接收后续事件,up事件的回调不会执行,所以菜单弹不出来。

解决办法

首先想到的是尝试让ListView无法获取焦点:
给ListView设置focusable\focusable_in_touch_mode = false, 问题并没有解决,因为调用父控件的clearFocus时,并没有判断父控件是否是可以获取焦点的.....

因为只有长按过程中焦点切换会导致当前长按事件异常,所以在页面加载完成后就让TextView获取焦点。问题得以解决....

但是, 如果List中多个元素需要支持选择复制呢?

多个TextView

发现ListView layout是由于unFocus导致的,所以可以重写ListView的requestChildFocus方法,把unFocus的逻辑去掉。

嗯... 看上去问题解决了...

    @Override
    public void requestChildFocus(View child, View focused) {
//        super.requestChildFocus(child, focused);
        try {
            Field preFocusedField = ViewGroup.class.getDeclaredField("mFocused");
            Method method = View.class.getDeclaredMethod("unFocus", View.class);
            preFocusedField.setAccessible(true);
            method.setAccessible(true);
            View preFocusView = (View) preFocusedField.get(this);
            if (preFocusView != child) {
                if (preFocusView != null) {
                    method.invoke(preFocusedField, focused);
                }
                preFocusedField.set(this, child);
            }
        } catch (NoSuchFieldException e) {
            e.printStackTrace();
        } catch (NoSuchMethodException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        } catch (InvocationTargetException e) {
            e.printStackTrace();
        }

        if (getParent() != null) {
            getParent().requestChildFocus(this, focused);
        }
    }

推荐阅读更多精彩内容