[CS] Unregister callback as part of onUnavailable dispatch

The onUnavailable semantics promise that it is equivalent to calling
the unregister callback method. But - it doesn't unregister the callback
allowing it to be reused. Fixed.

Additionally, modified the unregisterNetworkCallback method to not fail
on duplicate unregistration (since a callback could now self
unregister). Instead simply print a log.

Bug: 130651445
Test: atest ConnectivityServiceTest
Change-Id: I4c54b003a733eb0b1e4fd8674ed13081b1bef8e3

Merge "Update to follow the new google benchmark API"

Merge "Add IPsec tunnel mode feature flag"

Update to follow the new google benchmark API

Test: build
Change-Id: Icc25b81e9b863c35c02c9673518fa4d30e9ae321

Add IPsec tunnel mode feature flag

This patch adds a feature flag for IPsec Tunnel Mode. This implies VTI
(with output-mark updating), or XFRM-I in the kernels.

Bug: 117183273
Test: Compiles
Change-Id: I6dd0e429cc0bd100f2ef1140a6651f6ef5294c79
Merged-In: I6dd0e429cc0bd100f2ef1140a6651f6ef5294c79
(cherry picked from commit dc92c69b087d5fbecff76ef1e739ae415312734d)

Merge "Extend NetworkDnsEventReported and add new DnsCallReported to log per-query information"

Merge "Update getCardId doc with details"

Merge "Fix incorrect parsing of UVC interface descriptor"

Merge "Sorting and merge Emergency number better"

Sorting and merge Emergency number better

Apply more parameters to sort Emergency number list, it is important
to keep two lists Collection-equal to each other if they have the same
contents. Otherwise, it will introduce more unnecessary updates in
 EmergencyNumberTracker and it may confuse users.

Test: atest EmergencyNumberTest
Bug: 131922424
Change-Id: I64a43886d4000b261ea703b081e6278b0307805f
Merged-In: I64a43886d4000b261ea703b081e6278b0307805f
(cherry picked from commit a1f21a5c2cd4cdc14f4cbf7cd0fc0de6e4911c02)

Merge "Update path to audio files"

Merge "Track libziparchive API change."

Update getCardId doc with details

getCardId() returns the EID in some cases and ICCID in others. Add
comments explaining exactly when it returns what.

Test: no change in behavior
Bug: 131423779
Change-Id: Id0cbae3affdf4c034b2435bf77371ae1355d3cfa

Merge "Rollback SHELL UID check for CarrierConfigLoader"

Rollback SHELL UID check for CarrierConfigLoader

When a test calls adoptShellPermissionIdentity before
calling a command, it doesn't adopt the shell UID, only
the permissions. So, we can not use
Binder#getCallingIdentity to determine it the API is
being called as part of a test. For now, we must only
use MODIFY_PHONE_STATE until a better pattern is
established.

Test: Manual
Bug: 131185660
Merged-In: I607b7c9afe1c384a41ce12b92ae43747b2ee1bc3
Change-Id: I59d32a2e36950b2485e1a46a25e3724928676219

Merge "Fix binder leakage when turning off Bluetooth"

Track libziparchive API change.

Bug: http://b/129068177
Test: treehugger
Change-Id: I88f49a06db416a7c6ec8afe87cc9cca825eb5ccb

Merge "Whitelist 30075 for Switzerland"

Merge "Remove JapaneseCalendar.CURRENT_ERA from Android public SDK"

Merge "Convert packages/Shell/**/Android.mk file to Android.bp"

Update path to audio files

The path to these audio files was changed in r.android.com/862589.
Update the configuration to read from the new path, too.

Bug: 131864467
Test: hear sound when locking/unlocking aosp_blueline
Change-Id: Ie0d69de6333a08d3e645829e13a707f83cb05e01
Merged-In: Ie0d69de6333a08d3e645829e13a707f83cb05e01

Remove JapaneseCalendar.CURRENT_ERA from Android public SDK

JapaneseCalendar.CURRENT_ERA is final static int, and the developer
may mis-use and could have wrong expectation on this API, since
1. New emperor in Japan could result in a new era, but old Android device
   still provides an outdated "current" era.
2. Developer may expect its value depending on system time. However,
   given it's a static final int,
   Android could intiailize the value at boot with an incorrect time,
   and thus results in incorrect era. Thus, on Android, The value is
   always hard-coded.

