[API feedback] fixed API for NR

This CL included the follow change:
1. Change getPreferredNetworkTypeBitmap to
getPreferredNetworkTypeBitmask [b/126552419]
2. Change setPreferredNetworkTypeBitmap to
setPreferredNetworkTypeBitmask [b/126552419]
3. Add @IntRange annonation to CellIdentityNr.getNrarfcn() [b/126552419]

4. In javadoc, explain the acronym NR to say: (New Radio 5G)
[b/126552559]

5. Mark CellInfoNr.getCellSignalStrength and CellInfoNr.getCellIdentity
    as @NonNull [b/126699572]

6. Mark CellIdentityNr.getMncString() and CellIdentityNr.getMccString()
    as @Nullable [b/126702744]

Bug: 126552419
Bug: 126552559
Bug: 126699572
Bug: 126702744
Test: build
Merged-In: I133075a3688c3fa7550c8a2f43bc13d8b25999b2
Change-Id: I133075a3688c3fa7550c8a2f43bc13d8b25999b2

Merge "pm: SharedLibraryInfo: toString closing bracket"

Merge "improve mism support for phonestatelistner"

improve mism support for phonestatelistner

1. throw exception if reuse the same listner for different sub
2. subId should come from TM instance not phonestateListener.
3. refactor settings to apply phonestatelistner with TM instances

Bug: 117555407
Test: Manual
Change-Id: I1af07798d7982575cccc27462af179182c033409
(cherry picked from commit 0ebc9b67e96d0c4a625837ce7a32dbea4afa44ab)
Merged-in: I1af07798d7982575cccc27462af179182c033409

Merge changes from topic "framework-no-networkstack-aidl"

* changes:
  Remove the framework dependency on netd_aidl_parcelables-java.
  Move TcpKeepalivePacketData out of the framework.
  Stop using netd parcelables in the framework.

Merge "Fix the problem of CaptivePortalLogin app cannot show SSL error page"

Merge "Renamed NetworkRegistrationState to NetworkRegistrationInfo"

Merge "Add documentation for CaptivePortal APP_RETURN_*"

Merge "Convert tests/**/Android.mk files to Android.bp, part II"

Merge "Add Parcelable implementation for ApfCapabilities"

Remove the framework dependency on netd_aidl_parcelables-java.

In its own change because it conflicts in internal master.

Test: m
Bug: 128804404
Change-Id: I74531ef11d504b232d4f377532d5e0812dbc88db
Merged-In: I74531ef11d504b232d4f377532d5e0812dbc88db

Move TcpKeepalivePacketData out of the framework.

This class is not used in framework.jar, only in services.jar.
Move it out of the framework so framework.jar can stop depending
on the NetworkStack stable AIDL interfaces.

This should really under com.android.server as well, but that's
a cross-project change to be done in another CL.

Test: m
Bug: 128804404
Change-Id: Ib2805f7fcc516a5f6989c252365d1c95b8042703
Merged-In: Ib2805f7fcc516a5f6989c252365d1c95b8042703

Stop using netd parcelables in the framework.

The only actual users of these classes are in services.jar, not
in frameworks.jar. The only reason the framework depends on them
is that the code that converts to and from stable parcelables is
currently in the framework. Move that code to services and cut
the dependency.

These classes aren't used in the networkstack app so they don't
need to be in shared. They also can't be in shared because the
classes are not in the SDK. So put the conversion functions
directly inside their only user (NetworkManagementService).

Also remove the jarjar rules that rename the classes for use by
the NetworkStack app.

This does not actually remove the dependency from the build file,
that will be done in a future CL.

Bug: 128804404
Test: builds, boots
Test: atest FrameworksNetTests android.net.cts.ConnectivityManagerTest HostsideVpnTests
Change-Id: I027d50ba56091f5558f45e6e08f32e5912b2a82a
Merged-In: I027d50ba56091f5558f45e6e08f32e5912b2a82a

Merge "Add class comment to avoid extending ServiceState class."

Merge "Zygote: Load system server code early"

Merge "Add uid information in PackageListObserver"

Merge "Notify caller when unlockUser() bails out"

Zygote: Load system server code early

In an effort to allow loading integrity-checked artifacts from
the dalvik-cache, attempt to create and cache the system server's
classloader early, while still being in the system_server_startup
selinux domain.

The advantage of this approach is that allowances for loading
from the cache are restricted to startup.

