Merge "Temporarily disable clang-tidy for dex_builder.cc."

Temporarily disable clang-tidy for dex_builder.cc.

Bug: 123880763
Test: build with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=*,-readability-*,
-google-readability-*,-google-runtime-references,-cppcoreguidelines-*,
-modernize-*,-llvm-*,-bugprone-narrowing-conversions,
-misc-non-private-member-variables-in-classes,
-misc-unused-parameters,-hicpp-*,-fuchsia-*

Change-Id: I7be72b9db232a375bf7caf3b2dbeb9c6f7368a8e

Merge "Add IntDef for SIP response code."

Merge "Separate privapp grant for dialer"

Merge "Improve AsyncTask API documentation"

Merge "Replace hardcoded cgroup paths"

Replace hardcoded cgroup paths

Replace hardcoded cgroup paths with new processgroup library API calls,
include required headers. Add required dependencies.

Bug: 111307099
Test: builds, boots

Change-Id: Ic2dc2a94328cf1927fafaa9506537aeb213df413
Merged-In: Ic2dc2a94328cf1927fafaa9506537aeb213df413
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

Merge "Add NR to preferred network type combination"

Merge "Fix wrong userData length and excessive userData issues"

Merge "Add signal strength converter"

Merge "[viewcompiler] Support more than 16 registers in invoke instructions"

Merge "Refresh cardId before using it."

Merge "Deprecate acceptRingingCall and endCall."

Merge "Fix the wrong assignment"

Merge "Add owners for KeyChain code"

Fix the wrong assignment

the format value is erroneously assigned to sinkChannelMask.
Therefore, it modify to assign the format value to sinkFormat.

Test: manual
Change-Id: Iaa2f2cef0f1448a432279483b23e0057ae3ed381

Merge "Fix to register the receiver for all users"

Merge "Handle permission check for multi-active SIMs devices."

Merge "[viewcompiler] Add PrecompiledLayoutTest to TEST_MAPPING"

Improve AsyncTask API documentation

Bug: 34278255
Test: TreeHugger
Change-Id: I0347072112254f98d257a72ef1fc7c21b086df39

Add signal strength converter

This add a converter for SignalStrength response and indication.

Bug: 123532650
Test: build & current test
Change-Id: Id4cf874f6cc1798c732d6afb4a22f4d6bc4047fd

Merge "Fix read/write parcel of CallAttributes"

Fix wrong userData length and excessive userData issues

Added checking size excess in encodeUCS2() and an exception cause in
EncodeException to distinguish between unencodable char and size excess.
It will prevent wrong userData length and excessive userData issues
during encoding userData. Please see below one issue scenario.

stringToGsm7BitPackedWithHeader() throws an EncodeException when
septeCount > 255. The EncodeException is caught and encoding again
through encodeUCS2(). However, the length of encoded userData as UCS-2
is converted as Byte without checking size excess. Thus first byte of
userData will be wrong and total userData will exceed maximum User Data.

Test: Manual

Change-Id: Ib6df5484072d77548e8adcbcbd15ad7a401c99f8
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>

Merge "Add new ImsException to better handle ImsService errors"

[viewcompiler] Support more than 16 registers in invoke instructions

This was causing the viewcompiler to crash on deeper layout hierarchies.

We fix this by reserving several scratch registers. When an invoke instruction
uses registers that don't fit in a 4-bit field, we move all of these values into
the scratch registers and use an invoke/range instruction instead. The scratch
registers are all above the highest allocated register, so they are guaranteed
not to clobber meaningful values.

Supporting more registers for invoke also required supporting
register-to-register moves, which some how we'd gotten by without so far.

Finally, to make viewcompiler fail more loudly when things go wrong, many
DCHECKs have been changed to CHECKs.

Bug: 123517491
Test: atest
Change-Id: I9eb7c9bcf1fc7d713e664b331804bdcddafc95a4

Refresh cardId before using it.

Bug: 122978614
Bug: 111614811
Test: on phone
Change-Id: I30d70b839f05183ad201a114ea12c79c730f4206
Merged-In: I30d70b839f05183ad201a114ea12c79c730f4206

Merge "Update android.hidl.manager-V1.0-java to V1.2"

Update android.hidl.manager-V1.0-java to V1.2

