Merge changes from topics "cdma-null-island", "cdma-signal-strength"

* changes:
  Set CDMA Location to Invalid if on Null Island
  Fix Range-Checking in CellSignalStrengthCdma
  Fix CDMA Range Checks for SignalStrength

Merge "Expose Carrier Name Override CarrierConfig Values"

Merge "Expand use of 'Str' suffix in CellIdentity to 'String'"

Merge "Extend network request to query networks w/o capabilites"

Merge "Add result code for eUICC card not found."

Merge "Give apps with NETWORK_SETTINGS right to see any VPN."

Merge "Give VPNs the INTERNET capability when they route most of the IP space"

Give apps with NETWORK_SETTINGS right to see any VPN.

...not only the ones that apply to them.

Bug: 73217368
Test: runtest frameworks-net and CTS
Change-Id: I436972a3e51e98bdd815771b451bcedadf684763

Give VPNs the INTERNET capability when they route most of the IP space

Test: manual, plus wrote some new tests for this
Bug: 72765718
Change-Id: I9759da72b752fd8eeb1d0647db9ab341f04c0528

Expose Carrier Name Override CarrierConfig Values

These CarrierConfig values are needed for a
CC app to override the SPN dynamically. Since
Carrier names are very complicated, it's extra
important that these values be exposed so that we
can properly enforce correct behavior through CTS.

Bug: 73734601
Test: compilation (no functional change)
Change-Id: I26dab1bd32d1d5273f707a37026cffca7fd639a3

Extend network request to query networks w/o capabilites

Add another bit mask into NetworkCapabilities class that represents
unwanted capabilities and corresponding methods to add and remove them.

Bug: 72828388
Bug: 68762530

Test: runtest -x frameworks/base/tests/net/
Change-Id: Ie291167eb74fdb696c7ee145c8cc46ea8115d6ba

Merge "Remove cell info from legacy apps without location on"

Add result code for eUICC card not found.

Bug: 38206971
Test: test on phone
Merged-In: Ia783fe68389d950b664ad312d98c7e3aa200b471
Change-Id: Ia783fe68389d950b664ad312d98c7e3aa200b471

Merge "Added indication update mode support"

Merge "Bluetooth: Remove adjustAvrcpAbsoluteVolume"

Bluetooth: Remove adjustAvrcpAbsoluteVolume

The AudioManager now handles all calculations for volume stepping now
and no longer uses adjustAvrcpAbsoluteVolume.

Bug: 68812037
Test: Compile
Change-Id: I9cbf7989e49738c7a43fe3142aced5803111271e

Added indication update mode support

Adde the indication update mode support so that a system
component can control the behavior of indication update. This
will be used by the bluetooth stack when some BT devices such like
carkit is connected, modem will continue update the signal strength
even when the screen is off.

Test: Manual
Bug: 65112388
Merged-In: I4bb4894eaaba401f655e5dc25138275f5e8498e1
Change-Id: I4bb4894eaaba401f655e5dc25138275f5e8498e1
(cherry picked from commit 1a1c35750b2612e040c9392b4d0d6f4e1a2b3b8a)

Merge "Tweak DateFormat.is24HourFormat javadoc to be more explicit."

Tweak DateFormat.is24HourFormat javadoc to be more explicit.

Bug: 32761394
Test: make docs
Change-Id: If10da5b9d63e6e88774e333d5c266e423e6bc7c8

Merge "frameworks/base: Set LOCAL_SDK_VERSION where possible."

Merge "Fix up NetworkCapabilities' toString"

Merge "Plumb PhysicalChannelConfig all the way up."

Merge "pm: Fix typo in error"

Expand use of 'Str' suffix in CellIdentity to 'String'

Expand use of 'Str' suffix in CellIdentity to 'String' to match general
usage across Android according to the API Review.

Bug: 73751308
Test: Unit test
Change-Id: Ib25a06056832be610ff1fb9c025467259eb543ed

pm: Fix typo in error

instnat -> instant

Change-Id: Ie2f06840575fc266e3510252635ad17be65945b8
Signed-off-by: Joe Maples <joe@frap129.org>

Merge "Mark EUICC_PROVISIONED as @SystemApi."

frameworks/base: Set LOCAL_SDK_VERSION where possible.