Bug: 131893534
Test: make
Change-Id: I16ea841975dd8e913409c884123003f842c55bb8

Extend NetworkDnsEventReported and add new DnsCallReported to log per-query information

NetworkDnsEventReported -The message represents the entire lookup operation,
which may result one or more queries to the recursive DNS resolvers.

DnsCallReported - Those DNS queries are individually logged in DnsCallReported
to enable computing error rates and network latency and timeouts broken up by
query type, transport, network interface, etc

Test: make statsd_testdrive
     ./out/host/linux-x86/bin/statsd_testdrive 116 => pass
Bug: 129253665

Change-Id: Ib75affea5d849437a293affbe294732153511d97
Merged-In: Ib75affea5d849437a293affbe294732153511d97

Convert packages/Shell/**/Android.mk file to Android.bp

See build/soong/README.md for more information.

Bug: 122332340
Test: atest ShellTests
(same errors as baseline)
Change-Id: I63b777342359e0b42ccd554b7d056c7431178380

Merge "Pass in external packageName"

Pass in external packageName

Use external packageName when ActivityThread couldn't get calling package
due to calling identity cleaned.

Bug: 131287254
Test: Manual
Change-Id: Icec3ad7193bc5988799188a7a1e68aebc0dc109d

Merge "Add calling package parameter to IRcs methods"

Merge "Move to libunwindstack from libunwind."

Merge "Added riddlehsu@, louischang@, and erosky@ to WM OWNERS file"

Move to libunwindstack from libunwind.

This already matches internal master.

Test: Builds.
Change-Id: I1f9b886668569dd03a2a073f6d79f4fb959a2d32

Merge "AddRestrictCause"

Added riddlehsu@, louischang@, and erosky@ to WM OWNERS file

With great power comes great responsibility.

Test: none
Bug: none
Change-Id: Ibea2b226bac196bb340395426f0abdacbcff9495

Merge "Inline calls to MimeUtils."

Merge "Use dev/null instead of tmp file"

Fix binder leakage when turning off Bluetooth

* In current design, Bluetooth AdapterState stops all BR/EDR
  profiles' service and triggers onServiceDisconnected callback to
  all binder clients before BluetoothManagerService invokes
  onBluetoothStateChange(false), which means unbind service
  would never be called in framework.
* Do unbind service when onServiceDisconnected is invoked.
* Move profile binder logic to BluetoothProfileConnector except:
  - BluetoothHeadset: its binder logic is in BluetoothManagerService
  - BluetoothPbap: it has an individual ServiceListener

Bug: 129037442
Bug: 129437895
Test: Bluetooth ON/OFF stress test.
      adb shell dumpsys activity services | egrep "com.android.bluetooth"
      to check whether AppBindRecord for com.android.bluetooth grows

Merged-In: Id0d85866d386962b94d2d966f0a864b1da165d13
Change-Id: Id0d85866d386962b94d2d966f0a864b1da165d13

Merge "Freeze networkstack-aidl-interfaces as version 3."

Merge "Correct buffer size limitation for DnsResolver API"

Correct buffer size limitation for DnsResolver API

Bug: 131055651
Test: atest DnsResolverTest
Change-Id: I29decd480172eacb452ccb0da524ef1bf2ffadcf

Merge "Add CTS API coverage for APIs which are used by NetworkStack"

Merge "Update getUiccCardsInfo doc"

Merge "Add EuiccManager#EXTRA_PHYSICAL_SLOT_ID to detect it's an operation for pSIM."

Add EuiccManager#EXTRA_PHYSICAL_SLOT_ID to detect it's an operation for pSIM.

Platform needs to pass LPA the pSIM slotId so that LPA knows it's a
toggle action on pSIM.

Bug: 123945155
Test: build
Change-Id: Ie0db50e8ade4c6ce385ebfbfe7cf9e2982d67e34
Merged-In: Ie0db50e8ade4c6ce385ebfbfe7cf9e2982d67e34

Update getUiccCardsInfo doc

This method will return info for inactive eUICCs, filtered by carrier
privilege. (Since we can't read carrier privilege on an inactive eUICC,
we always assume that the caller does not have carrier privilege).

Bug: 130826215
Test: manual
Change-Id: Id89fa48f119004b6c18bd21791396bc6c3df2c95
Merged-In: Id89fa48f119004b6c18bd21791396bc6c3df2c95

Merge "Support IPv6 Bluetooth tethering"