Test: Manual
Bug: 123307146
Change-Id: I8dcd291c187af2cb20fd9a8f09bd55e75a2762e1

[viewcompiler] Add PrecompiledLayoutTest to TEST_MAPPING

PrecompiledLayoutTest makes sure that layouts generated by the viewcompiler are
equivalent to inflating from the resource XML file. We'll want to run these
tests when making changes to the viewcompiler.

Bug: 111895153
Test: atest
Change-Id: Iffa6f0f01c3f43d43feaabd1a5cd36be76650975

Fix read/write parcel of CallAttributes

Fixes: 123363462
Test: manual
Change-Id: I947005d43930ad136e5d6933c950d0f27f8fbadd

Merge "Do not generate greylist if UNSAFE_DISABLE_HIDDENAPI_FLAGS=true"

Deprecate acceptRingingCall and endCall.

These APIs are replaced by CallScreeningService and the new
companion app roles in InCallService.

Test: Update Apis and verify deprecated.
Bug: 63966743
Change-Id: Ib4c4ac6b723ccd46d3052fb2b9c9f58b3151cc29

Merge "Compile CaptivePortalLogin as system_current"

Merge "Add the DynamicAndroid Service"

Merge "Add NetworkStackPermissionStub definitions"

Merge "Prepare CaptivePortalLogin for system_current"

Add NetworkStackPermissionStub definitions

The NetworkStackPermissionStub package is used to enforce that
permissions used by the NetworkStack are only used in packages
sharing signature with NetworkStackPermissionStub.

Permissions defined in this package are intended to be used only by the
NetworkStack: both NetworkStack and the stub APK will be signed with
a dedicated certificate to ensure that, with permissions being signature
permissions.

This APK *must* be installed, even if the NetworkStack app is not
installed, because otherwise, any application will be able to define
this permission and the system will give that application full access
to the network stack.

Test: flashed, booted
Bug: 112869080
Change-Id: Ia13a9e6a703cb7b4403697a7f7bfff0f6f3b813e

Merge "Fix bluetooth tethering on multi-user"

Add owners for KeyChain code

Add the Android Enterprise Security team as OWNERS for KeyChain and
KeyChain-related code.

The KeyChain code currently lives under keystore/, which means every
change requires Keystore owners approval, but it does not make sense for
KeyChain as KeyChain is a Keystore client and is developed
independently.

Test: Gerritt upload.
Bug: 33166666
Change-Id: Idfedda9553add303439179ce10a1e75e437bbe83

Compile CaptivePortalLogin as system_current

Test: make CaptivePortalLogin
Test: flashed, boots, login works
Bug: 112869080
Change-Id: I955c1b111ce76262fe20f5891b1853de0889f4b8

Prepare CaptivePortalLogin for system_current

Includes small changes to stop using non-SystemApi endpoints.

Test: make CaptivePortalLogin
Test: flashed, login works, SSL certificate invalid flow works
Bug: 112869080
Merged-In: Ic6ce98942959705ea7ef14a7a62d33e3abf7eb72
Change-Id: Ie7ea3bab6aa8629ce4c111adee0c0699d95d4b0e

Add NR to preferred network type combination

This add NR 5G to preferred network type and move preferredNetworkMode
from SystemApi to hidden api as we are not going to support
preferredNetworMode enum in the future.

Bug: 111453000
Test: build
Change-Id: I08a8f7bf44a62ef109583b82b22f65fd6f89c71d
(cherry picked from commit 8bcdc4cf66e9d654fa31d7c62654ff81ee04cdc6)
Merged-in: I08a8f7bf44a62ef109583b82b22f65fd6f89c71d

Merge changes from topic "bindprocess-resolv"

* changes:
  Allow use of bindProcessToNetwork only to change private DNS bypass.
  Allow bypassing private DNS via bindProcessToNetwork.

Merge "Add get last entitlement value callback API"

Add the DynamicAndroid Service

  Define the DynamicAndroid with AIDL.
  Add a java implementation.
  Start a service instance in the system server.
  Add a permission test.

Bug: 122015653
Test: Build & Test on a pixel phone with following command \
  ./frameworks/base/services/tests/runtests.py -e class com.android.server.DynamicAndroidTest

Merged-In: I2e54b6b71fac4a4c5a9c9c25ce6bdac74cddcfb7
Change-Id: I2e54b6b71fac4a4c5a9c9c25ce6bdac74cddcfb7