Bug: 128688902
Test: m
Test: Device boots, picks up /system artifacts
Test: Device boots, picks up integrity-checked /data artifacts
Merged-In: If4a75fa106db09f1bd666d6d8df7ac3ac3e35a8c
Change-Id: If4a75fa106db09f1bd666d6d8df7ac3ac3e35a8c

Merge "Add unit tests related to data accounting for VPNs with one underlying network."

Merge "Extend light-greylist of some TelephonyManager APIs until Q."

Merge "Revert "Rename some failed causes""

Merge "Mark product apks as unbundled"

Add class comment to avoid extending ServiceState class.

ServiceState implements Parcelable and all 'Parcelables' are required to
be final, per the Android API Guidelines (go/android-api-guidelines).
This won't be changed for historical reasons, but clients should avoid
extending this class.

Bug: 128865231
Test: non-functional change that won't impact tests.
atest CtsTelephonyTestCases:ServiceStateTest
Change-Id: Ib369f1449a3e085f404fff1c706fda0b76d42890

Renamed NetworkRegistrationState to NetworkRegistrationInfo

API review changes. The class covers more information other than
registration state.

Test: Build
Bug: 127657991
Merged-In: I8af36b388182ab08257b64c37078f156d4821709
Change-Id: I8af36b388182ab08257b64c37078f156d4821709
(cherry picked from commit e27d3fde88d37332cc9970c79b25060c003e9f17)

Merge "Move the IpMemoryStore to the network stack."

Fix the problem of CaptivePortalLogin app cannot show SSL error page

This commit is trying to fix the limitation of older design:
1.There is no guarantee that onReceivedSslError() will be called
after onPageStarted(). So, onReceivedSslError() may compare the
host of SSL error url with mHostName which is unset.
2.Some WIFI APs may redirect to the probing url(mUrl) along with
special parameter first then redirect to the real log-in page,
it will make onPageStarted get unexpected hostname.
Above 2 situations will make captive portal app cannot show SSL
error page and result in captive portal app is blank.

The new design is trying to get the main frame url and compare
it with SSL error url. If the SSL error url is equal to main
frame url, then show the SSL error page.

Bug: 123208856
Bug: 122991421
Test: build pass
Change-Id: I27be7258436ccb833ff75ebcfed3af811467fa47

Add documentation for CaptivePortal APP_RETURN_*

This fixes API review requests.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 126883311
Change-Id: I0632a98675e65c5e722e81a2c1e0cf1403b54395

Merge "Fix API for SocketUtils"

Merge "Replace Handler with Executor for async DNS API"

Merge "Refactor answer callback for async DNS query JAVA API"

Revert "Rename some failed causes"

This reverts commit 1c405b73b096c5d20c610fb97012f7576bc0cf4e.

Reason for revert: <INSERT REASONING HERE>

Change-Id: I4af7c87d8a8a988c2f1c3dc6662027c6fcd34088

Merge "Fixing engineGetCertificateChain exception"

Merge "Remove deprecated entitlement API"

Add uid information in PackageListObserver

The uid information of a removed package cannot be retrieved by the
packageName anymore once it is removed. So it would be useful to provide
the uid of removed package in the onPackageAdded and onPackageRemoved
method of the PackageListObserver. This modification helps simplify the
design in PermissionMonitor.

Bug: 125396053
Test: dumpsys netd trafficcontroller

Exempt-From-Owner-Approval: Already merged in internal master
Merged-In: I2bd4bdf924687960a4fa3a47235bae68d885e445
Change-Id: I2bd4bdf924687960a4fa3a47235bae68d885e445

Merge "Address API review comments for VPN changes."

pm: SharedLibraryInfo: toString closing bracket

before:
SharedLibrary[...

now:
SharedLibrary{...}

Switched from square to curly braces because curly braces appear to be
the standard in this directory.

Bug: N/A
Test: manual
Change-Id: I02c63dd594fde68e645348cb1227f8594e2a7d3e

Merge "MediaHTTPConnection: move connection states into an inner class"

Merge "Emergency API review"

Merge "Add 5G icon to System UI"

Merge "Set sdk_version 28 in aidl_interface"

Merge "Mark EXTRA_INCOMING_NUMBER for PHONE_STATE broadcast as deprecated."

Emergency API review

1) Remove 'current' in Emergency APIs
2) Remove nullable
3) Handled the null Telephony service case
4) Hide getEmergencyServiceCategoryBitmask
5) Make getEmergencyUrns unmodifiable

