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

Commit efb4c580 authored by Richard Coles's avatar Richard Coles
Browse files

Document permissions for WebViewUpdateManager. 

WebViewUpdateManager.getValidWebViewPackages() requires
INTERACT_ACROSS_USERS and QUERY_ALL_PACKAGES - this is already enforced
by the service implementation but was not documented in the client API.

Bug: 319292658
Test: n/a
Flag: android.webkit.update_service_ipc_wrapper
Change-Id: I5a789b89235b328489ccb4f24f77f5f19500e059
parent f68c80b4

core/api/module-lib-current.txt

+1 −1
Original line number Diff line number Diff line
@@ -674,7 +674,7 @@ package android.webkit { 
    method @Nullable public String getCurrentWebViewPackageName();

    method @FlaggedApi("android.webkit.update_service_v2") @NonNull public android.webkit.WebViewProviderInfo getDefaultWebViewPackage();

    method @Nullable public static android.webkit.WebViewUpdateManager getInstance();

    method @NonNull public android.webkit.WebViewProviderInfo[] getValidWebViewPackages();

    method @NonNull @RequiresPermission(allOf={android.Manifest.permission.INTERACT_ACROSS_USERS, android.Manifest.permission.QUERY_ALL_PACKAGES}) public android.webkit.WebViewProviderInfo[] getValidWebViewPackages();

    method @NonNull public android.webkit.WebViewProviderResponse waitForAndGetProvider();

  }

core/java/android/webkit/WebViewUpdateManager.java

+2 −0
Original line number Diff line number Diff line
@@ -101,6 +101,8 @@ public final class WebViewUpdateManager { 
     * enabled for all users.

     */

    @SuppressLint({"ParcelableList", "ArrayReturn"})

    @RequiresPermission(allOf = {android.Manifest.permission.INTERACT_ACROSS_USERS,

            android.Manifest.permission.QUERY_ALL_PACKAGES})

    public @NonNull WebViewProviderInfo[] getValidWebViewPackages() {

        try {

            return mService.getValidWebViewPackages();