Merge "Make NetworkStack actually build in an unbundled setup."

Merge "hiddenapi: Support greylist-max-q"

Make NetworkStack actually build in an unbundled setup.

The Java code was already using sdk_version: "current", but the
JNI library did not have an sdk_version attribute, and had
forbidden dependencies on internal libraries such as libcutils,
liblog and libnativehelper.

This CL fixes things as follows:
- Use the NDK log functions, and switch to the NDK liblog.
- Switch to the NDK libnativehelper.
- Drop the dependencies on libcutils and libpcap, which were
  unused.
- Use the libc++ STL, and set the sdk_version of the library to
  current.

Bug: 131703417
Test: m
Test: installed NetworkStack APK via "adb install --staged" results in working networking
Test: builds, boots, wifi works
Test: atest FrameworksNetTests NetworkStackTests

(cherry picked from commit 3d680e887ba40cde436a5745578e67d4a6a6fce9)

Change-Id: I8a0a8d4b4e8872cb88ee526772ae519a8e6df615

Support IPv6 Bluetooth tethering

Test: build
Fix: 130843211
Change-Id: I6dac50b39aabad1fd907325090af6aaf53d68c67

hiddenapi: Support greylist-max-q

Test: set maxTargetSdk=Q on an API, compiles, check hiddenapi-flags.csv
Change-Id: I990972d652d4f4bc578592a9b3d9954f93b1ab9f

Freeze networkstack-aidl-interfaces as version 3.

IIpClient#addNattKeepalivePacketFilter is the only difference
between version 2 and 3.

Generated with: m networkstack-aidl-interfaces-freeze-api

Test: -builds, boots
      - atest NetworkStackTests
      - atest ConnectivityServiceTest
Bug: 128803828

Change-Id: I22333c3c5930415cc43d83bc8cc6f566d0acb1f2
Merged-In: I22333c3c5930415cc43d83bc8cc6f566d0acb1f2
(cherry picked from commit a9690c1f66c63ce4dd22f622cc9980d89ad40d20)

Merge "HwBinder: call _exit over exit."

Merge "Fix tests/runtests.sh"

AddRestrictCause

Bug: 130744052
Test: Build
Change-Id: I51c49b8736c42276f7fd66168eeb3f9959b4d492

Merge "Add PhoneAccountHandle method to check if 2 accts are from same CS."

Whitelist 30075 for Switzerland

This number is a visual voicemail SMS number for Swisscom.

Bug: 120403639
Test: N/A
Change-Id: I8d8d6413b7f9869c13605e5175e4a20d48b3a23d

Add CTS API coverage for APIs which are used by NetworkStack

Change-Id: I4f6c34f3bcfa65e60cf73fc1b7cab2d83c32836a
Bug: 129200003
Bug: 129200292
Bug: 129199996
Bug: 129200261
Bug: 129200513
Bug: 129200316
Bug: 129200589
Bug: 129200181
Bug: 129200669
Bug: 129200504
Bug: 129200478
Test: atest FrameworksNetTests
Test: atest CtsNetTestCases: added tests pass

Merge "Update team members in the OWNERS file. DO NOT MERGE"

Add calling package parameter to IRcs methods

This will be used for permission handling in the IRcs service.

Test: Existing tests pass, both unit and RcsCtsTestCases
Change-Id: I7f2183cd2f764b8293d29dc2c278479e06d9d8a2

Merge "Add EPROTONOSUPPORT to IpSecManager SSE map"

HwBinder: call _exit over exit.

Avoid calls to static destructors.

Fixes: 131589658
Test: TH
Change-Id: I52ddcda75e902965f8d4a13f4851e6bdcbe4b0e5

Merge "Support adding NATT keepalive packet filter"

Merge "Add Network, NetworkCapabilities, StaticIpConfiguration common test cases"

Merge "Freeze {ipmemorystore,networkstack}-aidl-interfaces as version 2."

Support adding NATT keepalive packet filter

Support adding NATT keepalive packet filter to APF
filter.
Generating APF program will be addressed in another CL.

Bug: 33530442
Test: - atest NetworkStackTests
      - atest FrameworksNetTests

Change-Id: I4961d5da343d8700600269632787c28112f0f9f1

Update team members in the OWNERS file. DO NOT MERGE

Change-Id: I7f23507e1ec9a5763579160f7942bf2f55df1bfb
Test: build/make/tools/checkowners.py -c -v OWNERS
Fixes: 129861520