Test: Treehugger
Bug: 122368381
Bug: 126699022
Bug: 126701211
Merged-In: I6eb1d8a3ae1625ff3f43f983c488c2969649713b
Change-Id: I6eb1d8a3ae1625ff3f43f983c488c2969649713b

MediaHTTPConnection: move connection states into an inner class

Bug: 119900000
Test: go/ag/5140159
Change-Id: I8edae85ea614dd537758f342c0c4b57be09c2da5

Merge "Cleanup annontations for Hearing Aids Profile API"

Merge "Remove unused HarfBuzz src in libandroid_runtime"

Merge "Add slot based ICC channel APIs"

Merge "Integrate FEATURE_TELEPHONY_IMS feature into APIs"

Merge "Add @UnsupportedAppUsage annotations"

Address API review comments for VPN changes.

Whitelist is now a Set rather than a list + javadoc improvement.

Bug: 124882903
Bug: 125346147
Test: atest MixedDeviceOwnerTest#testAlwaysOnVpn
Test: atest MixedDeviceOwnerTest#testAlwaysOnVpnAcrossReboot
Change-Id: Ieecaad083f2ab55de33587e6ce767f5f961683c9

Mark EXTRA_INCOMING_NUMBER for PHONE_STATE broadcast as deprecated.

In Q we have introduced new APIs which replace the need for an app to
get the phone number for calls through the phone state broadcast.  As
such the EXTRA_INCOMING_NUMBER is being marked deprecated.  Developers
are encouraged to use the new replacement APIs instead.

Test: Make API / build
Fixes: 128835519
Change-Id: If1bc0076392281887f6cc8017788a07f170c7a17

Merge "Add @UnsupportedAppUsage annotations"

Merge "Add hdmoon@/insun@ to OWNERS"

Add @UnsupportedAppUsage annotations

For packages:
  android.os

This is an automatically generated CL. See go/UnsupportedAppUsage
for more details.

Exempted-From-Owner-Approval: Mechanical changes to the codebase
which have been approved by Android API council and announced on
android-eng@

Bug: 110868826
Test: m
Merged-In: I4ece0a3f37f88fc2508cb965092aed7cabc61819
Change-Id: I0d942254f06c1a355fb906640e223ad34cced6ab

Fix API for SocketUtils

Addresses API review comments:
 - @NonNull on return values
 - short -> int for makePacketSocketAddress
 - Set class final while we're at it

Bug: 126700257
Test: m
Change-Id: I3b1775feee22b1e337b3e4e7a2ad16b28e305b67

Set sdk_version 28 in aidl_interface

default value of sdk_version in aidl_interface is changed.("28" -> "system_current")
so sdk_version of this module have to be "28" because it is included in framework.jar.

Test: m -j
Bug: 126469673
Change-Id: I515b1b236a7212198c419432b7c9fc70c623c63a

Move the IpMemoryStore to the network stack.

Test: atest FrameworksNetTests
Change-Id: Ic5bd6ff54b09a6fa92b6863f240a5b053011acb6

Merge "Remove NetworkParcelable"

Merge "Remove aidl wrappers for framework parcelables"

Merge "revent NFE in SystemUI when parsing invalid int (2)"

Replace Handler with Executor for async DNS API

Bug: 124882626
Test: built, flashed, booted
      atest DnsResolverTest DnsPacketTest

Change-Id: Ie1dc27643d4767f2a8a39da755edf388a00962d5

Refactor answer callback for async DNS query JAVA API

1. refactor AnswerCallback with a generic type
2. support onError in AnswerCallback
3. Fix minor problem reported from API Review

Bug: 124882626
Test: built, flashed, booted
      atest DnsResolverTest DnsPacketTest

Change-Id: I685c9989f8401acb63d2e83f552b2d5b20c41af0

Merge "CallRedirectionService API review"

Merge "ParcelFileDescriptor: improve javadoc."

Add unit tests related to data accounting for VPNs with one underlying
network.

This is to establish a baseline for the existing behavior, and to ensure
that following changes are not causing a regression in existing
behavior.

This CL is also adding missing cleanup for NetworkStatsCollectionTest
which was forcing all network types in NetworkTemplate that was causing
NetworkStatsService related tests to fail.

