Merge "Revert isResolvingImsBinding ITelephony interface"

Merge "Add TEST_MAPPING for DexManager etc."

Merge changes from topic "bugfix_content_path"

* changes:
  Add OEM_SOUNDS_DIR variable
  Fix the bug that secondary storage path is judged as internal
  Revert "MediaStore.Audio.Media.getContentUriForPath() returns unexpected content"

Merge "Check for null socket name in socket_connect_local."

Merge "Fix: statsd path in whilelist of native processes"

Add TEST_MAPPING for DexManager etc.

Test: atest
Change-Id: I2a92ab9f32faf905aa6dc591e1991ba30c6a26c9

Fix: statsd path in whilelist of native processes

This whitelist is used when calling Process.getPidsForCommands. That
method reads all pid directories under /proc and returns only those
which /proc/pid/cmdline matches the whitelist. Statsd returns
/system/bin/statsd in its cmdline and therefore was not included in the
results and ignored from reporting.

Bug: 116784348
Test: cat /proc/pid/cmdline for statsd
Change-Id: I2c37a25dd26baac1ac043f9a559a391bdc32041e

Merge "Cleanup."

Merge "ConnectivityService get INetd directly."

Merge "Remove spurious WTF in rematching networks."

Merge "Make requestRouteToHost a no-op for system callers."

Remove spurious WTF in rematching networks.

By construction, this WTF should never happen, since it's in an
if (nri.request.isRequest()) and by definition requests can only
be satisfied by one network at a time.

I don't think we've ever seen this particular WTF in an APR
report, which suggests that it's not happening in practice.

Test: atest FrameworksNetTests CtsNetTestCasesLegacyApi22 CtsNetTestCasesLegacyPermission22 android.net.cts.ConnectivityManagerTest
Change-Id: Icf4c7d2bb1da3c7db695cf0bcebc5806190a1677

Cleanup.

Test: runtest FrameworksNetTests
Change-Id: I8abb6de84dad79f6b15ef3af80da49da1d356cff

ConnectivityService get INetd directly.

This is the first step for ConnectivityService
call into INetd directly.
Import INetd and get it by using NetdService.

Test: runtest frameworks-net passes
Test: manual testing of wakeupAdd/DelInterface works
Change-Id: I643dba5206c66958134152d062f3f3a19a34cf2c

Make requestRouteToHost a no-op for system callers.

Everything in the system should now be using proper multinetwork
APIs instead of this insecure and error-prone API.

Make this method do nothing when called by the system. For now,
keep the code around for backwards compatibility for apps
targeting Android releases before M.

Bug: 25824776
Bug: 25876485
Test: FrameworksNetTests pass
Test: CtsNetTestCasesLegacyApi22 pass
Test: CtsNetTestCasesLegacyPermission22 pass
Test: android.net.cts.ConnectivityManagerTest passes
Change-Id: I9b3557faccccc95c7b954db6a13b853b4c7edea0

Merge "Tether-related commands porting"

Merge "Rename object mNetd to mNetworkManagementService"

Merge "Fix idle timer rule leakage"

Merge changes I0f997d30,I288ba1f4

* changes:
  Enable new DHCP server by default
  Use new DHCP server in TetheringTest

Merge "Handling potential SIGPIPE signals."

Merge "Increasing TombstoneSection timeout to 2 minutes."

Merge "Replace libsuspend with android.system.suspend@1.0"

Merge "Add vendor notice header at the top of Third-party licenses"

Merge "InetDiagSocketTest: enable UDP unit tests on all devices"

Merge "Add config to only allow auto select of networks"

Merge "Remove getNeighboringCell API"

Add vendor notice header at the top of Third-party licenses

Vendor can add notice header at the top of "Third-party licenses"
by overlay the string notice_header.

Fixes: 116298367
Change-Id: If80d69180664970441d0addccd81d65d7ab55c3b
Merged-In: If80d69180664970441d0addccd81d65d7ab55c3b

Revert isResolvingImsBinding ITelephony interface

Reverting the only user of this interface.

Bug: 109762920
Bug: 116766786
Test: GTS/Unit tests for ImsService
Merged-In: I9c8cc1c4401590865f33fa13402152287063cecf
Change-Id: Id2e1ac2e4fbbcb7f019e79568f21ee56c94b6c1f

Merge "Don't check qtaguid file unless using them"

Replace libsuspend with android.system.suspend@1.0

This change only replaces libsuspend's suspend loop with the one in
android.system.suspend@1.0.

Native wake lock behavior is unaffected as they are still handled via
/sys/power/wake_[un]lock.

Bug: 78888165
Test: device boots, suspends when left alone
Change-Id: I5e1e9048b0fc316aa6692687684ac11f44466f7e

