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

Commit ec4dd578 authored by Yasin Kilicdere's avatar Yasin Kilicdere Committed by Automerger Merge Worker
Browse files

Merge "Add UserSwitchObserver.onBeforeUserSwitching and make SysUI use it."... 

Merge "Add UserSwitchObserver.onBeforeUserSwitching and make SysUI use it." into udc-dev am: bd63b5d9 am: 8bcb8d80

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22238586



Change-Id: I2abde634e05c3f291205d731401a4588fd0cda1d
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0dd6937f 8bcb8d80

core/java/android/app/IUserSwitchObserver.aidl

+1 −0
Original line number Diff line number Diff line
@@ -20,6 +20,7 @@ import android.os.IRemoteCallback; 


/** {@hide} */

oneway interface IUserSwitchObserver {

    void onBeforeUserSwitching(int newUserId);

    void onUserSwitching(int newUserId, IRemoteCallback reply);

    void onUserSwitchComplete(int newUserId);

    void onForegroundProfileSwitch(int newProfileId);

core/java/android/app/UserSwitchObserver.java

+3 −0
Original line number Diff line number Diff line
@@ -29,6 +29,9 @@ public class UserSwitchObserver extends IUserSwitchObserver.Stub { 
    public UserSwitchObserver() {

    }



    @Override

    public void onBeforeUserSwitching(int newUserId) throws RemoteException {}



    @Override

    public void onUserSwitching(int newUserId, IRemoteCallback reply) throws RemoteException {

        if (reply != null) {

packages/SystemUI/src/com/android/systemui/settings/UserTrackerImpl.kt

+4 −3
Original line number Diff line number Diff line
@@ -161,6 +161,10 @@ open class UserTrackerImpl internal constructor( 


    private fun registerUserSwitchObserver() {

        iActivityManager.registerUserSwitchObserver(object : UserSwitchObserver() {

            override fun onBeforeUserSwitching(newUserId: Int) {

                setUserIdInternal(newUserId)

            }



            override fun onUserSwitching(newUserId: Int, reply: IRemoteCallback?) {

                backgroundHandler.run {

                    handleUserSwitching(newUserId)
@@ -181,8 +185,6 @@ open class UserTrackerImpl internal constructor( 
        Assert.isNotMainThread()

        Log.i(TAG, "Switching to user $newUserId")



        setUserIdInternal(newUserId)



        val list = synchronized(callbacks) {

            callbacks.toList()

        }
@@ -205,7 +207,6 @@ open class UserTrackerImpl internal constructor( 
        Assert.isNotMainThread()

        Log.i(TAG, "Switched to user $newUserId")



        setUserIdInternal(newUserId)

        notifySubscribers {

            onUserChanged(newUserId, userContext)

            onProfilesChanged(userProfiles)

packages/SystemUI/tests/src/com/android/systemui/settings/UserTrackerImplTest.kt

+3 −0
Original line number Diff line number Diff line
@@ -167,6 +167,7 @@ class UserTrackerImplTest : SysuiTestCase() { 


        val captor = ArgumentCaptor.forClass(IUserSwitchObserver::class.java)

        verify(iActivityManager).registerUserSwitchObserver(capture(captor), anyString())

        captor.value.onBeforeUserSwitching(newID)

        captor.value.onUserSwitching(newID, userSwitchingReply)

        verify(userSwitchingReply).sendResult(any())
@@ -290,6 +291,7 @@ class UserTrackerImplTest : SysuiTestCase() { 


        val captor = ArgumentCaptor.forClass(IUserSwitchObserver::class.java)

        verify(iActivityManager).registerUserSwitchObserver(capture(captor), anyString())

        captor.value.onBeforeUserSwitching(newID)

        captor.value.onUserSwitching(newID, userSwitchingReply)

        verify(userSwitchingReply).sendResult(any())
@@ -308,6 +310,7 @@ class UserTrackerImplTest : SysuiTestCase() { 


        val captor = ArgumentCaptor.forClass(IUserSwitchObserver::class.java)

        verify(iActivityManager).registerUserSwitchObserver(capture(captor), anyString())

        captor.value.onBeforeUserSwitching(newID)

        captor.value.onUserSwitchComplete(newID)



        assertThat(callback.calledOnUserChanged).isEqualTo(1)

services/core/java/com/android/server/am/UserController.java

+11 −0
Original line number Diff line number Diff line
@@ -2141,6 +2141,17 @@ class UserController implements Handler.Callback { 


        final int observerCount = mUserSwitchObservers.beginBroadcast();

        if (observerCount > 0) {

            for (int i = 0; i < observerCount; i++) {

                final String name = "#" + i + " " + mUserSwitchObservers.getBroadcastCookie(i);

                t.traceBegin("onBeforeUserSwitching-" + name);

                try {

                    mUserSwitchObservers.getBroadcastItem(i).onBeforeUserSwitching(newUserId);

                } catch (RemoteException e) {

                    // Ignore

                } finally {

                    t.traceEnd();

                }

            }

            final ArraySet<String> curWaitingUserSwitchCallbacks = new ArraySet<>();

            synchronized (mLock) {

                uss.switching = true;