This change sets LOCAL_SDK_VERSION for all packages where
this is possible without breaking the build, and
LOCAL_PRIVATE_PLATFORM_APIS := true otherwise.

Setting one of these two will be made required soon, and this
is a change in preparation for that. Not setting LOCAL_SDK_VERSION
makes the app implicitly depend on the bootclasspath, which is
often not required. This change effectively makes depending on
private apis opt-in rather than opt-out.

Test: make relevant packages
Bug: 73535841
Change-Id: Ibcffec873a693d1c792ca210fb597d2bf37e9068
Merged-In: I4233b9091d9066c4fa69f3d24aaf367ea500f760

Merge "Delay starting the webview_zygote until first use."

Merge "Don't use cutils/Atomic.h"

Fix up NetworkCapabilities' toString

Test: manual
Change-Id: I3bcec6a6873e8ec7ced0820d3d5b92249b19fe0a

Merge "Notify the user and turn off tethering when the service is disallowed."

Notify the user and turn off tethering when the service is disallowed.

Added UserRestrinctionListener for turning the service off one the
DISALLOW_CONFIG_TETHERING is on into Tethering.  Added notification
about
tethering being turned off. Also added Unit Tests to test the
functionality
of the UserRestrictionListener added.

Bug: 27936525
Test: Turn the tehering service on (either wifi, usb or bluetooth).
Automatically the system should send a notification about the service
being active.Close settings from recents. From TestDPC User Restrictions
switch on DISALLOW_CONFIG_TETHERING. The tethering should be turned off
and a notification should appear informing that the service is inactive.

Merged-In: Ib7ea8885cedc2a842ebd4487c8b366a6666996bc
Change-Id: I4a57137a7ad592ca186d9508d5cc2fad3f1bc985

Merge "Add OEM_PAID network capability"

Mark EUICC_PROVISIONED as @SystemApi.

Bug: 35851809
Test: test on phone
Merged-In: I1627aeaf6846e889767fb4223c46fa278a751b23
Change-Id: I1627aeaf6846e889767fb4223c46fa278a751b23

Merge "Don't throw exception if phone process is dead for carrier ID APIs"

Merge "Modify shouldProcessCall API to remove redundancy"

Merge "Add documentation for SECRET_CODE_ACTION according to API review process."

Don't throw exception if phone process is dead for carrier ID APIs

Bug: 73772776
Test: Build
Change-Id: I81638f52d5d8ccf1005878ba4f3967e07169284b
(cherry picked from commit 331965e5d6f753cd061303607cdfbe9dff4be896)
Merged-in: I81638f52d5d8ccf1005878ba4f3967e07169284b

Set CDMA Location to Invalid if on Null Island

If the reported CDMA location is ~= (0, 0), which
is in the middle of the Gulf of Guinea, assert that
there are no CDMA cell towers within range (there
are not) and force the location to a saner default
value of Integer.MAX_VALUE which is out of the range
of valid lats+longs.

Bug: 32364031
Test: runtest frameworks-telephony
Change-Id: I3f50054dd37cf7cef56b1bd16c3313c02da34c31

Fix Range-Checking in CellSignalStrengthCdma

The CellSignalStrengthCdma class previously allowed
the values in the class to be kept as negative ints
but expected them to be parceled as positive ints.
This led to a confusing mess that is best unwound
by calling the actual constructor for the class and
letting the parcel values be an implementation detail.
This CL removes all of the parcel-time coersion and
instead expects that the class be constructed using
a constructor rather than by manually parceling and
then using the class to un-parcel. In addition, the
range checking for inputs is now done only once, and
values are no longer mutated in the parcel/unparcel
process.

Bug: 32364031
Test: runtest frameworks-telephony
Change-Id: I59ce8c9df1bd99547f3de941a30d6c3cea8f2b8f

Fix CDMA Range Checks for SignalStrength

-Allow zero as a valid value for CDMA ECIO. Zero
 is allowed for EVDO ECIO and is equally valid for
 CDMA. Making them consistent by allowing zero here.
-Set EVDO ECIO to -160 if unreported rather than
 setting it to -1. The "unreported" value is
 undocumented, and since -1 is well within the range
 of valid values, makes no sense. Since CDMA ECIO was
 setting an unreported value to a very low number,
 again making them the same.
