PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Use MATCH_ALL to query available browsers."

Browse Source
This commit is contained in:
TreeHugger Robot
2019-02-20 21:31:21 +00:00
committed by Android (Google) Code Review
parent 569aa2594e 7bd98d2519
commit 1de49122d3
2 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/com/android/settings/applications/defaultapps/DefaultBrowserPreferenceController.java
View File

@@ -45,7 +45,8 @@ public class DefaultBrowserPreferenceController extends DefaultAppPreferenceCont
static final Intent BROWSE_PROBE = new Intent() static final Intent BROWSE_PROBE = new Intent()
.setAction(Intent.ACTION_VIEW) .setAction(Intent.ACTION_VIEW)
.addCategory(Intent.CATEGORY_BROWSABLE) .addCategory(Intent.CATEGORY_BROWSABLE)
.setData(Uri.parse("http:")); .setData(Uri.parse("http:"))
.addFlags(Intent.FLAG_IGNORE_EPHEMERAL);
public DefaultBrowserPreferenceController(Context context) { public DefaultBrowserPreferenceController(Context context) {
super(context); super(context);
@@ -112,11 +113,13 @@ public class DefaultBrowserPreferenceController extends DefaultAppPreferenceCont
final List<ResolveInfo> candidates = new ArrayList<>(); final List<ResolveInfo> candidates = new ArrayList<>();
// Resolve that intent and check that the handleAllWebDataURI boolean is set // Resolve that intent and check that the handleAllWebDataURI boolean is set
final List<ResolveInfo> list = packageManager.queryIntentActivitiesAsUser( final List<ResolveInfo> list = packageManager.queryIntentActivitiesAsUser(
BROWSE_PROBE, 0 /* flags */, userId); BROWSE_PROBE, PackageManager.MATCH_ALL, userId);
if (list != null) { if (list != null) {
final Set<String> addedPackages = new ArraySet<>(); final Set<String> addedPackages = new ArraySet<>();
for (ResolveInfo info : list) { for (ResolveInfo info : list) {
if (info.activityInfo == null || !info.handleAllWebDataURI) { if (!info.handleAllWebDataURI || info.activityInfo == null
|| !info.activityInfo.enabled
|| !info.activityInfo.applicationInfo.enabled) {
continue; continue;
} }
final String packageName = info.activityInfo.packageName; final String packageName = info.activityInfo.packageName;

5
tests/robotests/src/com/android/settings/applications/defaultapps/DefaultBrowserPreferenceControllerTest.java
View File

@@ -181,11 +181,11 @@ public class DefaultBrowserPreferenceControllerTest {
} }
@Test @Test
public void getCandidates_shouldQueryActivityWithFlagsEquals0() { public void getCandidates_shouldQueryActivityWithMatchAll() {
DefaultBrowserPreferenceController.getCandidates(mPackageManager, 0 /* userId */); DefaultBrowserPreferenceController.getCandidates(mPackageManager, 0 /* userId */);
verify(mPackageManager).queryIntentActivitiesAsUser( verify(mPackageManager).queryIntentActivitiesAsUser(
any(Intent.class), eq(0) /* flags */, eq(0) /* userId */); any(Intent.class), eq(PackageManager.MATCH_ALL), eq(0) /* userId */);
} }
@Test @Test
@@ -221,6 +221,7 @@ public class DefaultBrowserPreferenceControllerTest {
info.handleAllWebDataURI = true; info.handleAllWebDataURI = true;
info.activityInfo = new ActivityInfo(); info.activityInfo = new ActivityInfo();
info.activityInfo.packageName = packageName; info.activityInfo.packageName = packageName;
info.activityInfo.applicationInfo = createApplicationInfo(packageName);
return info; return info;
} }