Fix bluetooth tethering on multi-user

Bluetooth runs as UID 1001002 when on a secondary user. With this change
the NetworkStack verifies that the calling UID matches the Bluetooth app
regardless of the user.

Test: flashed, BT reverse tethering still working as primary user
      (no option to turn on as secondary user on phones)
Bug: 123655057
Change-Id: I23f9c5fa40f3bb676ac65dd8c15106c9d78309a4

Merge "Add UserHandle.getAppId to SystemApi"

Merge "Add a method to determine if a reboot is required to alter the modem config"

Merge "[KA06] export keepalive offload permission as system api."

Handle permission check for multi-active SIMs devices.

For download and switch, if the caller can manage (has carrier privilege)
on any active sub on any other slots && the caller can manage the active
sub on the target slot or there is no active sub on the target slot, we
allow the caller to continue download. Otherwise, we require user consent.
For getting EID, we now check whether the caller has privilege on the
target card with the given cardId.
Other logic including the logic for single-active slot still keep the
same.
Also, open disable sub (switch to empty sub) to privileged carrier apps
instead of just system.

Bug: 36260308
Bug: 120621262
Test: unit test, test on phone
Change-Id: Icfc0c9e9f4303876bceb1adbe217b39b538b347e
Merged-In: Icfc0c9e9f4303876bceb1adbe217b39b538b347e

Add get last entitlement value callback API

The callback would be used to notify entitlement value. If the
cache value indicates entitlement succeeded, it just fire
callback with cache value instead of run entitlement check.

Bug: 120887283
Test: atest FrameworksNetTests
Change-Id: I8afe928423bd75c54c61533a50a5c0814922ceb1

Add a method to determine if a reboot is required to alter the modem config

Bug: 123101701
Test: sanity
Change-Id: I841b947efda0016c003f54ecf7853624582abab9
Merged-In: I841b947efda0016c003f54ecf7853624582abab9
(cherry picked from commit 7eca4afab4565ca1bd0f9827e93370885ba5ccea)

Allow use of bindProcessToNetwork only to change private DNS bypass.

Currently it is not possible to change private DNS bypass by
doing:

  setProcessDefaultNetwork(network.getPrivateDnsBypassingCopy());
  setProcessDefaultNetwork(network);

because the code will ignore the change. Fix this by ensuring
that we always call bindProcessToNetwork (which does not have
side effects) and then only performing the expensive operations
(flushing DNS cache, upating socket pools) if the netId changed.

Bug: 112869080
Test: None
Change-Id: I5e8999cb11d8b8c1e9eb583fa8b3932f212accff

Add UserHandle.getAppId to SystemApi

This is consistent with other methods in this class, and unbundled
system apps such as the network stack need this API to verify their
callers regardless of the userid. The method is already marked as
TestApi.

Test: m
Bug: 123655057
Change-Id: I0ecef4abcd850dc641cd70681a56cb038778c04c

Merge "Add telephony API to switch multi sim config"

Merge "Call Screening / Caller ID API Changes"

Merge "Add an API that allows VPNs to declare themselves as metered."

Merge "Informing app idle listeners on enabled state changes."

Merge "Add sandness@ and rubinxu@ to OWNERS for UserRestrictionsUtils"

Merge "Added OWNERS entries for some Zygote related files."

Merge "Fixed crash in data service"

Fixed crash in data service

Test: telephony sanity tests
Bug: 123598340
Change-Id: Ie15f8869849a2aed4a32f4f6731abd5ba1eb05cd

Add telephony API to switch multi sim config

Bug: 122115649
Test: sanity
Change-Id: Idace8ed7f3fe78a6ce831225bda499a58a34c80f
Merged-In: Idace8ed7f3fe78a6ce831225bda499a58a34c80f
(cherry picked from commit 1e031bde1a85048bc43e72b38869d296db9ae228)

Merge "Add SystemApi for captive portal metrics"

Merge "Nuke old setAlwaysOnVpnPackageForUser method."

Add an API that allows VPNs to declare themselves as metered.

For VPN apps targeting Q and above, they will by default be treated as
metered unless they override this setting before establishing VPN.

