Merge "Plumb PhysicalChannelConfig all the way up."

Merge "pm: Fix typo in error"

pm: Fix typo in error

instnat -> instant

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

Merge "Mark EUICC_PROVISIONED as @SystemApi."

Merge "Delay starting the webview_zygote until first use."

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

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

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

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

Merge "Cleanup TelephonyManager's log"

RTT bugfixes, part 5

* Add a new API to allow Telecom to inform ConnectionServices when the
RTT text stream changes
* No longer set the RTT property from ConnectionService. Client apps
should be setting properties themselves.
* Add Dialer-side RTT property in order to remove the dependence on
checking the RTT streams, which have a complex lifecycle

Bug: 72951201
Bug: 72648661
Test: manual, with real RTT calls and Dialer's SimulatorConnection, also
cts
Change-Id: Ic4c7d883d2dc6baf8e8c0eaa4df58d7de8762b9e
Merged-In: Ic4c7d883d2dc6baf8e8c0eaa4df58d7de8762b9e

Factor out telephony permission checks into a helper class.

The aim is to cut down on duplicate code (for a particularly high-risk
area) and make it easier to expand READ_PHONE_STATE access to
carrier-privileged apps in a follow-up CL.

No major functional changes are intended, although some logging will
change slightly (different tag / less verbose carrier privilege
logging without DBG flag).

Bug: 70041899
Test: bit FrameworksTelephonyTests:*, nothing visibly broken on boot
Merged-In: I3b2aa9b8d6177a25d532060009508ef31baea69c
Change-Id: I3b2aa9b8d6177a25d532060009508ef31baea69c

Merge "EMBMS API tweaks"

Merge "Check for location access on TelephonyRegistry"

Merge "Explicitly restart the killed wallpaper service"

Merge "Fix low probability deadlock"

Build android.test.base/legacy against SDK

android.test.legacy is now build against SDK, thus available to apps
build with SDK.

In doing so, android.test.mock.stubs had to be built with SDK (because
it is used by android.test.legacy). However, this wasn't trivial.

The problem was that some classes in test-mock implemented abstract
methods of platform classes. Even though they are marked as @hide,
doclava automatically emit them to the stub file because otherwise the
class in the stub file does not implement some abstract methods from its
parent class, which in turn makes the stub file non-compilable.

This CL solves the problem by introducing an adapter class in between
the class in test-mock and the class in the framework. The adapter class
implements abstract methods which are @hide and having reference to
types that aren't available to SDK. The abstract methods are removed
from the original class. Then doclava does not emit the abstract
methods.

Also by @hide'ing the adapter classes, they are not compiled when
building the stub library.

Bug: 30188076
Bug: 73339598
Test: m -j android.test.base
Test: m -j android.test.legacy
Test: m -j android.test.mock.stubs
are successful and do not show link-check warning
Merged-In: I8e432950e693ee3c6f0240852e62da6133d31571
Change-Id: I8e432950e693ee3c6f0240852e62da6133d31571

Cleanup TelephonyManager's log

Don't log success calls to TelephonyManager.setTelephonyProperty.

Bug: 71514398
Test: manual test
Change-Id: I2383f1e2925bff959c3d925d7fdc8be891630c61

Fix low probability deadlock

Low probability deadlock in com.android.phone and system_server.
Bug: 70865607

Test: Build pass

EMBMS API tweaks

Make changes per API council recommendations
Also clean up the download flow -- provide option for app to specify a
destination as long as it's on the same filesystem.

Bug: 72837388
Fixes: 72231372
Test: CTS and testapp

Change-Id: Ia2a68a73e6d61f426b55068542a66995afc20407

Merge "Reparent the webview_zygote to be a child of the main app_process zygote."

Reparent the webview_zygote to be a child of the main app_process zygote.

This uses the new ZygoteProcess.startChildZygote() method to launch the
webview_zygote, rather than having init start it. This will share more
memory between the app_process and the webview_zygote, reducing the
overall system footprint.

Bug: 63749735
Test: m
Test: Launch "Third-party licenses" activity from Settings, and it
      renders correctly via the WebView.
Change-Id: I3e39cd8adb9c099c92ee34640428916d90cb2b8f

Merge "Consider shared libs when precreating WebView classloader."

Merge "Add new reason code for call failed by unobtainable number"

Merge "Change KEY_CARRIER_VOLTE_TTY_SUPPORTED_BOOL to false"

Consider shared libs when precreating WebView classloader.

The change to handling of the deprecated Apache HTTP library means that
the WebView implementation package might have a non-empty shared library
list. Make sure to fetch shared libraries when querying WebView
implementations, and take them into account when constructing both the
actual classpath to be used to precreate the classloader, and when
deciding what the cache key for the precreated classloader should be.

Change-Id: I5e1409358d935e1c9f325db434bc6d4ef8ead759
Merged-In: I5e1409358d935e1c9f325db434bc6d4ef8ead759
Fixes: 65574359
Test: launch anything that uses WebView
(cherry picked from commit 4fd8aa51e4de5a7c93841571e30ff10077d1b125)

Merge "Fix ConnectivityServiceTest for NATT Keepalive"

Merge "Pass DEXOPT_INSTALL_WITH_DEX_METADATA_FILE for first boot"

Modify shouldProcessCall API to remove redundancy

No need to differentiate between CSFB for emergency
and non-emergency calls.

Test: Telephony unit tests
Bug: 72642113
Change-Id: Ibc5aed284be030cc584d774f122d6082ff013f5f

Add more sms error codes

To bubble up more accurate codes from the radio so that the IMS provider
can handle accordingly.

Bug: 73487630
Change-Id: Ie482ca57504e1ecf77360940b7e1d3948d5f3e82

Merge "WifiTile: Avoid NPE during boot phase"

Merge "Add flag to AMS.startInstrumentation() to disable hidden API checks"

Hide APN, USER and Password from log in SETUP_DATA_CALL

Some operators don't want to show apn, user, password.

Test: manual
Bug: 35411757
Change-Id: I357fd561d5b8aa9c6596786c9731e8f3cd9c532c

Change KEY_CARRIER_VOLTE_TTY_SUPPORTED_BOOL to false

Bug: 73609752
Test: compile

Change-Id: Ie9615e63d136f40ee301504b3e2b3b7ea4aec129

Add flag to AMS.startInstrumentation() to disable hidden API checks

Some tests need to use hidden APIs to check the internal state of
the framework. For those special use cases, we add a new flag to
ActivityManagerService.startInstrumentation that enables to start
instrumented processes without hidden API enforcement. Individual
test harnesses can change their Am command to request the exemption.

Bug: 64382372
Test: adb shell am instrument --no-hidden-api-checks <component>
      adb logcat | grep 'Accessing hidden'
Merged-In: I1d734a95423fae90dae63ff09d5f606495830905
Change-Id: Ifbb0d19d95737a9f601b11e20352cdc11f0ca5b6

Merge "Turn on DepAnn for test-base and test-runner"

Add new reason code for call failed by unobtainable number

To show the "Invalid number" on the end call screen when user makes a
IMS call with unobtainable number, this reason code is used.

Test: manual - Checked that "Invalid number" is shown when make the IMS
MO call to unobtainable number.
Bug: 69940434

Change-Id: If3c54e6f55850141efad56ad7712db3c82a14f43