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

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

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"

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

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

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."