InetDiagSocketTest: enable UDP unit tests on all devices

Bug: 116849885
Test: atest InetDiagSocketTest
Change-Id: I3b036818cccd0959feb3a93bc722d70e3ef2c4d7

Check for null socket name in socket_connect_local.

socket_bind_local checks if the name is null before using it, but
socket_connect_local doesn't, resulting in a segfault if you pass null
as the socket name. Check for this and convert it to
NullPointerException to make the crash more reasonable.

Test: am instrument -w -e class android.net.LocalSocketTest com.android.frameworks.coretests/android.support.test.runner.AndroidJUnitRunner
Change-Id: I67dedbe9ca1be6425e047496d0ead92eb7f64691

Fix idle timer rule leakage

Idle timer rule is not cleared as expected if there is a
default network replacement.

Bug:37080406
Test: 1.run frameworks-net
      2.check iptables rule with default network replacement

Change-Id: I6bd29d79e4ca3e8de4b867c4fcb5f81d02ba6de4

Merge "Call clearSettingsProvider before and after test"

Merge "Add new DelegateLastClassLoader API"

Rename object mNetd to mNetworkManagementService

No functionality change

Test: runtest frameworks-net passes
Change-Id: Ic3977eed9a86deb76ef0e4e12ffa17597bdecc30

Call clearSettingsProvider before and after test

FakeSettingsProvider requires this method to be called before and after
use. Without this, the settings value or content provider may be cached
statically, so the test will be affected by code accessing settings
before it is run.

Bug: b/116668105
Test: atest FrameworksNetTests
Change-Id: I1480f3f3bbb17791752582a70327bb5c7c348d7c

Enable new DHCP server by default

Regression tests pass with aosp/764042.

Test: regression automated tests, manual (verified WiFi, bluetooth
      tethering working).
Bug: b/109584964

Change-Id: I0f997d302a1d85930b24139af76e9b9c9a1b5f78

Use new DHCP server in TetheringTest

Run most tests with TETHER_ENABLE_LEGACY_DHCP_SERVER set to 0 (will be
the default value). Add one test to verify that the new server is not
started when TETHER_ENABLE_LEGACY_DHCP_SERVER is 1.

Bug: b/109584964
Test: atest FrameworksNetTests
Change-Id: I288ba1f434918e62ff29f7ace00856108c9730f7

Merge "Don't throw a NPE in SharedLog#e"

Merge "Catch IllegalArgumentException on NetworkScan.stopScan"

Merge "Implement equal method in SubscriptionInfo."

Implement equal method in SubscriptionInfo.

Bug: 115541873 92796390
Test: manual with test app
Change-Id: I217f9d612df5cecac8f0359649f3e79a60a95dde
Merged-In: I217f9d612df5cecac8f0359649f3e79a60a95dde

Catch IllegalArgumentException on NetworkScan.stopScan

IllegalArgumentException is thrown when a terminated
scan calls stop. This change softens the exception and
error type that is logged and considers this normal.

Bug: 113823866
Test: compilation
Change-Id: I7ce60d6badfac52156cceca83b39c1afe21b4eda

Add config to only allow auto select of networks

Make it possible for carriers to prevent users from
doing a manual search of networks when in the home network.
This affects Advanced Networks Settings when the user
is not roaming.

This is sometimes called "Permanent automatic mode".

Test: Add an entry in vendor.xml
Bug: 70499144
Change-Id: I9fe91a8510683e4bb3d51efb168c0d159dbb4591

Merge "Revert "Revert "new System API for preferred network mode"""

Revert "Revert "new System API for preferred network mode""

This reverts commit dada4c60d83730d706c4ebe549c3460e596a187e.

Reason for revert: Breaking aosp-master builds.

Change-Id: Ie48b2125025878ce13953706a7eec24bce38594b

Merge "Make pre-scale for absolute volume configurable"

Merge "Revert "new System API for preferred network mode""

Revert "new System API for preferred network mode"

This reverts commit 8a8832fd810710fd8fbd7868be49abff270a1013.

Reason for revert: Causing failures on git_pi-dev-plus-aosp for docs and aosp_sailfish.

Change-Id: I1801188e66420a67244b3223e26334c4650d56be
Merged-In: Ic108c484905f80783982a22e8152609257d684b5

Don't throw a NPE in SharedLog#e

Some DhcpServer error code paths could cause a NPE, when the logError
callback was called without an exception. Allowing SharedLog#e to be
called with a null Throwable is less error-prone.

Bug: b/109584964
Test: atest FrameworksNetTests
Change-Id: Idbcdd330a9d1951b27aaf525aaf12e52e102872c

Merge "Fix trial run in AppLaunch test"

Merge "new System API for preferred network mode"

Make pre-scale for absolute volume configurable

