Minor changes on UserHal:
- Explicitly sets DEFAULT mode on result bundle.
- TODOs cleanup
Test: atest CarServicesTest:com.android.internal.car.CarHelperServiceTest \
CarServiceUnitTest:com.android.car.user.CarUserServiceTest
Bug: 150222501
Bug: 146207078
Bug: 150399261
Change-Id: I36c70e1d3b080d5772485d24cfd69bce7c1e8e6c
diff --git a/service/src/com/android/car/ICarImpl.java b/service/src/com/android/car/ICarImpl.java
index 1923d9a..f06d060 100644
--- a/service/src/com/android/car/ICarImpl.java
+++ b/service/src/com/android/car/ICarImpl.java
@@ -356,7 +356,7 @@
mCarUserService.onSwitchUser(userId);
}
- // TODO(b/146207078): this method is currently used just for metrics logging purposes, but we
+ // TODO(b/145689885): this method is currently used just for metrics logging purposes, but we
// should fold the other too (onSwitchUser() and setUserLockStatus()) onto it.
@Override
public void onUserLifecycleEvent(int eventType, long timestampMs, int fromUserId,
@@ -1264,7 +1264,7 @@
+ "), timeout=" + timeout);
UserHalService userHal = mHal.getUserHal();
- // TODO(b/146207078): use UserHalHelper to populate it with current users
+ // TODO(b/150413515): use UserHalHelper to populate it with current users
UsersInfo usersInfo = new UsersInfo();
CountDownLatch latch = new CountDownLatch(1);