Freeze {ipmemorystore,networkstack}-aidl-interfaces as version 2.

Generated with:
m networkstack-aidl-interfaces-freeze-api \
    ipmemorystore-aidl-interfaces-freeze-api

Test: builds, boots
Bug: 128803828
Change-Id: I079018679766738a97b827bc87a6903655f308dd

Merge "Add AIDL parcelable for NattKeepalivePacketData"

Add Network, NetworkCapabilities, StaticIpConfiguration common test cases

Bug: 129200415
Bug: 129200142
Bug: 129200418
Test: atest FrameworksNetTests
Test: atest CtsNetTestCases: added tests pass
Change-Id: I568d516e1ffc9e32061ca6860224c102bdfd7445

Merge "Revert "Make the network stack a persistent process.""

Merge changes Ia9865c75,I47d9d53d,Idbe10c36

* changes:
  Fix some checkstyle nits in DhcpPacketTest.
  Additional code for server name in DHCP packets.
  Parse the server host name field of the dhcp package

Revert "Make the network stack a persistent process."

This reverts commit 796afddd6c7012dc7f10bc2557eb06767c5d873a.

Reason for revert: breaks updatability because persistent processes are not updatable.

Change-Id: I35ded113b31e742f13ad8523a10af3a83a111374

Fix some checkstyle nits in DhcpPacketTest.

Use the Generated code suppression comment filter instead of the
IndentationCheck suppression comment filter because the latter
complains that '+' should be on a new line.

Bug: 120584519
Bug: 127423755
Test: atest NetworkStackTests
Test: atest FrameworksNetTests
Change-Id: Ia9865c75f940e2fbb3aef374a297339e8ea2fd1b

Additional code for server name in DHCP packets.

- Add the hostname to DhcpResultsParcelable.
- Don't store the server hostname if option overload is in use,
  as this is not valid.
- Add unit tests.

Bug: 120584519
Bug: 127423755
Test: atest NetworkStackTests
Test: atest FrameworksNetTests
Change-Id: I47d9d53d1fb58968322cc8b071a44fbc8f7156e1

Fix tests/runtests.sh

The location of FrameworksWifiTests.apk installed by the script
with adb seems to have changed. The path of the APK contains the
TARGET_ARCH environment variable, which is now fetched
with a similar method as in system/netd/tests/runtests.sh.

Noticed the issue with "lunch aosp_x86_64-eng". Before this fix,
the test script produced an error:

    adb: failed to stat [...]/FrameworksWifiApiTests.apk:
    No such file or directory

Test: ./frameworks/base/wifi/tests/runtests.sh

Change-Id: Id4a33ca89fa21a9a4ef791c5c1c5760e0e2badaf
Merged-In: Id4a33ca89fa21a9a4ef791c5c1c5760e0e2badaf
Signed-off-by: Otto Seiskari <otto.seiskari@indooratlas.com>

Inline calls to MimeUtils.

A separate CL in this topic adds MimeMap to the @CorePlatformApi and
deletes MimeUtils. This CL inlines calls to MimeUtils to call MimeMap
instead.

Bug: 120171727
Test: Treehugger
Change-Id: Ibad1e62a63342fa7527332d2b156b69517956364

Parse the server host name field of the dhcp package

Some hotspot devices will fill in the server host name field of the DHCP
package, such as iphone. Parsing the server host name of the DHCP
package can help identify ios hotspots.

Bug: 120584519
Test: 127423755
Test: builds, boots, wifi works
Change-Id: Idbe10c36332ce421f1bba93eb87e9e1e12463088

Merge "Convert misc tests Android.mk files to Android.bp"

Merge "Add validation type definition to support multiple probe result"

Merge "Make DNS cache lifecycle management explicit"

Merge "Fixed the comment of carrier config for CDNR"

Fixed the comment of carrier config for CDNR

Bug: 122921456
Test: build
Merged-In: If5cc8d37ba1e66d59e5bfe9444fe4108f2263be9
Change-Id: If5cc8d37ba1e66d59e5bfe9444fe4108f2263be9
(cherry picked from commit 7ac9a8f314ad8be530f6745a947b90bfcff9495c)

Add validation type definition to support multiple probe result

In order to allow NetworkMonitor to tell ConnectivityService
if the network is either parital, validated or both.
NetworkMonitor could update the probe result by passing the
bitmask to ConnectivityService.