Bug: 120145746
Test: atest FrameworksNetTests
Test: On device tests verifying meteredness setup correctly for apps
targeting Q and apps targeting P.
Change-Id: Ia6d1f7ef244bc04ae2e28faa59625302b5994875

Merge "Use default argument for VintfObject::CheckCompatibility."

Merge "Added a broadcast intent for clients to listen for country changes"

Merge "Some of us are weird."

Nuke old setAlwaysOnVpnPackageForUser method.

All callers have migrated to 4-argument one.

Test: builds
Bug: 77468593
Change-Id: I253515f6b0100b675505646339e5b82d373c9429

Added a broadcast intent for clients to listen for country changes

Added a broadcast intent ACTION_NETWORK_COUNTRY_CHANGED for clients
to listen for network country code changes. The same value can be
retrieved from TelephonyManager.getNetworkCountryIso().

Test: Manual + unit tests
Bug: 122900588
Merged-In: Ifec87dda7bae386118b7bc7523844342bd16c3ea
Change-Id: Ifec87dda7bae386118b7bc7523844342bd16c3ea
(cherry picked from commit 3d33698bf5d98a6480a9f6928d1f1e23d519cc08)

Merge "VpnDialogs should be able to control always-on VPN."

Merge "new API for IMS call fail cause"

Merge "[layout precompilation] Add testing hook for precompiled layouts"

VpnDialogs should be able to control always-on VPN.

Bug: 77468593
Test: atest HostsideVpnTests#testIsAlwaysOnIsLockdown
Change-Id: I0a0c6b138c748a6cb8681c20c137f70ee1fc6cf9

Merge "Initialise hidden api usage logger pre-fork"

[layout precompilation] Add testing hook for precompiled layouts

We want to be able to control at a fine grained level whether precompiled
layouts are enabled so we can compare inflation results with and without. This
changes adds a @TestApi method that supports this.

Bug: 111895153
Merged-In: Ib6b62d79a9ca7aefefff8639752aa5838e491038
Change-Id: I2f28f6912499825e52c17cc163be9c0fe93855cb

Allow bypassing private DNS via bindProcessToNetwork.

Currently, bypassing private DNS requires calling the deprecated
setProcessDefaultNetworkForHostResolution. Allow apps to do this
via the non-deprecated binProcessForNetwork as well.

This has fewer backwards compatibility concerns than the
alternative approach of having setProcessDefaultNetwork call
setProcessDefaultNetworkForHostResolution. That approach would
have been problematic, for example, if an app did:

  cm.bindProcessToNetwork(network);
  ...
  cm.bindProcessToNetwork(null);

In this case, it would be difficult to know whether to clear the
resolver mapping as well: what if an app had also called
setProcessDefaultNetworkForHostResolution?

Similarly, it would be difficult to know what to do if an app did:

  cm.setProcessDefaultNetworkForHostResolution(network);
  cm.bindProcessToNetwork(null);

This approach does not have these concerns, and has no effect
on apps that don't call Network.getPrivateDnsBypassingCopy, which
regular apps don't have permission to use. It also provides a
path to deprecate setProcessDefaultNetworkForHostResolution.

Bug: 112869080
Test: atest android.net.cts.ConnectivityManagerTest android.net.cts.MultinetworkApiTest
Change-Id: I4158a37b6ed87a9a9b2677c526dcfee8af48e483

Add SystemApi for captive portal metrics

The metrics go through NetworkMonitor in the NetworkStack so that they
can be upgraded to new metrics in the future.

Test: flashed, captive portal login works, metrics shown in events log
Bug: 112869080
Merged-In: I4bccfbd87bae5b2d65e45c7a5918aa45ab5d76e8
Change-Id: Ib5e2126788f8d56a00a56d7efcd33c5f9a37a6de

Initialise hidden api usage logger pre-fork

Topic: hidden-api-reporting-upcall
Test: m
Bug: 119217680
Change-Id: I8722f0bf4275fea4a8faf41d7b862c04b4014824
Merged-In: I80f82092ef779a7dc1f1322cb7c05ebf5e59939b

Merge "Track ZygoteHooks API changes in libcore"

Informing app idle listeners on enabled state changes.

Currently, if AppStandbyController is enabled after some listeners have
already registered, the listeners will be told that the system is in a
state of parole even though AppStandbyController thinks it's not in
parole. This change informs AppIdleStateChangeListeners when
AppStandbyController's enabled state changes as well so that they can be
in the correct state.