Audio gain for lower volume steps are restricted for
Bluetooth Absolute Volume scenario, but it's better
not to use the fixed value.
This CL makes it possible to configure the value of
pre-scale.

Bug: 114220617
Test: manual - gain is restricted as per the configuration

Change-Id: I1fd0c77476386ba9518e1819d5ea2c7b0c344a40
Merged-In: I1fd0c77476386ba9518e1819d5ea2c7b0c344a40

Merge "Dismiss SIM lock page after correct SIM"

Merge "Change FileStream test buffer from char* void*"

Merge "Cleanup deprecated or removed telephony methods."

Change FileStream test buffer from char* void*

It seems like GoogleTest prints string parameters before checking them.
This can be a problem in FileStream tests since these buffers are not
null terminated.  ASan reports heap-buffer-overflow.

Test: m SANITIZE_HOST=address
out/soong/.intermediates/frameworks/base/tools/aapt2/aapt2_tests/linux_glibc_x86_64_asan/aapt2_tests \
      LD_LIBRARY_PATH=out/soong/.intermediates/external/libcxx/libc++/linux_glibc_x86_64_shared
out/soong/.intermediates/frameworks/base/tools/aapt2/aapt2_tests/linux_glibc_x86_64_asan/aapt2_tests

Change-Id: I9a1956d815729de23c7038fb0d498240eb917070

new System API for preferred network mode

1. new System API for getPreferredNetworkType
2. new System API for preferred network mode

Bug: 115894190
Test: unit test
Change-Id: I34b060d3f915f2c74b2b9412d03f849e9d037c0b
Merged-in: Ic108c484905f80783982a22e8152609257d684b5

Fix trial run in AppLaunch test

If the app does not exist log an error but do not fail the test.

Test: atest google/test/performance/app-install-startup -v

(cherry-picked from commit 55db8ff9a6fa383134142f65d0b1d0da9957411d)
Merged-In: Idb6f1d0ba3c5bfb6c7985cd84f5aa4e0599c6b59
Change-Id: Idb6f1d0ba3c5bfb6c7985cd84f5aa4e0599c6b59

Merge "expose getAidForApptype to system API"

Dismiss SIM lock page after correct SIM

Previously the criteria was to dismiss if:
1. There is no pin/password/pattern set AND
2. the lockscreen is disabled.

This behaved incorrectly in the case where the lockscreen is enabled
but there is no pin/password/pattern set.

We fix the condition so that now we dismiss the SIM lock if:
1. There is no pin/password/pattern set OR
2. the lockscreen is disabled.

Fixes: 115780994
Test: manual
Change-Id: I88da896add635126934eb0ed22e617ac2607f15a

Merge "Make SubscriptionManager.isActiveSubId public"

Merge "Fix ExternalStorageProvider always throw exception when rename"

Merge "Make ext.jar only dependent on public SDK APIs"

Merge "Mark event-based handover constants as deprecated."

Fix ExternalStorageProvider always throw exception when rename

ExternalStorageProvider always throw exception when renameDocument
because it will check file exist when getFileForDocId. However,
beforeVisibleFile is not exist after rename, it will throw
exception. Fix it by put getting beforeVisibleFile before rename.

Test: manual test rename in DocumentsUI
Test: atest RenameActualProviderUiTest
Fix: 113302685
Fix: 115304092
Change-Id: I45d60d2f940e3dbc2b121f7cd28acc3c2d085ec4
Merged-In: I45d60d2f940e3dbc2b121f7cd28acc3c2d085ec4

Merge "Rename method isConnectable()"

Merge "Add intdef for Telephony network type constants"

Merge "Add hidden API to return "Uploading" bit value from SDP record's MapSupportedFeatures."

Merge "Show "Call Blocking disabled" notification always when emergency call"

Merge "expose getIsimDomain to systemAPI"

Merge "Update the reference profile from .dm files only during installs"

Add hidden API to return "Uploading" bit value from SDP record's MapSupportedFeatures.

Bug: 111614861
Test: tested w/ KitchenSink App
Change-Id: I43895183d7b315f57257e1d2045f17dedcb0cfcd

expose getAidForApptype to system API

Bug: 115891723
Test: telephony unit test
Change-Id: Ica1d6facba229781806c2c205406c658ac5ad838

Merge "Add the EMBMS group call API"

Update the reference profile from .dm files only during installs