-Allow 0 for EVDO SNR. This value has a range that is
 documented both in the RIL and in SignalStrength to
 include zero, but we were previously disallowing 0.
 Making the range check inclusive in line with the
 existing documentation, which was self-consistent.

Bug: 32364031
Test: runtest frameworks-telephony
Change-Id: Ie0ca5abb4998d1b0b5abdbff9d51f364fe6db858

Merge "Use 4 thresholds instead of 6"

Merge "Fix: Double-free error on RemoteFillService"

Use 4 thresholds instead of 6

Min and max thresholds are fixed.

Bug: 73775507
Bug: 70698348
Test: manual and ServiceStateTrackerTest
Change-Id: Ie7fbda0627615f49b6205142c22ad48e88735f80
Merged-In: Ie7fbda0627615f49b6205142c22ad48e88735f80

Merge "AudioManager: Use setAvrcpAbsoluteVolume in adjustStreamVolume"

Merge "Frameworks: Annotate JUnit4 test with @Test"

AudioManager: Use setAvrcpAbsoluteVolume in adjustStreamVolume

Use setAvrcpAbsoluteVolume() instead of adjustAvrcpAbsoluteVolume in
adjust stream volume in order to better sync device volume and remote
Bluetooth device volume. This will allow Bluetooth to avoid calculating
its own volume step size.

Bug: 68812037
Test: Connect to device that supports Absolute Volume and see that
      volume changed occur when changing volume on device.

Change-Id: I26d5812ee1a5953b6527561ce5ad1c5d944c244d

Merge "BackgroundDexOptService: skip low battery check if not present"

Merge "Remove icu4j dependency."

Merge "Bluetooth: Add config value for max connected audio devices"

Bluetooth: Add config value for max connected audio devices

Bug: 64767509
Test: read config values in Bluetooth app
Change-Id: Ia5dd2fc1c16272082f11ac886e02b838575ceedd

Merge "Add more sms error codes"

BackgroundDexOptService: skip low battery check if not present

If no battery is present, assume no need to check for low battery.

Do not interpret battery level values if battery is not present.

Bug: 34507420
Test: manual (temporary log messages with battery forced not present)
Change-Id: Iebc64bc890808d583d0bd95a31f17b6363b7ba40
Merged-In: Iebc64bc890808d583d0bd95a31f17b6363b7ba40

Merge "Hide APN, USER and Password from log in SETUP_DATA_CALL"

Merge "Frameworks: Annotate JUnit4 test with @Test"

Merge "Add support for sending message with priority, link control and validity period options"

Delay starting the webview_zygote until first use.

During boot, WebViewZygote.setMultiprocessEnabled() is called by the
WebView initialization logic. Starting the WebViewZygote here causes a
slowdown in the system_server boot process, so delay launching the
zygote until it is needed.

Previously the webview_zygote was launched by init, and merely
connecting to it in the boot process didn't have significant overhead.

Bug: 73743583
Bug: 63749735
Test: Boot a device, verify that webview_zygote process is not running.
Test: Launch "Third-party licenses" activity from Settings, and it
      renders correctly via the WebView.

Change-Id: I1352a5df95e4a793ac64862c439ba2573ddd2d18

Remove icu4j dependency.

This dependency is no longer required: it was added
for libcore when libcore was implemented in terms of
com.icu classes. libcore has since moved to android.icu
(in core-libart) and so the droiddoc dependency is no
longer required.

Test: make droid docs
Merged-In: I6e4e79a7df201fdc2cefa60037adf831593eaa0f
Change-Id: I6e4e79a7df201fdc2cefa60037adf831593eaa0f

Merge "Add hashCode() in UiccAccessRule."

Add hashCode() in UiccAccessRule.

hashCode() and equals() should appear together.

Bug: 73785903
Test: test on phone
Merged-In: I7f9d8943f6a0e1039815e21ff6dfe035bd674639
Change-Id: I7f9d8943f6a0e1039815e21ff6dfe035bd674639

Fix: Double-free error on RemoteFillService

Symptom:
RemoteFillService was crashed due to IllegalArgumentException
"Service not registered:" at onServiceConnected.

Root cause:
RemoteFillService#onServiceConnected tries to unbind the connection
if mDestroyed is flagged or mBinding is not flagged. It always fails
with IllegalArgumentException.
Both mDestroyed and !mBinding mean the connection was unbound.
You can't unbind the unbound connection. It's not allowed.