I also removed the call to setAppIdleEnabled() in onBootPhase() since
updateSettings() is called right afterwards and that also calls
setAppIdleEnabled().

Bug: 112329453
Test: atest AppStandbyControllerTests
also check logs to confirm that NetworkPolicyManagerService and
JobSchedulerService's listeners switch from ON to OFF when the system is
ready.

Change-Id: I88bc293c70b459f54f75f92126ad306d6ab8d9b7
Merged-In: I88bc293c70b459f54f75f92126ad306d6ab8d9b7

Merge "API: Fix missing @NonNull annotations"

Merge "API: Add missing nullability annotations"

Merge "move ServiceState.getDataRegState to greylist"

Merge "Add APIs to get/set if usage of multiple SIMs is restricted."

Merge "Compile NetworkStack as system_current"

Merge "Remove last NetworkStack usage of hidden APIs"

Add APIs to get/set if usage of multiple SIMs is restricted.

Add API that indicates if the usage of multiple SIMs at the same time is
allowed or restricted in order to support requirements from carriers for
dual SIM devices

Bug: 123537730
Test: Verified with test application.
Change-Id: I5e3e5ff41927e43ead4a30988630cf9125d5a048
Merged-In: I5e3e5ff41927e43ead4a30988630cf9125d5a048
(cherry picked from commit 398d5d91136d6e2146bf56ad3bdbb326a835c8ef)

Merge "Whitelist packages from VPN lockdown: DPM API."

Compile NetworkStack as system_current

Test: make NetworkStack
Test: flashed, WiFi and tethering working
Bug: 112869080
Change-Id: I2d805c0fefe271199f4f0e0d49d91e03f41a7eef

Remove last NetworkStack usage of hidden APIs

Includes various small changes to stop using hidden APIs

Test: make NetworkStack
Test: flashed, booted, WiFi and tethering working
Bug: 112869080
Change-Id: Id2830795a444f484b377ed6437435a1cd833697a

Use default argument for VintfObject::CheckCompatibility.

The default argument has been changed to ENABLE_ALL_CHECKS to DEFAULT.
Just use the default argument implicitly.

Test: boots (sanity)
Bug: 117914319
Change-Id: Ib3d44c89625abcce50c6f22253461417dd130f80

Merge "Add IntRange and FloatRange annotations"

move ServiceState.getDataRegState to greylist

given we don't have a public alternatives for dataregstate for Q,
thus we shouldn't move this API to app-compat dark list.

Bug: 123099466
Test: Build
Change-Id: I67531ce536e004e1bc0f764824a261fb51cd27fa

Add IntDef for SIP response code.

Add IntDef for ImsReasonInfo codes and fix typo based on API review
result.

Bug: 123223018
Test: Compilation
Change-Id: I97b56805d14ec27a45de4b7dbd9a29e612020a76

Call Screening / Caller ID API Changes

- Fix bug in call identification parcelable; was reversing the app name
and package name.
- Add @hide RoleManager consts for the Telecom roles; this will be
replaced by values in the library for public consumption as a later
date.
- Add new call direction attribute to the Call.Details class; this is
needed so that a CallScreeningService can know if a call is incoming
or outgoing.  This way it knows whether it needs to perform call blocking
or just caller ID.
- Update CallScreeningService API docs to explain how to request the
call screening role.
- Update InCallService API docs to explain the 3 roles related to this
service and how you declare an appropriate IncallService to fill each of
them.

Bug: 63966743
Test: Manual, CTS
Merged-In: Ic4547b8ead791690305f7be58e43272c4155b7fb
Change-Id: Ic4547b8ead791690305f7be58e43272c4155b7fb

Merge "Card ID APIs are public"

Add IntRange and FloatRange annotations

Bug: 123223163
Test: Build
Change-Id: I17f27075007df642ec93e5b762b0717da4a1e927

Whitelist packages from VPN lockdown: DPM API.

Bug: 77468593
Test: atest MixedDeviceOwnerTest#testAlwaysOnVpn
Test: atest MixedDeviceOwnerTest#testAlwaysOnVpnAcrossReboot
Change-Id: I7508070f828fb7150df5d9e34846535d57c6a7c3
Merged-In: I7508070f828fb7150df5d9e34846535d57c6a7c3