Merge "Remove unused / invalid import"

Merge "Dismiss Dialog in post instead of immediately"

Merge "Add libbinderthreadstate to LOCAL_JNI_SHARED_LIBS for some tests"

Merge "Allow wifi and ethenet connected at same time."

Merge "using the correct intent extra for subId"

Merge "Add noexcept to move constructors and assignment operators."

Merge "Prototype XML view compiler"

Merge "Do not use StringPiece in ExtractJavaIdentifier"

Add libbinderthreadstate to LOCAL_JNI_SHARED_LIBS for some tests

libbinder and libhwbinder now also depend on libbinderthreadstate. Add
this dependency to LOCAL_JNI_SHARED_LIBS since these libraries need to
be packaged together.

Bug: 110364143

Test: atest FrameworksNetTests
Test: atest FrameworksServiceTests (no new failures)
Test: atest FrameworksUiServiceTests

Change-Id: Ibc63f041b8ee23cc4d0e9d82a508a128104e6025
Signed-off-by: Jayant Chowdhary <jchowdhary@google.com>

using the correct intent extra for subId

Some telephony subscription related intent uses hidden extra
PhoneConstants.SUBSCRIPTION_KEY. We should sent the public extra
SubscriptionManager.EXTRA_SUBSCRIPTION_INDEX instead.

Bug: 116754807
Test: telephony unit test
Change-Id: I842df9b3fa7c0c8efed2db6ac5cd6a8c34ab3ee4

Prototype XML view compiler

This is an initial step towards a tool for pre-compiling layout XML files.  It
accepts an XML file and produces a Java language class called CompiledView with
a static method, inflate. Calling CompiledView.inflate should then return a view
object that is equivalent to calling LayoutInflater.inflate on the same
resource.

There are still several important limitations, but this works well enough to do
some experimentation. The limitations include:
* Currently only one layout can be compiled at a time.
* `merge` and `include` nodes are not supported.
* View compilation is a manual process that requires code changes in the
  application.
* The tests in this CL do not yet exercise any interesting behavior.

Bug: 111895153
Change-Id: I3e6880b08c52087d24ae7486495bd7fa282f4ff7

Merge "Revert isResolvingImsBinding ITelephony interface"

Add noexcept to move constructors and assignment operators.

Bug: 116614593
Test: build with WITH_TIDY=1
Change-Id: Ic3993d88eb3066aef402d74c4d543c56a1dfd5c3

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"

Allow wifi and ethenet connected at same time.

This will improve the user experience on Android TV devices,
see bug for details.

In addition when connecting adb to the device by ethernet
for cts, wifi will not connect, causing lots of tests to fail.
For example:
[CTS7.1]android.net.wifi.cts.WifiInfoTest#testWifiInfoProperties
[CTS7.1]android.net.cts.ConnectivityManagerTest#testConnectivityChanged_
manifestRequestOnlyPreN_shouldReceiveIntent

Use command:settings to put global wifi_data_always_on 1 to enable it.

Bug: 26102779
Test: Manual, CTS.
Change-Id: I711d93061a6bc7164d98a858912f781e1b967406

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

Remove unused / invalid import

NavigationBarCompat imports sun.misc.Resource though
it doesn't need it.

Bug: 113148576
Test: build only
Change-Id: I26d6e0b3d9a8948333fdf9f3659b87b3b46a75d5

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

Do not use StringPiece in ExtractJavaIdentifier

Bug: http://b/91353691

After assigning the result of TransformToFieldName to 'result', the
underlying storage is destroyed after the 'if' statement of the function
call.  'result' ends up with garbage if the identifier has a '-'.

ManifestClassGeneratorTest.NormalizePermissionNames is broken for this
reason in 32-bit Windows when using libc++ and 32-bit Linux.  ASAN also
reports this failure for both 32-bit and 64-bit linux.

Test: Run test on the cases mentioned above and ensure all of them pass.
Change-Id: I69163c423c1171b7ac7838f2abe06bdf8058df4c

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"