Merge changes Ia15f77e2,Icaa1f95c
am: c923752929

Change-Id: I98b29c91c68a63d6ddeffd03a690b7e841c2b5bc

Merge changes Ia15f77e2,Icaa1f95c

* changes:
  Captive portal: regroup hardcoded http response codes.
  Captive portal: rotate fallback urls

Merge "Extract SimChangeListener to its own file"
am: 0ef8df6d52

Change-Id: Ib83aa0acc1cf19c04ed94a198bb8333c7e3a60b8

Merge "Extract SimChangeListener to its own file"

Captive portal: regroup hardcoded http response codes.

This patch regroups various hardcoded http codes into well defined
constants. This reduces risk of errors and makes the captive portal
logic clearer.

This patch also fixes the logging when a captive portal detection
probe fails, to take into account https ssl handshake failures: for
well-behaved portals it is expected that the https probe will fail,
however the error message was written before the introduction of the
https probe and had become ambiguous.

Test: built, flashed, tested manually with various portal networks
Bug: 36532213

Change-Id: Ia15f77e268cb414816fc52f92835289f9a9ce92b

Captive portal: rotate fallback urls

This patch introduces a new settings value to specify more than one url
for the fallback http probe in addition to the existing settings value.
If more than one url exists, a network will rotate urls for the fallback
probe one by one everytime the fallback probe is sent.

Test: built, flashed, tested manually with various portal networks.
Bug: 36532213
Change-Id: Icaa1f95c5914e8840c83ccdf071047358a5b760f

Merge "Ignore potential SystemProperties errors when setting net.dns"
am: f854c9034c

Change-Id: Ic9ac3a2b1162524369f19b876e91a234e56fdc37

Merge "Ignore potential SystemProperties errors when setting net.dns"

Merge "Move some NetworkStats tests to tests/net."
am: ab3901dc8b

Change-Id: I68214c14658afbb5c92facb018988d2ed27bc351

Merge "Move some NetworkStats tests to tests/net."

Ignore potential SystemProperties errors when setting net.dns

Test: as follows
    - built
    - flashed
    - booted
    - "runtest frameworks-net" passes
Bug: 33308258
Bug: 36249702

Change-Id: I76ae853ceb61b7b900f36f9c3cd67b2d2284aab0

Merge changes from topic 'network_specifier'
am: d0f476bdae

Change-Id: I783b2d06f3a5e8b9cd7940f1f1ef0dd951c734f1

Merge changes from topic 'network_specifier'

* changes:
  [CM] Unhide the NetworkSpecifier as object API
  Make the NetworkSpecifier a class instead of a string.

Merge "Bluetooth 5 PHY simplification"
am: a9acf9db97

Change-Id: Ibd712a7629d946c24640da0e07496c229c35f7ad

Merge "Bluetooth 5 PHY simplification"

Merge "ECBM SystemProperty cleanup"
am: 4718606368

Change-Id: Ie6b43fd0a0bfa7c813960d1bb53b7b57a4f82721

Merge "ECBM SystemProperty cleanup"

Merge changes from topic 'ipsec-api-tweaks'
am: fddf677b8b

Change-Id: I0c0812164e8fa4b36fd43f00b7388d4c0cc337f1

IpSecManager and IpSecAlgorithm API Tweaks
am: 6045429b35

Change-Id: I93d58f64834e5a4c7bc1bf03a5baf2eb5364a36d

Merge changes from topic 'ipsec-api-tweaks'

* changes:
  Add FileDescriptor Versions of applyTransportModeTransform()
  IpSecManager and IpSecAlgorithm API Tweaks

Bluetooth 5 PHY simplification

Having PHY_LE_* constants defined in four different places, with one
value being different than others is misleading. Leave just PHY_LE_*
definitions in BluetoothDevice, and add PHY_LE*_MASK for the mask used
in PHY update API.

This patch also removes need to translate PHY value between PHY update
request and event, as mask is used for request, and the value is
returned in event.

Bug: 30622771
Test: manual
Change-Id: I897effa1204a024465d55501c83c542566c4d37c

Move some NetworkStats tests to tests/net.

This groups them together with the rest of the networking unit
tests. It also speeds up compile/test cycles ("runtest -x" of one
file goes from 1m15s to 30s).