Fixes: 73864601
Fixes: 69905688

Change-Id: If5481468ddac7be41accad63e9d5382bc6c029fd

Merge "Use assigned UUID for Hearing Aid Service"

Use assigned UUID for Hearing Aid Service

Test: manual
Bug: 64038649
Change-Id: Iab748db944df3bba8927973d0a55d95514b9f8ed

Frameworks: Annotate JUnit4 test with @Test

Mollify Errorprone.

Bug: 72076216
Test: m javac-check RUN_ERROR_PRONE=true
Test: atest IpSecServiceParameterizedTest
Change-Id: Ia3a253c4c5994937efc0f498ac047c5fb4eee3e9

Frameworks: Annotate JUnit4 test with @Test

Mollify Errorprone.

Bug: 72076216
Test: m javac-check RUN_ERROR_PRONE=true
Test: atest CredentialTest
Change-Id: I07635124a306f82e5442a45a1fd61edc63c015a2

Merge "Slightly refactor IPv4 default router interface selection"

Merge "Definitely abort reconnecting to native daemon during shutdown"

Merge "Add device capabilities."

Merge "Fix NPE in NetworkMetrics"

Merge "Let sendShowMessage called from hide to show"

Add support for sending message with priority, link control and validity period options

1. Added support for message priority(indicates the priority of a
   SMS to the network in the PDU) in 3gpp2
2. Added support for link control(indicates whether network can
   expect more messages or not so it can decide whether to retain
   the radio link or not) while sending multiple messages
3. Added support for validity period(indicates how much time the
   SMS is valid at the SMSC) support in 3gpp
4. Amended UT test cases for API changes and add UT validity period
   test case

Test: Verfied sending short SMS, long SMS, MMS, SMS with emoji on
      live network

Bug: 34962629

Merge "Wrap raw binder interface into public classes"

Add device capabilities.

Bug: 72223709
Test: Unit test
Merged-In: I35a74e88767f772f764734d1f535848ba43629b9
Change-Id: I35a74e88767f772f764734d1f535848ba43629b9

Don't use cutils/Atomic.h

Test: builds
Change-Id: I74485a5cbecb8710714f7bf3e54da61dd787838f

Fix NPE in NetworkMetrics

addTcpStatsResult() in NetworkMetrics was not checking for the nullness
of the pendingSummary object and trying to add stats regardless.

This patch fixes addTcpStatsResult() so that it populates pendingSummary
if necessary, similarly to addConnectResult() and addDnsResult()

Bug: 73825228
Test: runtest frameworks-net
Change-Id: I616a4be0e6e5920bd2f5c58292fea769bd516df7

Slightly refactor IPv4 default router interface selection