In order to speed up first boot time we only create the current profile
and do not update the content of the reference profile. A system image
should already be configured with the right profile keys and the
profiles for the speed-profile prebuilds should already be copied (that's
done in #performDexOptUpgrade).

Test: manual (install apps and splits, take OTA)
Bug: 114046578

(cherry picked from commit 5c9a0085a3366b4c7f00586dba9691fac9016f22)

Merged-In: I99db76684430b929c52634c25b457e874648e205
Change-Id: I99db76684430b929c52634c25b457e874648e205

Add intdef for Telephony network type constants

Change-Id: Id212f5009ff2d57344dd747c1837ad9ff1546e31
Fixes: 116592397
Test: compiles

Add new DelegateLastClassLoader API

Add
public DelegateLastClassLoader(java.lang.String, java.lang.String, java.lang.ClassLoader, boolean)

Test: DelegateLastClassLoaderTest
Change-Id: I207962a6bfbdad720c187f9657b17a15c9a89470

Make ext.jar only dependent on public SDK APIs

ext.jar can be built using only public SDK APIs. This
makes that obvious.

Bug: 113148576
Test: make ext
Change-Id: I792b14924878623f832f3afbb1d3e23fba34d5bf

Merge "Reduce visibility on PackageDexUsage methods."

Merge "Fix minSdkVersion for test."

Fix minSdkVersion for test.

P is 28 not 27, and the test requires P.

Test: atest DexLoggerIntegrationTests
Change-Id: Ib83ec92bf5d7e791bac90331b5dcc3c4b826a4fd

Reduce visibility on PackageDexUsage methods.

Not a big win, but it makes it clearer that the data is never directly
changed by PackageManager etc.  (We could extract the two nested
classes and then PackageDexUsage wouldn't need to be public at all.)

Test: Still builds.
Test: atest services/tests/servicestests/src/com/android/server/pm/dex

Change-Id: I0a5c0bff473a03178f730e354a2f60694729ba10

Merge "Inline two short methods from Conscrypt"

Show "Call Blocking disabled" notification always when emergency call

Currently "Call Blocking disabled" notification is shown only when any
setting of "Enhanced Blocked number" is enabled, and it's not shown even
if only "blocked number" is enabled.

This patch adds new CarrierConfig key to show "Call Blocking disabled"
notification always when "Enhanced Blocked number" is enabled and
making emergency call.

Test: manual - Verified behavior about Block numbers settings
Test: auto - Passed BlockedNumberProviderTest
Bug: 115697577
Change-Id: I2281c21992928037ea38227b7476adc76b6755e8

Merge "Add comments for offline-sdk-referenceonly docs"

Add the EMBMS group call API

Add APIs for apps to access a LTE broadcast group call service provided
by a preinstalled middleware app, and add system APIs for the
preinstalled middleware app to communicate with frontend apps.

Bug: 112731375
Test: CTS
Change-Id: Ie6817cbc6c3b69de7a4d66c4cfc103b02e15ad6b

Merge "Add support for network identified emergency calls."

Make SubscriptionManager.isActiveSubId public

Rename subId to subscriptionId

Bug: 113072551
Test: Treehugger
Change-Id: I6f7e0d06a2ad2367dad04723fea808cad8f9597a

Add comments for offline-sdk-referenceonly docs

Add comment to provide some cautions in case of we make some changes on
the doc target names in the future.

Test: N/A
Bug: b/116221385
Change-Id: Ibb21ca51c1eed990fe5dfceb6a8170821e70f8cf

Merge "Fix clang-tidy performance warnings in frameworks/base."

Cleanup deprecated or removed telephony methods.

Mark previously deprecated TelephonyManager APIs as @remove; some of these
have valid public APIs; for now just calling the public APIs to give more
time for API consumers to transition over to the new methods.

Remove ITelephony AIDL calls for removed methods.

Removed ITelephony light greylist for removed methods; see bug for details.
There are public alternatives for most of the remove AIDL methods; a couple
of the subscription specific ones are not available directly but the
non-subscription specific versions should suffice.

Test: Compile/build/manual
Bug: 115776310
Change-Id: Iff4bf99b34b8e14d5a4dc1d2f4e4926880b6434c

Fix clang-tidy performance warnings in frameworks/base.

* Use more efficient overloaded string methods.

Bug: 30411878
Test: build with WITH_TIDY=1
Change-Id: Ia387e61770e1e7211280ed028ef0eef6e632a1d2

Inline two short methods from Conscrypt

We're trying to reduce unnecessary direct dependencies on Conscrypt.
These two methods are simple and the implementations can't change, so
they're good candidates for inlining directly instead of depending on
the Conscrypt implementation.

Bug: 110404540
Test: atest NetworkSecurityConfigTests (same failures pre/post)
Change-Id: I303d955e3f49885326fe75f451c06a52af745053

Merge "Add serialization methods to whitelist not light greylist"

Merge "Expose android.icu.text.Bidi API"

Merge "Enable excluded-input-devices settings for vendor"

Merge "Frameworks: Remove AsyncTask from preloaded-classes"

Merge "Update network selection API"