Test: runtest frameworks-net passes on internal tree
Change-Id: I53cb0c51355fe4b4b30e451fa09fbbf58da39efd

Merge "Adding 20 free short codes for DCB and other purposes"
am: 70fea2df2a

Change-Id: Ic63e088d49796b6e98dc92bc3b0b261958d73e8c

Merge "Adding 20 free short codes for DCB and other purposes"

Merge "Captive portal: fix probe urls at network creation"
am: 923ef69ae7

Change-Id: I82eb06ee1f9d69a1a3ccb5208947bdfcdbba6d81

Merge "Captive portal: fix probe urls at network creation"

Extract SimChangeListener to its own file

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - runtest frameworks-net passes
    - runtest -x full/path/to/SimChangeListenerTest.java passes
Bug: 32163131
Change-Id: If90539fe9a5dc07c7c24ed0b70b920d94065a623

Merge "Fix some maps to be HEAP_DALVIK_CODE_CACHE instead of HEAP_DALVIK_ACCOUNTING"
am: d269f30d62

Change-Id: I2830356fd9288feef334e19a1281c720c6fd7fca

Merge "Fix some maps to be HEAP_DALVIK_CODE_CACHE instead of HEAP_DALVIK_ACCOUNTING"

Merge "jni: isLoggable: adapt to removal of property name size limit"
am: cf39eea407

Change-Id: I225add287a014fa8e2623d1e61456a5c523d5377

Merge "jni: isLoggable: adapt to removal of property name size limit"

ECBM SystemProperty cleanup

- Add getEmergencyCallbackMode() method in the TelephonyManager.java
to get the boolean value for ECBM callback mode from the phone through
ITelephony.aidl/java interface

- Use the added TelephonyManager Api to replace the 'get' of
PROPERTY_INECM_MODE system property in the framework/base

Test: manual
Bug: 30361624

Change-Id: I355d69820b157f23e077a95a13f8509ee0fa5874

Merge "Add p2 option in openLogicalChannel."
am: f954f2d5cc

Change-Id: I3437793f39ba6747554fec383ac7ab9f88df9266

Merge "Add p2 option in openLogicalChannel."

Fix some maps to be HEAP_DALVIK_CODE_CACHE instead of HEAP_DALVIK_ACCOUNTING

dalvik-data-code-cache and dalvik-CompilerMetadata should be counted
in JITCache instead of .GC in dumpsys.

Bug: 37224159
Test: adb shell dumpsys meminfo -d

(cherry picked from commit 874c4cf56c0a9ea3b48468a13ec7fb78a3e2594b)

Change-Id: I41def949d91b2fdef0b3f502fe16ae436d058051

Add FileDescriptor Versions of applyTransportModeTransform()

Because there is no way using the Java sockets API to actually
get a socket of AF_INET on mode machines, it is necessary to
provide a way to apply transforms to sockets made using the
native wrapper API, which uses POSIX APIs and will create a
socket that is AF_INET.

Bug: 36073210
Test: b/34811227
Change-Id: I28ac7cc4f36045ce523a54111e5be975b0331356

IpSecManager and IpSecAlgorithm API Tweaks

-Add a reserveSecurityParamterIndex() function that allows the
 system to select an SPI.
-Disallow INVALID_SECURITY_PARAMETER_INDEX from being passed as
 an explicit SPI request.
-Remove the ALGO_ prefix from constants in IpSecAlgorithm

Bug: 36073210
Test: Updated CTS tests still pass on bullhead
Change-Id: Ic94809996076b0718f153f550b82192fe7048a2e

Merge "Remove .(ll)ndk suffix from (ll)ndk_library"
am: 4a43598bd6

Change-Id: Ib66ddf53cbe91d2064f6268f9115cd577883acae

Merge "Remove .(ll)ndk suffix from (ll)ndk_library"

Merge "OtaDexOptService: Log post-reboot moves"
am: ac293515b3

Change-Id: I76ded23a21610405079f275a61e1b61c23b46de7

Merge "OtaDexOptService: Log post-reboot moves"

Remove .(ll)ndk suffix from (ll)ndk_library

Soong handles these automatically now.

Bug: 33241851
Test: Android-aosp_arm.mk is the same before/after
Test: build.ninja is the same before/after
Test: build-aosp_arm.ninja is the same before/after
Merged-In: Ic10796e75bb886cc3f92c234714ab3e6c85112df
Change-Id: Ic10796e75bb886cc3f92c234714ab3e6c85112df