Bug: 113122541
Bug: 120145746
Test: atest FrameworksNetTests
Change-Id: I285f186cfb16bc9fa704c797996b1e4f8a73dee4

Cleanup annontations for Hearing Aids Profile API

Remove the @UnsupportedAppUsage and add the @IntDef for the Profile
Connection States in the Hearing Aids Profile API. Also, the parent
class BluetoothProfile has its Profile Connection States annontated.

Bug: 128523382
Test: Compile only
Change-Id: Ibd02516fa637ddb48d70a8dfacf607f047aec282

Merge "Cleanup unsupported appusage in Telecom apis."

Merge "Moved transport to IntDef"

Merge "Convert Android.mk file to Android.bp"

Extend light-greylist of some TelephonyManager APIs until Q.

The reasonable replacement for these APIs is
ImsMmtelManager#isAvailable(int, int); however that class is a system
API.  Given how late we are in the release cycle,  can make
ImsMmtelManager public R and leave these greylisted for now.

Test: Manual
Bug: 128696104
Change-Id: I64fa9e079e26ba16ff9ace098091c4094e9a7b33

Merge "[API feedback] api fixed for getSlotMapping"

CallRedirectionService API review

Bug: 126699669
Test: Treehugger
Change-Id: I56944f6d487660ed447d081b6dfd2cb625087869

Merge changes from topic "Porting-ONSAPIChanges"

* changes:
  Should not turn on validation for data
  Provide network bands in AvailableNetworkInfo

Merge "Update feature flag mapping owners"

Remove NetworkParcelable

With the @JavaOnlyStableParcelable annotation, Network can now be used
as-is as a framework parcelable.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 126477266
Change-Id: I5d9695a31b3ac48d10edcf08ccd0c34375353b79

Remove aidl wrappers for framework parcelables

The new @JavaOnlyStableParcelable annotation allows using the framework
parcelables directly, which removes a lot of boilerplate.

Includes new copy constructors for IpPrefix and LinkAddress so they can
be copied easily as-is into parcelables.

Test: atest FrameworksNetTests
Test: booted, WiFi works
Bug: 126477266

Change-Id: Icac8afe498d0b5ebf9a0d0b9eceb14d64a29b381

Merge "Integrate testNetworkService and Manager with Connectivity stack"

Merge "Fix ServiceState.hashCode() bug."

Fix ServiceState.hashCode() bug.

ServiceState currently overrides hashCode() incorrectly. It hashes an
Array field without using Arrays.hashCode(), meaning that the default
Object.hashCode() implementation is used (which is based on the address
of the Array field). This allows equivalent ServiceState instances to
produce different hashes.

Bug: 128609620
Test: atest CtsTelephonyTestCases:ServiceStateTest
Change-Id: I40a7d0b7ebab6d2a1be47ea63a6d223c4c13700b

Merge "Bugreport Manager service clear binder identity"

Add @UnsupportedAppUsage annotations

For packages:
  com.android.internal.telephony
  com.android.internal.telephony.cat
  com.android.internal.telephony.cdma
  com.android.internal.telephony.gms
  com.android.internal.telephony.imsphone
  com.android.internal.telephony.uicc

This is an automatically generated CL. See go/UnsupportedAppUsage
for more details.

Exempted-From-Owner-Approval: Mechanical changes to the codebase
which have been approved by Android API council and announced on
android-eng@

Bug: 110868826
Test: m
Merged-In: Ib70dd739d8067ca2e37f0dbc6c48a2d625935a4b
Change-Id: Ib7d46597f0d43a9e61141c96737976b02a1df008

Merge changes I39e84219,I8d0e12cb

* changes:
  Bugreporting API: Take ownership of fds.
  Don't manage dumpstate lifecycle from system server

Remove deprecated entitlement API

The deprecated entitlement APIs[1] is introduced in the Q
development cycle. It should be replaced by new APIs[2][3].

[1] https://android-review.googlesource.com/c/platform/frameworks/base/+/874220
[2] https://android-review.googlesource.com/c/platform/frameworks/base/+/920354
[3] https://android-review.googlesource.com/c/platform/frameworks/base/+/927513

Test: -build, flash, boot
bug: 126701557
bug: 126392011
Change-Id: I96487e9038c07363680a76db840994719d7ff768

revent NFE in SystemUI when parsing invalid int (2)

