Locking on an interned string can cause unexpected locking collisions, and we should lock the mLock object here.
Change-Id: I8b6557371b0cb79e1bcbc732982cf66b9de663a5
Now that these methods are no longer called, and none of them are a
public API or have @UnsupportedAppUsage, they can be removed.
inCryptKeeperBounce() actually had one known app user via reflection,
despite the method not having @UnsupportedAppUsage. However, that user
only made the call if Build.VERSION.SDK_INT < VERSION_CODES.P, so it is
not being used anymore.
Bug: 208476087
Change-Id: Idc218e5f355bb61257b07cf5b5b6df5f4c6ece11
Since FDE (Full Disk Encryption) is no longer supported, there is no
need for DevicePolicyManagerService.getEncryptionStatus() to check for
FDE-specific encryption statuses.
There is more that needs to be cleaned up here, but this gets rid of the
dependency on some obsolete StorageManager methods.
Bug: 208476087
Test: atest DevicePolicyManagerTest
Change-Id: I26c3fafc1b134177324de929d3600fb2dd7f4d1d
These will later be referenced by a google3 gcl file as part of a new
libcore test set. Doc: go/libcore-benchmarks-doc
Test: N/A
Bug: 210606618
Change-Id: I4f38b797f8f4978d00e92da07c3db8df366a6897
The command set-strong-auth can help to require strong auth.
Thus we can use this command to simulate the lockdown mode on cts.
Bug: 173721373
Test: atest LockSettingsShellCommandTest
Change-Id: Ifabb14c77b15e55e663902647310496f5519d5a5
Stop parsing the IKE options from the input IkeTunnelConnectionParams
if exists. The caller shuould get the relevant infortation the
params.
The field is also stored in the VpnProfile if it built from an
Ikev2VpnProfile.
Bug: 184750836
Test: atest FrameworksNetTests HostsideVpnTests
Test: build ; flash ; connect VPN successfully
Change-Id: Id05c17285915462c7a6cd51d5fb4ee5da7f465ba
Revert "Deactivate data network if TDs are empty"
Revert submission 17146764-deactivate_no_tds
Reason for revert: Internet broken
Reverted Changes:
I6dd9d3e01:Add data fail cause for empty TD list
I3b474e10f:Deactivate data network if TDs are empty
Change-Id: I43bcb551bbb8775ed489822ca3ad3fe3af7e9c0b
Merged-In: I43bcb551bbb8775ed489822ca3ad3fe3af7e9c0b
Remove network on all errors instead of only RADIO_NOT_AVAIALBLE.
Test: atest DataNetworkTest
Bug: 219998284
Change-Id: Ia1f2e5cd63bcf93967ea4fd4a35fcb9515c5e3f3
Merged-In: Ia1f2e5cd63bcf93967ea4fd4a35fcb9515c5e3f3
This permission will eventually replace the sepolicy that we have now,
making it possible to test unique id attestation, which is no longer
possible due to the deprecation of shared uids.
Skipping automerger because the framework manifest has diverged too
much across aosp and tm-dev to allow for clean auto merges, and
presubmits block.
Test: KeyAttestationTest
Bug: 216778747
Merged-In: Iecde35b9a79456b293118d8089dd2a3b0905f5f3
Change-Id: Iecde35b9a79456b293118d8089dd2a3b0905f5f3
Document a section of code that prevents an NTP-related CTS test working
if the device is not connected to Wifi. The reason the code exists took
a little bit of research, so here it is.
Bug: 213393821
Test: Comment-only change
Change-Id: I0b5d9f298ef36e861f780a29fdf74f0226fa1567