jni: isLoggable: adapt to removal of property name size limit

There is no longer a name size limit to the properties, remove
illegalArgumentException if tag length is too large.

Test: build
Bug: 36696208
Change-Id: I4b4329c8c951082ed0d777cdd70ee3e773bed16c

Merge "Remove enable from PeriodicAdvertisingPariameters (1/2)"
am: 763765b1b1

Change-Id: Idb0c93f1f37b3fc4bd44bc4ad7d5474fd46e7413

Merge "Remove enable from PeriodicAdvertisingPariameters (1/2)"

Remove enable from PeriodicAdvertisingPariameters (1/2)

Instead of setting enable to true, one can just pass null
PeriodicAdvertisingParameters and achieve same result when starting the
set.
Passing the "enable" when updating the parameters make no sense, and
might be confusing.
Experience with "timeout" field, which was a part of AdvertiseSettings
show that merging fields that go into different HCI commands can cause
problems during processing, so keep enable as separate field.

Test: manual
Bug: 30622771
Change-Id: Ida02c59eb8433537179b4d22202fe745f8b4bb3e

Captive portal: fix probe urls at network creation

This patch changes how url configuration values for captive portal
detection are read: instead of re-reading the settings at every probe
detection, the settings are read once when the NetworkMonitor associated
to a network is created.

If the settings are updated, the new values are picked up by new
networks, but not by existing networks. Since captive portal detection
is most important when joining WiFi networks, the newest available
settings values will still be used.

This change prepares introducing further changes for rotating probe
urls. Especially it helps making the detection strategy stateful with
respect to the lifecycle of a single NetworkMonitor.

Test: built, flashed, tested manually with various portal networks
Bug: 36532213
Change-Id: I71cc8bb8b996462f27f50798d67bceee5ffb898d

Merge "Deprecate TelephonyManager.getCellLocation()"
am: bc0f610b25

Change-Id: Ic6f55f3f00b608f3b8c3212b0dea1114250ec1fe

Merge "Add developer setting and system API for inband ringing support"
am: 1da38b2bcd

Change-Id: I24f6eccb3a3dcc37fd21ee48d8844bd5a1b112f0

Merge "Deprecate TelephonyManager.getCellLocation()"

OtaDexOptService: Log post-reboot moves

Disable artifact checks for non-upgrade booting. Log the number of
successfully move artifacts.

Bug: 37158297
Test: m
Test: manual
Change-Id: Ic81531950582ce937e2e3b7815fb9132888d9a42

Merge "Add developer setting and system API for inband ringing support"

Merge "Add getSelfManagedPhoneAccounts() TelecomManager API."
am: c4ea961f91

Change-Id: I77e7353fe345fbf17f9499aea9e792a23e8725d4

Merge "Add getSelfManagedPhoneAccounts() TelecomManager API."

Add getSelfManagedPhoneAccounts() TelecomManager API.

This is consistent with the existing getCallCapablePhoneAccounts API
which is used to get the phone accounts capable of placing phone calls.

Test: CTS
Bug: 34159263
Merged-In: I1396c63a6882947cd66721528e53b6964a9091e0
Change-Id: I1396c63a6882947cd66721528e53b6964a9091e0

Add developer setting and system API for inband ringing support

* Add 1 system API to check inband ringing flag in config.xml
static isInbandRingingSupported(Context)
* Add developer menu options to enable this feature

Bug: 19171297
Test: mm -j 40, HFP regression testing, testplans/82144
Change-Id: Iaf56ea41911f546bbc7ae1f82e399d0f8d48f75f

Merge "Tethering: support Local-only Hotspot mode for downstreams"
am: d3128d4b55

Change-Id: Ic7d5e3e23f9ccc0f094ba5991e2fdefa82865b01

Merge "Tethering: support Local-only Hotspot mode for downstreams"

Tethering: support Local-only Hotspot mode for downstreams

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - "runtest frameworks-net" passes
Bug: 31466854

Change-Id: Ia50e28c8ce0af8cdd7ac63217d921aff213668e7

Merge "Add an API hint for metered multipath traffic."
am: 7a12f7efc7

Change-Id: Ie0d289460f93d3b93d91545eb3dfa925f0bdee5f