A user can change various tunable settings setting via adb.

If the value set is not an integer, SystemUI will end up in an exception
loop.

Test: No crash when running:
      adb exec-out settings put secure sysui_qs_move_whole_rows a
      adb exec-out settings put secure qs_show_brightness a
      adb exec-out settings put secure clock_seconds a
      adb exec-out settings put secure sysui_volume_down_silent a
      adb exec-out settings put secure sysui_volume_up_silent a
      adb exec-out settings put secure sysui_do_not_disturb a
Change-Id: If9c565cbdd44db25ba7fce381c98aa0ab735bfc4

Merge "[KA02.5] Use binder thread and executor to invoke callback"

Add hdmoon@/insun@ to OWNERS

Test: N/A
Change-Id: I53a29c098f3b201049baa39ea187cd2f9b5ef167

Add Parcelable implementation for ApfCapabilities

This is necessary to allow usage of ApfCapabilities as-is in AIDL,
instead of relying on ApfCapabilitiesParcelable, assuming that stable
AIDL starts allowing usage of @SystemApi classes. The Parcelable
implementation would be convenient for clients in any case.

Bug: 126477266
Test: atest FrameworksNetTests
Change-Id: Id2ef3cad261832a2918ccb6bb6bc154d99d75746

Merge "Remove SMSDispatcher.sendData() from greylist as it's params changed"

Moved transport to IntDef

Moved out from the sub class per API review feedback.

Test: Build
Bug: 128607082
Merged-In: I1b513bcaaa0ebf47c14593d962579ed48a7c6db0
Change-Id: I1b513bcaaa0ebf47c14593d962579ed48a7c6db0

Merge "Initialize the NetworkStack before WiFi"

Merge "API review: Telecom outgoing call Call redirection API"

Merge "Rename some failed causes"

Update feature flag mapping owners

Test: rebuild

Change-Id: I288e039ffaf88ce271ba17bcc26f0e038314b354

Initialize the NetworkStack before WiFi

The NetworkStack needs to be marked as start requested before any client
can use it from the system server, so it knows to queue requests until
it actually comes up.

This fixes a bug introduced in commit:
I93315ad31925bd436e21d1eabb21d5967e6aae60
where a client trying to use the network stack in the system server
before ActivityManager.systemReady() could block and eventually timeout,
resulting in a null INetworkStackConnector.

Test: booted, WiFi working
Bug: 128620654

Change-Id: Ieca2657373eba78195b387fb266fa0424fce8ada

Merge "Avoid IllegalArgumentException when bluetooth is disabled"

Merge "Explain the difference of isValid, isActive, isUsable"

Integrate FEATURE_TELEPHONY_IMS feature into APIs

Bug: 118823732
Test: manual
Merged-In: Iabb0e7cadae52d8ebe922f61a878298ad4dd04b0
Change-Id: Iabb0e7cadae52d8ebe922f61a878298ad4dd04b0

Convert tests/**/Android.mk files to Android.bp, part II

See build/soong/README.md for more information.

Bug: 122332340
Test: treehugger
Change-Id: I886b6536a0ef1c8d21f15ec7ff9fdb9784d5b865
Merged-In: I886b6536a0ef1c8d21f15ec7ff9fdb9784d5b865

Add slot based ICC channel APIs

These are SystemApis which require MODIFY_PHONE_STATE, and can be used
to open ICC channels when there is no available subscription, using the
slot index instead of subId.

Bug: 126578014
Test: builds
Change-Id: I3f47a8ac07813ea9e326c7d76123352708bfeea2
Merged-In: I3f47a8ac07813ea9e326c7d76123352708bfeea2

Remove SMSDispatcher.sendData() from greylist as it's params changed

Test: none
Bug: 120403639
Merged-in: Ib70dd739d8067ca2e37f0dbc6c48a2d625935a4b
Change-Id: I0218764fc02d3f6b9f12b3480aba9742a52b8413

Bugreport Manager service clear binder identity

Binder identity of the caller needs to be cleared before making a call
to ensureIsPrimaryUser() and is later restored for the rest of the code
to execute.

Test: Builds
Change-Id: Ida781b633efbea57a9442fb9a8a5f823f9acbe8b
Merged-In: Ida781b633efbea57a9442fb9a8a5f823f9acbe8b
(cherry picked from commit 99766179567464860585feeb669eb01619598a50)