Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e237eb89 authored by Hai Zhang's avatar Hai Zhang
Browse files

Fix filtering for browser apps. 

Despite what the javadoc on PackageManager.MATCH_ALL says, it actually
still performs all kinds of filtering and only affects whether to
return only the default browser for browser intent. So no need for
extra filtering on the result.

The previous filtering based upon ActivityInfo.enabled and
ApplicationInfo.enabled is incorrect because they only reflect the
state declared in manifest, not the actual state.

Fixes: 137731581
Test: presubmit
Change-Id: I61a1dcd572d2ada01ec96c9399be38f66ad88bbb
parent 7bb22d54

src/com/android/packageinstaller/role/model/BrowserRoleBehavior.java

+1 −3
Original line number Diff line number Diff line
@@ -105,9 +105,7 @@ public class BrowserRoleBehavior implements RoleBehavior { 
        for (int i = 0; i < resolveInfosSize; i++) {

            ResolveInfo resolveInfo = resolveInfos.get(i);



            if (!resolveInfo.handleAllWebDataURI || !resolveInfo.activityInfo.enabled

                    || !resolveInfo.activityInfo.applicationInfo.enabled

                    || resolveInfo.activityInfo.applicationInfo.isInstantApp()) {

            if (!resolveInfo.handleAllWebDataURI) {

                continue;

            }

            packageNames.add(resolveInfo.activityInfo.packageName);