Merge "DO NOT MERGE IP connectivity metrics: fix tests."
am: b2adaad26b  -s ours

Change-Id: I47dbd0feaa99ff6018efc8703cfa29a265bc73fd

Merge "Add an API hint for metered multipath traffic."

Merge "DO NOT MERGE IP connectivity metrics: fix tests."

DO NOT MERGE IP connectivity metrics: fix tests.

This patch fixes mismatch between current proto compiler and what is
expected in several IP connectivity metrics unit tests.

Test: built, flashed, $ runtest frameworks-net
Bug: 37126521
Change-Id: I1476c923d61f748ecac1146423127f54ba117502

Merge "Improve utility of IntArray and LongArray"
am: 35786735df

Change-Id: Id50add8d7e3fc45978752a6a31760bc69df0cc4a

Merge "Improve utility of IntArray and LongArray"

Deprecate TelephonyManager.getCellLocation()

This API has been long-marked for deprecation. It has numerous problems:
-It does not support LTE
-It returns a limited subset of information
-It uses old (and poorly-chosen) sentinel values for fields that are
 not provided.

Bug: 36076138
Test: compilation
Change-Id: I87004710e5d046d4534ad3a0a938540440b0e83b

Merge "Bluetooth 5 fix default AdvertisingSetParameters values"
am: 7bf58d5975

Change-Id: Idd2aad81fa1d6cc39d509c1556e8d1f7eb24a3b2

Merge "Bluetooth 5 fix default AdvertisingSetParameters values"

Merge "Add NETWORK_STACK permission"
am: b23fabf3fd

Change-Id: I20f0278e080ccc957ce6342581f3d0f18dfb4b9a

Merge "Add NETWORK_STACK permission"

Improve utility of IntArray and LongArray

This patch adds several utility methods to IntArray and LongArray to
make them more useful in the context of a Parcelable class.

More specifically, it is now possible to parcel and unparcel a
{Int, Long}Array with the following one liners:
  - mIntArray = IntArray.wrap(parcel.createIntArray());
  - parcel.writeIntArray(mIntArray.toArray());

This patch also
  - adds unit tests for IntArray and LongArray.
  - adds a method for setting a field at a given index.
  - adds a method for changing the effective array length.
  - adds missing bound checks for negative index values.

Test: added test coverage
Bug: 34901696
Change-Id: Ia0febf4dae048b0a45c75310b5d90b56499320da

Add NETWORK_STACK permission

Add a new permission that only allows other system services to call
networking related methods.  This permission is intended to prevent
other apps from calling these methods that might leave networking
services in intermediate states.

Bug: 36538042
Test: build, run wifi unit tests, boots, wifi connects
Change-Id: Icba6ad5280e9a8863bd1c706ca34b7add8de5294

Merge "NetworkMonitor: improve captive portal validation logs"
am: d6933a6c14

Change-Id: I8f0fd26d6a9f036adc9cc895690aac27d4b8e9cd

Merge "NetworkMonitor: improve captive portal validation logs"

Merge "ConnectivityService: safer locking"
am: b8dc7fbc6d

Change-Id: I88a118bb59f8a870d42d2d665f838f5236ebd21c

Merge "ConnectivityService: safer locking"

Merge "Use hwservicemanager getTransport."
am: 562ed84ff4

Change-Id: I39ee473166dce391f3cad4acf62821a4dfb9f444

Merge "Use hwservicemanager getTransport."

NetworkMonitor: improve captive portal validation logs

This patch adds probe type and url to all validation logs about web
probes sent for captive portal detection.

Test: built, flashed, looked at $ adb shell dumpsys connectivity
Bug: 36830206, 36532213
Change-Id: Iee2caaf6664f3d097f9d1888ccc5fece0a95994c

Use hwservicemanager getTransport.

libhidl no longer provides a getTransport function. Now, call into the
hwservicemanager which directly interfaces with libvintf.

Test: extensive, see Ia5d1eb41b057ab5d6800f6c3fd22658adecc4be7
Bug: 36377072

Merged-In: I8b0ca845251cd7cd156f3471cbd4b0ce17617be0
Change-Id: I8b0ca845251cd7cd156f3471cbd4b0ce17617be0
(cherry picked from commit f8202e464e09618c2b780d331541f32cc186598e)

Merge "Add RecommendationPlugin for Cloud Print"
am: b2c75d062d

