commit acf3858bf3e99ba354fdf082f1547832485d5368
author Makoto Onuki <omakoto@google.com> Fri Oct 04 11:53:42 2019 -0700
committer Makoto Onuki <omakoto@google.com> Mon Oct 07 10:09:45 2019 -0700
tree c2958519708c7d4de9e7a7434e3840f13bd738a6
parent 496d47ab030e56022ede4012cba8771f54d079ad

Optimize UserHandle.of()

There will be more calls to *ForUser() APIs over time by mainline modules.
Let's avoid creating new objects in common cases.

Bug: 142134660
Fix: 140429319
Test: atest cts/tests/app/src/android/app/cts/UserHandleTest.java
Change-Id: Ie71850bd08a8d9831232ddb40807ae49e3a56841

services/core/java/com/android/server/pm/UserManagerService.java

diff --git a/services/core/java/com/android/server/pm/UserManagerService.java b/services/core/java/com/android/server/pm/UserManagerService.java
index 8431fa3..95baa01 100644
--- a/services/core/java/com/android/server/pm/UserManagerService.java
+++ b/services/core/java/com/android/server/pm/UserManagerService.java
@@ -209,7 +209,8 @@
             | UserInfo.FLAG_DEMO;
 
     @VisibleForTesting
-    static final int MIN_USER_ID = 10;
+    static final int MIN_USER_ID = UserHandle.MIN_SECONDARY_USER_ID;
+
     // We need to keep process uid within Integer.MAX_VALUE.
     @VisibleForTesting
     static final int MAX_USER_ID = Integer.MAX_VALUE / UserHandle.PER_USER_RANGE;