Merge "Handle user create failure correctly for lifecycle atom." into rvc-dev am: 22e9ee7459
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11905939 Change-Id: I09907a3693bda4f0f5815d4db1555fa6f2646e58
This commit is contained in:
@@ -3291,11 +3291,13 @@ public class UserManagerService extends IUserManager.Stub {
|
||||
final TimingsTraceAndSlog t = new TimingsTraceAndSlog();
|
||||
t.traceBegin("createUser-" + flags);
|
||||
final long sessionId = logUserCreateJourneyBegin(nextProbableUserId, userType, flags);
|
||||
UserInfo newUser = null;
|
||||
try {
|
||||
return createUserInternalUncheckedNoTracing(name, userType, flags, parentId,
|
||||
preCreate, disallowedPackages, t);
|
||||
newUser = createUserInternalUncheckedNoTracing(name, userType, flags, parentId,
|
||||
preCreate, disallowedPackages, t);
|
||||
return newUser;
|
||||
} finally {
|
||||
logUserCreateJourneyFinish(sessionId, nextProbableUserId);
|
||||
logUserCreateJourneyFinish(sessionId, nextProbableUserId, newUser != null);
|
||||
t.traceEnd();
|
||||
}
|
||||
}
|
||||
@@ -3314,10 +3316,11 @@ public class UserManagerService extends IUserManager.Stub {
|
||||
return sessionId;
|
||||
}
|
||||
|
||||
private void logUserCreateJourneyFinish(long sessionId, @UserIdInt int userId) {
|
||||
private void logUserCreateJourneyFinish(long sessionId, @UserIdInt int userId, boolean finish) {
|
||||
FrameworkStatsLog.write(FrameworkStatsLog.USER_LIFECYCLE_EVENT_OCCURRED, sessionId, userId,
|
||||
FrameworkStatsLog.USER_LIFECYCLE_EVENT_OCCURRED__EVENT__CREATE_USER,
|
||||
FrameworkStatsLog.USER_LIFECYCLE_EVENT_OCCURRED__STATE__FINISH);
|
||||
finish ? FrameworkStatsLog.USER_LIFECYCLE_EVENT_OCCURRED__STATE__FINISH
|
||||
: FrameworkStatsLog.USER_LIFECYCLE_EVENT_OCCURRED__STATE__NONE);
|
||||
}
|
||||
|
||||
private UserInfo createUserInternalUncheckedNoTracing(@Nullable String name,
|
||||
|
||||
Reference in New Issue
Block a user