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

Commit b1fe5159 authored by Mårten Kongstad's avatar Mårten Kongstad Committed by Adam Lesinski
Browse files

OMS: fix offset-by-one error when setting overlay priority 

setPriority(A, B) used to change A's priority to just higher than B's,
but in commit "OverlayManagerService: Make broadcasts/updates explicit"
this was modified so that A's priority is set to just lower than B's.

Revert the behaviour to the original since that is what is documented in
IOverlayManager.aidl

Test: manual
Change-Id: I87e96fb49d5ad58528dc0448aed907b405db1945
parent d95e1d6e

services/core/java/com/android/server/om/OverlayManagerSettings.java

+1 −1
Original line number Diff line number Diff line
@@ -230,7 +230,7 @@ final class OverlayManagerSettings { 
        }



        mItems.remove(moveIdx);

        final int newParentIdx = select(newParentPackageName, userId);

        final int newParentIdx = select(newParentPackageName, userId) + 1;

        mItems.add(newParentIdx, itemToMove);

        return moveIdx != newParentIdx;

    }