Merge changes from topic "hidl-system-api"

* changes:
  HIDL: add APIs used by subclasses to SystemApi
  Update HIDL-related documentation.
  Add remaining missing HIDL SystemApi methods.
  HIDL: reportSyspropMethod -> enableInstrumentation
  Allow HIDL to be used in priv-apps.

Merge "Revert "Don't unnecessarily reevaluate tethering provisioning""

Revert "Don't unnecessarily reevaluate tethering provisioning"

This reverts commit 8067d78c32f545db6d35c660279ab3f2326ba41d.

Change-Id: I9098f98cb134c35692c1384339f9407b94ce6f28

Merge "Don't unnecessarily reevaluate tethering provisioning"

Merge "Trigger NetworkCallback events when private DNS usage has changed."

Trigger NetworkCallback events when private DNS usage has changed.

Relies on events sent from netd in aosp/578162.

Test: Added tests to ConnectivityServiceTest. Added a new test
class DnsManagerTest. Built a simple app that appears to
receive onLinkProperties events correctly upon manual changes
to the private DNS settings on a Pixel.
Bug: 71828272
Change-Id: I68665aaf74b7d59182cc6f9586b80b55b0dfe427

Merge "Support bluetooth not using MultinetworkPolicyTracker"

Support bluetooth not using MultinetworkPolicyTracker

This required advancing the IpManager to IpClient refactoring
(Bluetooth used IpManager and friends). Most importantly, the
Bluetooth code used WaitForProvisioningCallback, so this is
moved into IpClient proper now. Also: some more renaming
cleanup.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 62476366
Bug: 72663849

Change-Id: I9eee2a9d5c31cdd48a054a7edc3403584efb1864

Don't unnecessarily reevaluate tethering provisioning

Registering for carrier config changes can deliver a sticky broadcast
and can cause Tethering to think something has changed and reevaluate
provisioning status, even though this has been checked before it
entered tethering mode alive state.

Additionally, move the provisioning_app{,no_ui} resources into the
TetheringConfiguration, if for no other reason than now we can log
it in .toString().

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - manual USB tethering toward WiFi works
Bug: 69565814
Change-Id: Ib8b2620ce44c55e5eb0afd3f00f3f5aa4fc8a593

Merge "Bluetooth: Add hidden API to get current user of HID Device (1/3)"

Merge "Clarify doc comment for android.os.Build.VERSION.RELEASE"

Merge "Correct the value of RESOURCES_UNAVAILABLE_OR_UNSPECIFIED"

Clarify doc comment for android.os.Build.VERSION.RELEASE

The RELEASE field has no guaranteed internal structure and developers
should not attempt to parse it.

Test: no code changes
Change-Id: I0ea3ab5ba5dd4cc733feb62725cf7c62c00c857b

Correct the value of RESOURCES_UNAVAILABLE_OR_UNSPECIFIED

Test: Build
Bug: 77657541
Change-Id: Ia6b576c222776cbc7e54e15cb54e39775e76ea56

Merge "Remove an API that doesn't exist."

Merge "Move the logic of (re)evaluation of Private DNS"

Move the logic of (re)evaluation of Private DNS

