commit 3e376ab5d119f706b43ee891cc281d5921b8ff13
author Hui Yu <huiyu@google.com> Mon May 04 17:40:52 2020 +0000
committer Hui Yu <huiyu@google.com> Mon May 04 18:07:16 2020 +0000
tree 2b5b2581eb96870bbcfdd77c381b2b75980331d0
parent 9b52815e714ff95a8191923edefd50a74f40ff26

Revert "Add isUidActiveOrForeground() for camera/audio to use."

Revert "Add isUidActiveOrForeground for camera/audio to use."

Revert submission 10829580-isUidForeground

Reason for revert: In CameraService.cpp, before this change, around "am.isUidActive", there was up to 300 ms retry. After this change, the code could move forward fast without retry, but at "mAppOpsManager->startOpNoThrow" call, for the same reason as uid is not updated fast enough, "mAppOpsManager->startOpNoThrow" could also fail.

This CL does not really fix the root cause, but it changes the timing and now the code fails at "mAppOpsManager->startOpNoThrow" call.

Also the timing change may also cause recent multiple CTS test failures.

Bug: 154570809, 155032617, 154849083


Reverted Changes:
Iffed63293:Add isUidActiveOrForeground() for camera/audio to ...
I3685e0c8d:Add isUidActiveOrForeground() for camera/audio to ...
I51ed1fe78:Add isUidActiveOrForeground for camera/audio to us...

Change-Id: I07cbf45949d14489404cb304c80c9ba4276ebe63