Bug: 130078798
Bug: 130683832
Test: build
Change-Id: I429ccbd9b9afae3eafaf73b3c9eb835213c1f64d

Merge "Remove Settings constants for NetworkStack"

Merge "Update exception type thrown when NetworkStack is missing"

Remove Settings constants for NetworkStack

The definitions reference of these constants are moved from
Settings to DeviceConfig. These definitions are @hide and the
usage in the Settings are removed. Thus, remove the definitions
from API level.

Bug: 123167629
Test: make system-api-stubs-docs-update-current-api \
      test-api-stubs-docs-update-current-api
Test: atest NetworkStackTests
Change-Id: I28c4bf2c0b72e154cea91d11007a9497c7f21570
Merged-In: I7291cfa86cd6e907df35e8fbc97e2e9a77e56cab
Merged-In: Ic48bea07e79490dd3787068bc88c748525752a61

Merge "Add privapp permissions to network modules."

Merge "Use injected instance of RcsControllerCall"

Merge "Do not populate SLOT_KEY with wrong slot info"

Add EPROTONOSUPPORT to IpSecManager SSE map

This change maps EPROTONOSUPPORT to the list of error codes that map to
UnsupportedOperationException in IpSecManager.

Bug: 80103456
Test: Compiles, CTS tests ran
Change-Id: Iec3d5fc4a9bcad7c104414afefae775232d46558

Do not populate SLOT_KEY with wrong slot info

The slotId is not necessarily the phoneId, so do not broadcast as if it
is. SLOT_KEY is a hidden extra so this should not impact apps in general
(if they need the old extra they can still use the hidden PHONE_KEY).

For select broadcasts like SIM_CARD_STATE_CHANGED, we will manually
populate the slot key. This is a temporary solution to b/130664115 and
we should eventually review and consider making the SLOT_KEY public for
some broadcasts.

Bug: 130664115
Test: manual
Change-Id: I84bd4c6bc4277e32cee9916dd18dd80948a808a0
Merged-In: I84bd4c6bc4277e32cee9916dd18dd80948a808a0

Merge "Add 5G evolution icon"

Merge "Make the network stack a persistent process."

Add 5G evolution icon

This add the 5GE icon to the system UI for specific carriers
LTE CA network.

Bug: 124768697
Test: build & manually test:
fake the operator name with pattern, and check the status bar data icon.

Change-Id: Ife0401219138c7ed22e7bfbd4d5e8648c54a98a8
Merged-In: Ife0401219138c7ed22e7bfbd4d5e8648c54a98a8

Merge "Remove unused methods from INetworkManagementService"

Add privapp permissions to network modules.

When built as part of the system, the network stack and the
captive portal login app should have the BYPASS_PRIVATE_DNS
and CONNECTIVITY_USE_RESTRICTED_NETWORKS permissions.
These are necessary to validate restricted networks, bypass VPNs,
bypass private DNS for captive portal login, etc. Add these
permissions to privapp-permissions-platform.xml.

When installed as mainline modules that cannot use signature
permissions, the modules get this ability by virtue of having the
MAINLINE_NETWORK_STACK permissions.

Additionally, add the CONNECTIVITY_USE_RESTRICTED_NETWORKS
permission to the captive portal login app manifest, which did
not contain it.

Bug: 129789428
Test: builds, boots
Test: dumpsys package shows permissions
Change-Id: I632359f7eff09fed71167733ac75824a5aa57894

Update exception type thrown when NetworkStack is missing

rethrowFromSystemServer is throwing DeadSystemException which
is different from the original log message. Thus, update the
way to rethrow the same RemoteException.

Bug: 130028724
Test: atest FrameworksNetTest
Test: Kill NetworkStack and check the log message
Change-Id: I60862e276dd4e2d143278b272a9ba54219acce26

Make the network stack a persistent process.

The network stack should never be killed, because if it does,
recovering networking requires a reboot. Make the process
persistent to reduce the chances of it being killed by the OOM
killer.

Bug: 131046856
Test: builds, boots
Test: process oom_score_adj is -800
Test: process goes from "Persistent Service" to "Persistent" in dumpsys meminfo
Change-Id: Ic8ebf6356e23489f6cd984d680ff9f1b4cbbe52d

Merge "Switch to HexEncoding from ByteStringUtils"

Merge "Delete unused timeout constant."

Merge "Reinstate new VPN uid filtering unit tests"