Moves this out of ConnectivityService and into each NetworkMonitor
(where it's more self-contained).

Test: as follows
    - builds, flashes, boots
    - runtest frameworks-net passes
    - manual testing with working and non-working hostnames behaves
      somewhat (but not entirely) as expected, and not always quickly
Bug: 64133961
Bug: 72345192
Bug: 73872000
Bug: 77140445
Change-Id: Ic4322af3cb49149f2d975cb31f54b2ac7927f907

Remove an API that doesn't exist.

Apparently this was removed in Jelly Bean:
https://stackoverflow.com/q/24312632/632035. Keeping the decl in the
NDK just confuses people: https://stackoverflow.com/q/49646596/632035.

Test: make checkbuild
Bug: None
Change-Id: If549fcd8f4052e633646c4ce8a054a0bbcd8b2bf

Merge "Remove duplicate definitions of utility variables"

Merge "Add Polish keycharactermap"

Merge "Hearing Aid: change get/set active device (3/3)"

Merge "Modify isShowing method in Dialog"

Merge "Force creation of Socket upon Transform application"

Merge "throw runtimeException for applicable TM APIs"

Modify isShowing method in Dialog

The method isShowing in Dialog.java is to judge whether
the dialog is currently showing, but it will return true
when the dialog is hidden.

Bug: 74416088
Test: modify testShow in DialogTest.java (see 639879)
run cts -m CtsAppTestCases -t android.app.cts.DialogTest#testShow

Change-Id: Ibf15e080678f414f8a66ff0841be17a2f9d2d652
Signed-off-by: tiansiming [田思明] <tiansiming@xiaomi.com>

Merge "Add dedicated flag to control app image generation"

throw runtimeException for applicable TM APIs

1. This CL revert some API change from ag/3710629.
2. throw runtime exception when running in a non-1000 UID.
3. throw for methods that persist some sort of state and readers could continue to
return placeholder values.
4. for rest of TelephonyManager API cleanup b/74401420.

Bug: 74016743
Test: Build
Change-Id: I2846efc11eebce4a923762f56f09daaf37c44763
(cherry picked from commit f20f0d916f2326bee618600119107da1a8ae15a8)
Merged-in: I2846efc11eebce4a923762f56f09daaf37c44763

Force creation of Socket upon Transform application

This change forces Socket and DatagramSocket to populate the
SocketImpl, ensuring that the socket file descriptor can be
retrieved when applying Transport mode Transforms

This is done by calling getSoLinger(), triggering a getImpl(), which
triggers setImpl() if needed.

Bug: 77491294
Test: Added tests in IpSecManagerTest, ran on walleye
Change-Id: I40da08b031357710eb794e0f866aec5660c79594

Add dedicated flag to control app image generation

Also disable the flag for split APKs.

Bug: 72696798
Test: manual

(cherry picked from commit a9c343386d5505d7111aee3c1ffec409c9730f21)

Merged-In: I25007500e221744a218b165dde41ca9db19b1832
Change-Id: I1dc3137307834d948b52af18205ca47dc8dc7e67

Merge "Support bypassing TLS in the framework"

Bluetooth: Add hidden API to get current user of HID Device (1/3)

Bug: 69136526
Test: test with apps using HID Device profile
Change-Id: If0e49840257c877c975c2da176a08e613668cbc3

Merge "Frameworks: In the kill handler, ensure the logging handler was run"

Merge "Add an Executor for the onConnectedListener"

Merge "Address IMSI privacy issues."

Frameworks: In the kill handler, ensure the logging handler was run

In the KillApplicationHandler for uncaught exceptions ensure that the
LoggingHandler has been run. This ensures logging when code directly
calls getUncaughtExceptionHandler().uncaughtException().

Bug: 29624607
Bug: 73380984
Test: m
Test: manual
Change-Id: I9c9216714b4cf029d7ed21e29313c0e802345337

Support bypassing TLS in the framework

This change comprises the following parts:

    [1] android.net.dns.ResolvUtil, containing methods that encapsulate the
        use of the high bit in netids used in DNS resolution contexts.

    [2] Updates to captive portal apps to call the ResolvUtil method that
        enables DNS-over-TLS bypass for the captive portal app process.

Test: as follows
    - builds
    - flashes
    - boots
    - runtest frameworks-net passes
Bug: 64133961
Bug: 72345192

Change-Id: I2072c1f68d6978fa0d7e9d8693135a2c51bb0f87

Address IMSI privacy issues.

Added method resetCarrierKeysForImsiEncryption
to allow for keys to be reset.

Bug: 35606429
Test: manual
Change-Id: Id2e92917b50fa2163b2bab112df6e80b5e9c5795
Merged-In: Id2e92917b50fa2163b2bab112df6e80b5e9c5795

Merge "Change signature of requestBluetoothAudio"

Add an Executor for the onConnectedListener

Test: Run CtsOmapiTestCases
Bug: 76448978
Change-Id: Ieb30b8935961db25bd058e1c41c127fa5a4437ff

Change signature of requestBluetoothAudio

requestBluetoothAudio now takes a BluetoothDevice instead of a string.

Bug: 74408978
Test: unable to CTS -- can't emulate bluetooth devices
Change-Id: Id21e32b911da234f69a248e9937f9df0cf12a9a9

Merge "Refactor isTetheringSupported() calls"

Merge "Adding debug message inside NetworkServiceCallback."

Merge "APF: drop multicast ICMPv6 while in doze mode"

APF: drop multicast ICMPv6 while in doze mode

Some WiFi hotspots send excessive multicast IPv6 ping requests which
cause considerable power drain. Thus, drop all multicast ICMPv6 ping
requests while the device is in doze mode.

Bug: 66930077
Test: runtest -x tests/net/java/android/net/apf/ApfTest.java
Test: manual - compared generated APF bytecode using apf_disassembler
Test: manual - force idle mode while sending multicast pings to device
Change-Id: I25dbf99985cf4413b4d0e3920d281fe6d663197a

Merge "Make changes to MBMS API following recs"

Refactor isTetheringSupported() calls

Tethering currently wants access to complex isTetheringSupported
logic that is only available in ConnectivityService. Instead of
trying to access that via ConnectivityManager, pass this capability
in to Tethering directly, in the TetheringDependencies object.

Also:
    - ConnectivityManager is only a source of static constants now,
      so "import static" all the constants that are actually used.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net works
    - manual USB towards WiFi tethering works
Bug: 68951715

Change-Id: Ia64faaadefb4a5d84a50da98bdebd544b6fda101

Merge "Update Bluetooth codec related strings"

Adding debug message inside NetworkServiceCallback.

Adding debug message inside NetworkServiceCallback when mCallback
is null.

Bug: 76392715
Test: existing test
Change-Id: Icd27d4082e1e4b60efe65abfff99c4165717fd4a
Merged-In: Icd27d4082e1e4b60efe65abfff99c4165717fd4a

Merge "Make OMAPI stricter"

Make OMAPI stricter

- Channel implements java.nio.Channels.Channel.
- openBasicChannel(aid) and openLogicalChannel(aid) is added back.
- Add finalizers for the classes.
- SecureElementListener -> onConnectedListener

Bug: 76448978
Test: CtsOmapiTestCases
Change-Id: I7f92c4dcb5c08a72d32ac9440872ab853badec1d

Make changes to MBMS API following recs

* Change all error handling logic to return an error code via the async
callback
* Add an UNKNOWN code for errors for future backwards compatibility, and
prohibit the middleware from sending this code.
* Add IntDef for errors through the async callback
* Amend documentation for download()
* Implement support for arranging downloaded files into a hierarchy
following that of the server.

Change-Id: I4d5c8f6229b216d9aa84397e628e62279033cc74
Fixes: 76449215
Test: CTS

Merge "Fix socket leaks in various android.net.Network methods."

Update Bluetooth codec related strings

The settings related to Bluetooth codec selection
are actually triggers. Updated the corresponding
strings to reflect that.

Bug: 67682722
Test: Manual - examined the result UI
      runtest systemui
      make -j40 RunSettingsRoboTests
      make -j40 RunSettingsLibRoboTests
Change-Id: I872414f7620931a67b028c2997b0740ae3a45986
Merged-In: I872414f7620931a67b028c2997b0740ae3a45986
(cherry picked from commit a80a65a11fc6fd6aa7f3dfb19516c3d15c512dfb)

Fix socket leaks in various android.net.Network methods.

Bug: 72124526
Test: make CtsNetTestCases
Merged-In: I2a2254ce65c2176ae0be1d6276034b1bb0cad987
Merged-In: I23150482d6c2ab2eaa2b37732af68af69320792b
Change-Id: I3398b67272360f894e01a8cdfbc47b17d77c2330
(cherry picked from commit 89bb993293f0d7a524d1404184a55584b59bb67c)

Merge "apf: adjust JNI to accept_packet()'s new signature"

apf: adjust JNI to accept_packet()'s new signature

Test: Fixes the build breakage
Change-Id: Id5c3cec5459ce6f9693457448002e60bbca6c09b

Merge "Relax APF version check in ApfGenerator"

Merge "Mark networks with OEM_PAID always restricted"

Mark networks with OEM_PAID always restricted

Bug: 68762530
Exempt-From-Owner-Approval: OWNERS have approved, but gerrit doesn't see it
Test: runtest -x frameworks/base/tests/net/ -c android.net.NetworkCapabilitiesTest
Change-Id: Ieadef7c42634d890281543226203530fb18eb0a3

Merge "Limit the number of routes for performance"

Merge "Clarify documentation for getActiveNetworkInfo and friends"

Merge "Disallow Reserved SPI Allocation"

Merge "Require explicitly supplied truncation length"

Merge "Clarify UDP encapsulation socket API"

Relax APF version check in ApfGenerator

Instead of requiring an *exact* version, we now require a minimum
version. This is necessary if we wish to upgrade the APF interpreter in
vendor firmware independently of the framework code. Adding new opcodes
without changing the semantics of the old ones is always possible and
usually easy, so we can at least have a backwards-compatibility horizon
wider than a single version.

Change-Id: I4f1ed661afb2ede193188eb8086fbf2a8b45b33d
Test: Change extracted from aosp/647079 and tested there

Merge "Add the compilation reason to the dexopt dumps"

Merge "Use InterfaceSet for upstream interfaces."

Hearing Aid: change get/set active device (3/3)

* setActiveDevice() returns false in error case, e.g. when the device is
not connected
* add getActiveDevices() instead of isActiveDevice(), which returns a list
that must have two elements: left and right, or empty list on error

Test: manual
Bug: 69623109
Change-Id: I48f3388c56434d0c21e09bd8b794e58848cd8794

Merge "Dummy WorkChain class"

HIDL: add APIs used by subclasses to SystemApi

Two methods:
- HwBinder()
- HwBinder registerService
- HwBinder onTransact

are used by Stub classes generated by hidl-gen.

Bug: 72480743
Test: build hidl-gen generated stubs with system_current
    after manually copying stub jar to prebuilts.
Change-Id: I9b409271b683ad4769a0420e1688b14c561d39d1
Merged-In: I9b409271b683ad4769a0420e1688b14c561d39d1
(cherry picked from commit 4c0d6b62a9d60798f54de33d6cea1626cc21cc4a)

Update HIDL-related documentation.

This is a followup CL to previous @SystemApi CLs.

Bug: N/A
Test: N/A
Change-Id: I9c7dcc776dcfb89fd90afa4fc5d74e40ff0a5f94
Merged-In: I9c7dcc776dcfb89fd90afa4fc5d74e40ff0a5f94
(cherry picked from commit 739811a5c53e435b3f0cfd5abf58389416be429c)

Add remaining missing HIDL SystemApi methods.

A couple methods of Java classes which became SystemApis
but were used are still @hide.

Bug: 72480743
Test: hidl_test_java
Change-Id: I0cb82384932ab1758fa7576b2125825c48a4a6c9
Merged-In: I0cb82384932ab1758fa7576b2125825c48a4a6c9
(cherry picked from commit adcb896fd6e476f277d6f1d20e2e50b8e81c4369)

HIDL: reportSyspropMethod -> enableInstrumentation

Renaming this method with a semantic meaning which
better explains its purpose without caring about
its implementation details.

Bug: 72480743
Test: hidl_test_java
Change-Id: I4b0577b3f61180ecfd66ae146973fa5cc2a3b244
Merged-In: I4b0577b3f61180ecfd66ae146973fa5cc2a3b244
(cherry picked from commit 36be191dfdd220e24f166b96cbdd64cfa30b4eb5)

Allow HIDL to be used in priv-apps.

Marking HIDL as SystemApi to allow vendor apps to talk to their own
native HIDL services. This includes implementing HIDL callback objects.

HIDL (HAL interface design language) is a language that is used to
facilitate vendor<->system communication which provides guarantees
of ABI stability. For system and vendor services alike, this can also
be used for internal communication.

Bug: 67725182
Test: boot device, hidl_test_java
Change-Id: I9d028cc43ce6f81ea619f01e18f1cd772e17cbfa
Merged-In: I9d028cc43ce6f81ea619f01e18f1cd772e17cbfa
(cherry picked from commit e8e63aae6f705cbe316a1d9a7780f9f534bc3948)

Merge "Change shader precision because of CTS fail"

Merge "Fix seccomp filter set up in zygote"

Add the compilation reason to the dexopt dumps

Knowing why we compiled a package will make some investigations easier.

Test: adb shell dumpsys package dexopt
Bug: 76425903
Change-Id: I67b5bc980d198340aa52affb24fb3ce7e3080d67

Change shader precision because of CTS fail

Change-Id: I5d8747ecf9091a251b56d3a9de4bdb8ab0260898
Merged-In: I5d8747ecf9091a251b56d3a9de4bdb8ab0260898

Merge "Move alpha long/short to the base CellIdentity according API Review."

Dummy WorkChain class

To avoid future merge conflicts, a dummy WorkChain class must be
added temporarily. It goes along with the dummy StatsLog.java class.

Bug: 71607284
Test: none
Merged-In: Iff361eb98e079c7b2146c092dc27a3618a813c94
Change-Id: I66cf0c36c1842b99a5a4dd104fc5c06c0cfbe2b5

Merge "tools/bit: Avoid leak in error condition"

Use InterfaceSet for upstream interfaces.

Allows using different upstream interfaces for IPv4 and IPv6.

Bug: 38218697
Bug: 64382985
Bug: 64976379
Bug: 64995262
Bug: 64380515
Test: runtest frameworks-net, manual (with aosp/644099 applied)

Change-Id: I3db63f7aa5255a0229253187def7590a386c5133

Limit the number of routes for performance

In evaluating whether "most" of the addressing space is
covered, the list of routes are obtained from a third-party
app, so it's possbile the system service stalls unless
some limit is enforced on how much work it has to do.
This change limits the number of routes to 400, as determined
by time measurement on various devices.

Bug: 74176086
Test: runtest framework-net
Change-Id: Ie4a96098bc044ade87b188839586f14dd101c100

Clarify documentation for getActiveNetworkInfo and friends

For VPNs it's unobvious what these methods do, so document that.

Bug: 70917007
Test: Comment changes only. Checkdoc passes.
Change-Id: I31ec1aea356d88d969eaf9fc7d8c2ed13ce924eb

Merge "Fixed the Premium SMS bug reported by our partner The detail is that : ===  <shortcode country="ae" free="3214|1017"/> === As displayed above, Arab Emirates is registered in sms_short_codes.xml. There is only CATEGORY_FREE_SHORT_CODE number for the county code "ae". All other numbers which include length under 5 digits is set as CATEGORY_NOT_SHORT_CODE, and so Premium SMS pop-up does not appear for both case. pattern="\\d{1,5}"  is added to all similar entries BUG:66911380"

Merge "DCB whitelist short code for user association"

Merge "Fix network usage stats on 464xlat tethered."

Require explicitly supplied truncation length

Instead of providing default truncation lengths (based on RFC or
otherwise), this change imposes a restriction that the truncation length
must be supplied for all auth or aead algorithms.

Bug: 77204048
Test: Updated tests, ran on walleye
Change-Id: I4a0e2e71aa97259e56f44e7c8a2ce53135708d97

Fix seccomp filter set up in zygote

This needs to land with extra syscalls whitelisted, including capset and
setresuid.  These privileged syscalls are used in the setup after the
filter is initialized.

Test: system starts, different apps run
Bug: 63944145
Bug: 76461821
Change-Id: I49e6b292805f35baffb3530461c8741e75aceb32

Merge "Bluetooth HIDD: Remove unplug()"

Move alpha long/short to the base CellIdentity according API Review.

Bug: 74831568
Test: Telephony sanity
Change-Id: I0ef69e3f18fca111191e1c6329ae13c0860869ba
Merged-In: I0ef69e3f18fca111191e1c6329ae13c0860869ba

Merge "Add additional logging for EMBMS edge cases"

Merge "Address comments from API Review."

Remove duplicate definitions of utility variables

These are set by the core build system, and are becoming read-only.

Bug: 76424357
Test: diff build-aosp_arm.ninja
Change-Id: I2df878bb654db48a0f2b9956220a0367c32c7ffe

Merge "Remove non-tombstoned ANR path."

Bluetooth HIDD: Remove unplug()

Remove the hidden unplug() API because it is not used and it can cause
the remote device to be unpaired. System apps should use unbond() and
user apps should not use reflection to invoke it.

Bug: 67866553
Test: SL4A
Change-Id: I1bdc06dbb5460c9fd52230b78cbf9434a4349d24

Merge "Clean new TelephonyManager API."

Merge "MessageQueue: explicitly remove FD event listeners."

Remove non-tombstoned ANR path.

Bug: http://b/73140330
Test: boots with ANR
Change-Id: Ib58724a0a6b831c05cb59d053520a4169ea591fd
Merged-In: Ib58724a0a6b831c05cb59d053520a4169ea591fd

Merge "Parse SMS locally upon error to get message ref"