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

Commit 2053fa84 authored by Eugene Susla's avatar Eugene Susla
Browse files

updateUserSensitive asynchronously 

This is known to take 500ms and affects only UI,
so can be done async

Test: Ensure nothing looks badly broken; presubmit
Fixes: 139485700
Change-Id: I3837404ea7d9b1598efa0d35cc999af0e86cc7f2
parent 1719c99a

src/com/android/packageinstaller/permission/service/PermissionControllerServiceImpl.java

+4 −0
Original line number Diff line number Diff line
@@ -34,6 +34,7 @@ import android.content.Context; 
import android.content.pm.PackageInfo;

import android.content.pm.PackageManager;

import android.os.AsyncTask;

import android.os.Process;

import android.os.UserHandle;

import android.permission.PermissionControllerService;

import android.permission.PermissionManager;
@@ -569,6 +570,9 @@ public final class PermissionControllerServiceImpl extends PermissionControllerS 
        AsyncTask.execute(() -> {

            onGrantOrUpgradeDefaultRuntimePermissions();

            callback.run();



            // Async part

            Utils.updateUserSensitive(getApplication(), Process.myUserHandle());

        });

    }

src/com/android/packageinstaller/role/service/RoleControllerServiceImpl.java

+0 −5
Original line number Diff line number Diff line
@@ -29,7 +29,6 @@ import androidx.annotation.NonNull; 
import androidx.annotation.WorkerThread;



import com.android.packageinstaller.permission.utils.CollectionUtils;

import com.android.packageinstaller.permission.utils.Utils;

import com.android.packageinstaller.role.model.Role;

import com.android.packageinstaller.role.model.Roles;

import com.android.packageinstaller.role.utils.PackageUtils;
@@ -169,10 +168,6 @@ public class RoleControllerServiceImpl extends RoleControllerService { 
            }

        }



        // Load data on this thread instead of background.

        // TODO: Move out of this thread

        Utils.updateUserSensitive(getApplication(), Process.myUserHandle());



        return true;

    }