Test: as follows
    - built
    - flashed
    - booted
    - USB tethering to dualstack wifi yields log lines like:

        14:33:31.970 - Looking for default routes on: {InterfaceName: wlan0 ...
        14:33:31.970 - IPv4/IPv6 upstream interface(s): wlan0/null

      and working tethering
    - runtest frameworks-net passes
Bug: 64380515
Bug: 64382985
Bug: 64976379
Change-Id: Iac94d870653ae1b71df00e8179cc4ecdb7ac412f

Add documentation for SECRET_CODE_ACTION according to API review
process.

 * Document who can send and who can receive the broadcast. What are the security restrictions?
   The implication here is that there should be some. What are the wake-up semantics?
 * Document who can receive SECRET_CODE action and under what circumstances,
   and what permissions are needed to receive and send it.

Bug: 73751267, 73392896
Test: Basic telephony sanity
Change-Id: I1b6138c9ddf4cb3d84d8b652e18d4e57f410ee4d

Merge "Factor out telephony permission checks into a helper class."

Merge "Frameworks: Remove bogus test method"

Merge "Don't wait keyguard drawn if device has no keyguard"

Merge "Fixed service state CTS test failing"

Merge "Removed executable flag for Java source files"

Removed executable flag for Java source files

Test: Code compilation
Change-Id: Ibfb57c6af54ace94eb9f9915471738f2f09ca70a

Merge "Add comments according to the API review process."

Frameworks: Remove bogus test method

It's tripping Errorprone's JUnit3 test pattern.

Bug: 73797534
Test: m javac-check RUN_ERROR_PRONE=true
Change-Id: Ib0b9b342e085ba640d60e82ebd9a1bbb4fc4e916

Wrap raw binder interface into public classes

Test: Dummy application using the API
Bug: 73751309
Change-Id: If697100ed28b6b5217367d9988ed45f8baa14c34

Fixed service state CTS test failing

Added missing member variables in copyFrom().

Bug: 73298491
Test: atest CtsTelephonyTestCases:ServiceStateTest#testServiceState CtsTelephonyTestCases:TelephonyManagerTest#testGetServiceState
Change-Id: I750816a13801a911d68e0a774fd59602c81c5687

Merge "Allow callers to synchronously block for shutdown"

Merge "Add AudioManager.setHearingAidDeviceConnectionState"

Add AudioManager.setHearingAidDeviceConnectionState

This will be used by Bluetooth to notify audio framework when Hearing
Aid is connected or disconnected.

Bug: 69623109
Test: mm
Change-Id: I5664d8bd20e36601eea815c0d6e55c3b1d49e4ea

Remove cell info from legacy apps without location on

No longer allow legacy apps to access cell info location data when
location is turned off in settings.

Bug: 69637693
Test: manual
Change-Id: Ibff3cc75898dc189632f2f9427892423a404333f

Merge "RTT bugfixes, part 5"

Allow callers to synchronously block for shutdown

Add a simple CountDownLatch and a public method that callers can use
to block until IpClient has cleanly shutdown the state machine.

In cases where IpClients are created and destroyed dynamically for
the same interface name, this can be used to make sure only one
IpClient at a time is touching the given interface.

Test: as follows
    - built
    - flashed
    - booted
    - OTG ethernet works (i.e. client mode)
    - removing and reinserting the ethernet dongle continues
      to show the <-> Ethernet sysui icon and basic network
      connectivity works
    - removing and reinserting the cable (link up/down events)
      also shows the <-> Ethernet sysui icon and basic network
      connectivity works
Bug: 62476366

Change-Id: If4bffd54b7ebc088ec07cac10251e451f8161b6c

Merge "Populate hiddenapi whitelist."

Add comments according to the API review process.

 * Explain what developers should use instead in the @deprecated note for Telephony#BEARER
 * Please add more documentation for network type bitmask - reference to just TelephonyManager is not sufficient because it has so much stuff in it.
 * Also remove the @hide tag for private constants in ServiceState.

Bug: 73751228
Test: Basic telephony sanity
Change-Id: Ie7405a961db4494104750bf406f20b1b4fb8191a

Merge changes I8e432950,Id325b8cf

* changes:
  Build android.test.base/legacy against SDK
  Fix visibility issues with android.test.mock.stubs

Merge "Package whitelist for hiddenapi checks."

Populate hiddenapi whitelist.

This list of packages is derived from all packages in the tree that do not
specify LOCAL_SDK_VERSION in their Android.mk. It was done by:

1.  Modifying biuld/core/package_internal.mk to output a warning for all
    packages that do not specify LOCAL_SDK_VERSION, including some
    metadata about the target.

2.  Parsing the output fom the build, then filtering out those that specify
    a test certificate, or with LOCAL_MODULE_TAGS := tests, that depend
    on android.test.runner or whose name includes "Test"

3.  Look up the java package name of each & build the list from these.

Test: N/A, this change is a no-op while all system apps are excluded
from the checks.

BUG: 73244707

Change-Id: Ie3dcb1b4b83c32e93609bc58dc7a1cd513eeea64

Package whitelist for hiddenapi checks.

Some packages in the system image are not built against the SDK, so
enforcing hiddenapi checks for these will break them. Add a whitelist
for such packages.

For now, just add the contacts provider to the whitelist. The list will be
further populated later.

Test: Added test app package name to whitelist to verify.

Bug: 73244707

Merged-In: I94746b7f12dd9371d5068bb235eb853f63ee4d97
Change-Id: I1cbbd220c61b1b4b767c301f97096607ee902a7b

Don't wait keyguard drawn if device has no keyguard

Instead, call finishKeyguardDrawn() if device has
no keyguard to avoid redundant waiting times.

Bug: 72972510
Change-Id: Id756de05691be2593fa06d3b55e5e24ad1ea7b19