Change-Id: I4b018ac9763d9bf5c69d055b4a9d5ddf6dded3cb

Merge "Add RecommendationPlugin for Cloud Print"

Merge "Add KEY_ENABLE_APPS_STRING_ARRAY carrier config key."
am: ded8a6b7f6

Change-Id: I12f28266088cd9565d62097477c69dcb5ece54e2

Merge "Add KEY_ENABLE_APPS_STRING_ARRAY carrier config key."

Bluetooth 5 fix default AdvertisingSetParameters values

Having advertiser be scannable and connectable by default is a bad
choice for new advertising set.

Bug: 30622771
Test: sl4a Bt5ScanTest
Change-Id: I4fc270e78ca4e62d3077c5cd28aa59b0518d2e77

Merge "Update layoutlib to match java library changes"
am: ac6414524c

Change-Id: If673886a433d1c02ac19646ab8128ad1b36280c8

Merge "Update layoutlib to match java library changes"

Merge "Track libcore change dfd104e99d15d91dc21137b190fb99cab8a1180b"
am: 443af115ab

Change-Id: Id3eed59b96b9aba2f101bb3910f7eace84a6699e

Merge "Track libcore change dfd104e99d15d91dc21137b190fb99cab8a1180b"

Merge "Canonicalize tz IDs when finding exemplar location"
am: 5e603a4b6e

Change-Id: Ib4eb5ce215ec2564d220e1658f2dec4d0dc86d67

Merge "Canonicalize tz IDs when finding exemplar location"

Track libcore change dfd104e99d15d91dc21137b190fb99cab8a1180b

Test: make -j 32
Bug: 36461944
Change-Id: I2075d03c3143d0582a76d91bcbaccd83c193a552

ConnectivityService: safer locking

This path changes a dangerous lock path in reportNetworkConnectivity().
This methods is called outside of the main ConnectivityService handler
and takes a lock on a specific NetworkAgentInfo whose connectivity
status is being reported.

While this lock is held, reportNetworkConnectivity() goes on and query
the network policy state for that network, which may ends into
NetworkPolicyManagerService.

Instead, the lock on NetworkAgentInfo is only held long enough to make a
copy of LinkProperties, which is then passed to
NetworkPolicyManagerService without that lock.

Bug: 36902662
Test: could not repro b/36902662, reportNetworkConnectivity() works.
      $ runtest frameworks-net

Change-Id: Iac4b75bcecbdddb0ac695c8b1a87ae755f62f47f

Add an API hint for metered multipath traffic.

This allows an application that knows how to provide seamless
network connectivity (e.g., using QUIC multipath) to find out if
doing so is desired.

(cherry picked from commit 2de4925f5cc64aeb92e02a8f740d3ff20f36dddd)

Test: builds, boots, runtest frameworks-net passes.
Bug: 34630278
Change-Id: Ic7fd0b9e1cd879fdfaf84009d7125391895e9087

[CM] Unhide the NetworkSpecifier as object API

API visibility change: unhide allowing NetworkSpecifier
to be an arbitrary object.

Bug: 27533960
Bug: 36053921
Bug: 36275276
Test: builds and runs
Change-Id: I1d1705cca7ece077ef8d7c674c62d5369fedbb03

Add RecommendationPlugin for Cloud Print

Reuses already existing infrastructure. The mMDNSFilteredDiscovery lambda
is called for every network service that supports #PRIVET_SERVICE. Then
the plugin checks the txt fields to make sure the printer is valid. The
check is not complete but good enough to make sure this is either a
cloud print capable printer or something that tries really hard.

Test: Connected to network with three printers, 2 GCP capable. Found the
      two printers
Fixes: 35766193
Change-Id: I7c9180c8c154fa092fec5b943a94bad77da74c86

Merge changes I9ad4ce81,I360d539e
am: 9680667b1d

Change-Id: I7c64e3db6a1d153339f50db292761131dbfa2ae8

Merge changes I9ad4ce81,I360d539e

* changes:
  NsdService: minor cleanups
  Nsdmanager/NsdService: add logging

Make the NetworkSpecifier a class instead of a string.

Bug: 27533960
Bug: 36053921
Bug: 36275276
Test: connectivity, wifi, telephony unit tests
Change-Id: Idd9b10a8418c53c8cf386d9ff8252226b076bbf9