344ce7ce703f806f50ea253047d6dbbb10eaa424 - platform/frameworks/base

commit	344ce7ce703f806f50ea253047d6dbbb10eaa424	[log] [tgz]
author	Jeff Sharkey <jsharkey@android.com>	Wed May 29 14:52:59 2019 -0600
committer	Jeff Sharkey <jsharkey@android.com>	Thu May 30 11:29:49 2019 -0600
tree	6bd74ba11afcc31dcafe1b83f0118f953e191e34
parent	190e99cf3ab47b2eb196999aeab92a9eee98cae5 [diff]

Collect NeededUriGrants without holding locks.

In preparation for a future security fix, we need to determine Uri
permission grants before acquiring the WM lock.  This CL is mostly
mechanical refactoring to split the calculation and granting logic
into two phases, and pass around the calculated NeededUriGrants.

There is no other logic changes included in this CL, so it should
have no effects.

Bug: 115619667
Test: atest WmTests:ActivityStarterTests
Test: atest CtsWindowManagerDeviceTestCases:ActivityStarterTests
Test: atest android.appsecurity.cts.AppSecurityTests
Change-Id: I033e23549008bea26528781f16caa92427b39c71

services/core/java/com/android/server/uri/UriGrantsManagerInternal.java[diff]
services/core/java/com/android/server/uri/UriGrantsManagerService.java[diff]
services/core/java/com/android/server/wm/ActivityRecord.java[diff]
services/core/java/com/android/server/wm/ActivityStack.java[diff]
services/core/java/com/android/server/wm/ActivityStackSupervisor.java[diff]
services/core/java/com/android/server/wm/ActivityStartController.java[diff]
services/core/java/com/android/server/wm/ActivityStarter.java[diff]
services/core/java/com/android/server/wm/ActivityTaskManagerService.java[diff]
services/core/java/com/android/server/wm/TaskRecord.java[diff]
services/core/java/com/android/server/wm/WindowProcessController.java[diff]
services/tests/wmtests/src/com/android/server/wm/ActivityStackTests.java[diff]
services/tests/wmtests/src/com/android/server/wm/ActivityStartControllerTests.java[diff]

12 files changed