Android 7.0 PopupWindow的一个展示Bug

起因与Bug详情

前段时间有反馈过来说,在Android7.0中,PopupWindow的展示位置出了问题(最开始以为没弹出来,后来发现是位置错了),通过查谷看歌源搜码索到了可能的情况和一些文章.

但找到的文章对于问题的表述不太完整,还有一些文章根本就是搞错了原因(大量文章说位置错误是由于PopupWindow的宽高设置的太大了…醉了),后来结合源码和实际的Bug情况看到了具体的详细原因.

这个Bug的具体情况是:在使用PopupWindow调用showAtLocation方法showAsDropDown方法和update方法时,如果传入的Gravity参数不为Gravity.START|Gravity.TOPGravity会被设置为Gravity.START|Gravity.TOP,PopupWindow的位置即发生了改变,可以通过反射来改掉这个Bug,下面是这个Bug的详细解法

Bug原因

具体问题发生在computeGravity方法

private int computeGravity() {
    int gravity = Gravity.START | Gravity.TOP;
    if (mClipToScreen || mClippingEnabled) {
        gravity |= Gravity.DISPLAY_CLIP_VERTICAL;
    }
    return gravity;
}

可以看到返回的gravity值,在方法的一开始就被强制设置为了Gravity.START|Gravity.TOP,所以我们传入的参数并没有起到任何作用,而这个Bug只有API版本24,Android 7.0的SDK是这样,无责任猜想可能是Android系统开发的某位大哥,在写分屏相关的UI代码的时候,出于测试方便或者什么的,直接将这里写死了╮(╯_╰)╭
不过我们可以通过反射来改成正确的代码.

如何解决

首先查看一下都哪里用到了这个computeGravity,然后通过搜索看到,分别是1418行的createPopupLayoutParams方法里出现了

p.gravity = computeGravity();

以及在1096行和2081行的两个update方法内部出现了同样的一行代码

final int newGravity = computeGravity();

由于懒在实际使用中并没有通过update来更新PopupWindow的位置,并且也只是用了showAtLocation,所以暂没有对updateshowAsDropDown进行反射来重写方法,只重写了showAtLocation,理论上讲,createPopupLayoutParams同时被showAtLocationshowAsDropDown方法用到了,且这是一个私有方法,所以需要分别重写showAtLocationshowAsDropDown方法以及两个update方法

这里提供出一个修改方案,importpackage已去掉,如果要使用请自行添加,这里特别说明一下TransitionManager.endTransitions(mDecorView);,这行代码在IDE中很可能会报错标红,原因是使用了高版本API (Android M SDK 23)而没有进行版本判断,但实际上可以不用理会,因为目前这个Bug只有7.0这一个版本出现了,所以我在方法的最开始进行了当前Android版本的判断,如果不是版本号24的7.0版本,直接执行super.showAtLocation调用原PopupWindow的方法,之后return掉了这个方法,毕竟反射也是会带来额外一丢丢的性能和内存占用.如果不想让他报红可以选择改成if else的形式.

public class NougatPopupWindow extends PopupWindow {
    public NougatPopupWindow(View contentView, int width, int height, boolean focusable) {
        super(contentView, width, height, focusable);
    }
    @Override
    public void showAtLocation(View parent, int gravity, int x, int y) {
        if (Build.VERSION.SDK_INT != 24) {
            super.showAtLocation(parent, gravity, x, y);
            return;
        }
        Object obj = getParam("mContentView");
        View mContentView = (View) obj;
        if (isShowing() || mContentView == null) {
            return;
        }
        obj = getParam("mDecorView");
        ViewGroup mDecorView = (ViewGroup) obj;
        //RequireAPI M but if SDK_INT != N,super.showAtLocation and returned;
        TransitionManager.endTransitions(mDecorView);
        execMethod("detachFromAnchor", new Class[]{}, new Object[]{});
        setParam("mIsShowing", true);
        setParam("mIsDropdown", false);
        obj = execMethod("createPopupLayoutParams", new Class[]{IBinder.class}, new Object[]{parent.getWindowToken()});
        final WindowManager.LayoutParams p = (WindowManager.LayoutParams) obj;
        p.gravity = computeGravity(gravity);
        execMethod("preparePopup",new Class[]{WindowManager.LayoutParams.class},new Object[]{p});
        if (gravity != Gravity.NO_GRAVITY) {
            p.gravity = gravity;
        }
        p.x = x;
        p.y = y;
        execMethod("invokePopup",new Class[]{WindowManager.LayoutParams.class},new Object[]{p});
    }
    private Object getParam(String paramName) {
        if (TextUtils.isEmpty(paramName)) {
            return null;
        }
        try {
            Field field = PopupWindow.class.getDeclaredField(paramName);
            field.setAccessible(true);
            return field.get(this);
        } catch (Exception e) {
            e.printStackTrace();
        }
        return null;
    }
    private void setParam(String paramName, Object obj) {
        if (TextUtils.isEmpty(paramName)) {
            return;
        }
        try {
            Field field = PopupWindow.class.getDeclaredField(paramName);
            field.setAccessible(true);
            field.set(this, obj);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
    private Object execMethod(String methodName, Class[] cls, Object[] args) {
        if (TextUtils.isEmpty(methodName)) {
            return null;
        }
        try {
            Method method = getMethod(PopupWindow.class, methodName, cls);
            method.setAccessible(true);
            return method.invoke(this, args);
        } catch (Exception e) {
            e.printStackTrace();
        }
        return null;
    }
    private Method getMethod(Class clazz, String methodName,
                             final Class[] classes) throws NoSuchMethodException {
        Method method = null;
        try {
            method = clazz.getDeclaredMethod(methodName, classes);
        } catch (NoSuchMethodException e) {
            try {
                method = clazz.getMethod(methodName, classes);
            } catch (NoSuchMethodException ex) {
                if (clazz.getSuperclass() == null) {
                    return method;
                } else {
                    method = getMethod(clazz.getSuperclass(), methodName,
                            classes);
                }
            }
        }
        return method;
    }
    private int computeGravity(int mGravity) {
        setParam("mGravity", mGravity);
        int gravity = mGravity == Gravity.NO_GRAVITY ? Gravity.START | Gravity.TOP : mGravity;
        Object obj = getParam("mIsDropdown");
        boolean mIsDropdown = (boolean) obj;
        obj = getParam("mClipToScreen");
        boolean mClipToScreen = (boolean) obj;
        obj = getParam("mClippingEnabled");
        boolean mClippingEnabled = (boolean) obj;
        if (mIsDropdown && (mClipToScreen || mClippingEnabled)) {
            gravity |= Gravity.DISPLAY_CLIP_VERTICAL;
        }
        return gravity;
    }
}

文中部分内容参考了作者Kinva的文章,链接:http://www.jianshu.com/p/0df10893bf5b

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 157,198评论 4 359
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 66,663评论 1 290
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 106,985评论 0 237
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 43,673评论 0 202
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 51,994评论 3 285
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,399评论 1 211
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,717评论 2 310
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,407评论 0 194
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,112评论 1 239
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,371评论 2 241
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 31,891评论 1 256
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,255评论 2 250
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 32,881评论 3 233
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,010评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,764评论 0 192
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,412评论 2 269
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,299评论 2 260

推荐阅读更多精彩内容