Move NetworkStackTests to presubmit

The tests have been green in postsubmit for a while and run very
quickly.

Test: atest NetworkStackTests
Change-Id: I9dbbf9046dc0878c355a5ae791e31166038df796

Remove NetworkStack test dependency on testutils

frameworks-base-testutils is close to unused in the tests, and has the
side-effect of adding some unrelated tests to be run in
NetworkStackTests like TestLooperTest. TestLooperTest is also failing in
some cases.

Bug: 127908503
Test: atest NetworkStackTests
Change-Id: Iebc174ed1362e8c341d765129a7fc51f8ed1999d

Merge "LocalLog: add overloaded dump and reverseDump methods."

Merge "Expose some APIs as test APIs to facilitate CTS"

Merge "Remove gaillard@google.com from OWNERS of statsd"

Merge "Check if DUN APN exists to decide isDunRequired"

Merge "Change return value of getTetherApnRequired() to a boolean."

Merge "Fixed all APIs not meeting API review requirement"

Merge "Ensure handleUpdateLinkProperties runs on the CS handler thread."

Merge "getConnectionOwnerUid: Clarify documentation"

Merge "Add @NonNull and @Nullable annotations to VpnService#Builder."

Merge "Fix Automated API Review: InetAddresses"

Merge "Revert "Revert "Revert "Enable static overlay for SettingsLib resources""""

Revert "Revert "Revert "Enable static overlay for SettingsLib resources"""

This reverts commit 007c0b0a4346a1304a09864caa71dc872bd4d18e.

Reason for revert: broke git_pi-dev-plus-aosp build

Change-Id: Iaf901d36ab37a5be6f8e0ee3c8a190f78bdd443a

Remove gaillard@google.com from OWNERS of statsd

Test: not needed for OWNERS change
Change-Id: I6c15067c86dc38784979d6da1265ceb0a3fc5ce5

Merge "API tweaks round 2"

Fixed all APIs not meeting API review requirement

To address the API review feedback provided by
the API council.

Bug: 126702995
Bug: 126701710
Bug: 126702357
Bug: 126701113
Bug: 126701239
Bug: 126700713
Bug: 126700064
Test: Manual
Change-Id: I4b902c1fc2c09133022b05dacdda8964436205d2

Expose some APIs as test APIs to facilitate CTS

Test: added new cts tests
Bug: 123896388
Merged-In: I742f23be57e731307338591d816f7888e198b333
Change-Id: I742f23be57e731307338591d816f7888e198b333

Merge "Preload libnativehelper.so which cannot be loaded through the Java classloader."

Merge "Revert "Revert "Enable static overlay for SettingsLib resources"""

Merge "add Nullable annotation to TelephonyAPIs"

Preload libnativehelper.so which cannot be loaded through the Java classloader.

Test: atest CtsHardwareTestCases (between 2 and 4 failing tests, but that
  happens on clean ToT too; no dlopen failures)
Bug: 126601993

Change-Id: Icfe58fad93b97992e39758c02c92c488a13c717f

Merge changes I2eef0a3c,I263a1774

* changes:
  Also check Zygote runtime options when testing flag `enable_apex_image`.
  Refactor Zygote runtime option checking in runtime_native_boot-flags-test.sh.

Add @NonNull and @Nullable annotations to VpnService#Builder.

Bug: 126699070
Test: n/a
Change-Id: Ief0066636ca6ae4fefbdf70b8dfdfacf2a44e0e6

add Nullable annotation to TelephonyAPIs

Bug: 126702627
Test: Build
Change-Id: I0fa9370840ceda70b181976fa671cd962bdfcbca

Merge "Migrate frameworks/base/tests/testables and its users to androidx.test"

Merge "Add euicc privileged actions and extras."

Migrate frameworks/base/tests/testables and its users to androidx.test

See go/jetpack-test-android-migration

Test: atest TestablesTests NetworkStackTests FrameworksServicesTests SystemUITests FrameworksUiServicesTests ExtServicesUnitTests
Exempt-From-Owner-Approval: already reviewed
Change-Id: I083fcdaa71b503535aa2b3c257740f3ecf055373
Merged-In: Ib04d80954bd8536914d88c66b28e6632e60a6245

Merge "Fix ImsMmTelManager API in accordance with the council"

Merge "Make Uri.toSafeString() also handle 'rtsp'"

Also check Zygote runtime options when testing flag `enable_apex_image`.

Test: core/jni/runtime_native_boot-flags-test.sh
Bug: 119800099
Change-Id: I2eef0a3c1f5254b5e270b35477649047c9db40a6

Refactor Zygote runtime option checking in runtime_native_boot-flags-test.sh.

Generalize runtime option checking functions, so that they can be used
to check options other than the ones starting with `-Xgc:`

Test: core/jni/runtime_native_boot-flags-test.sh
Bug: 72446017
Bug: 120794191
Bug: 123754583
Bug: 119800099
Change-Id: I263a177486f8e24ce92928973f7c8853baf0983b

Merge "Support dexopting shared libraries."

Merge "Switch android.test and junit classes to use UnsupportedAppUsage"

Merge changes I10c6ed68,I14d6defe

* changes:
  Check Android Runtime (Boot) device configuration flag `enable_apex_image`.
  Refactor unit test for Android Runtime (Boot) device configuration flags.

Fix Automated API Review: InetAddresses

Add @NonNull annotations as required by the automated review.

Bug: 126702338
Test: make checkbuild
Change-Id: I9057f06de8d7ea846af00d5cefa744d4206b4b63

Switch android.test and junit classes to use UnsupportedAppUsage

The UnsupportedAppUsage annotations could not be added directly to the
java files in src/ as they have to be built against the current api
which does not include the annotation. Instead this uses the same
technique as used for libcore/ojluni files and adds the annotations to
stub files (in hiddenapi/src) which are built as part of the
android.test.base-hiddenapi target. That target is added to a special
whitelist in build/soong/java/config/config.go which causes the
hiddenapi information to be extracted from the target.

Also, updates the preupload check to prevent anymore entries being
added to the config/hiddenapi-greylist.txt for android.test or junit
classes.

Bug: 73711752
Test: m cts-hiddenapi_flags-csv and check that it contained the
        correct entries even though they had been removed from
config/hiddenapi-greylist.txt
Change-Id: Ifaf15d2751f54cb03f8402b866a0ee4da7acc4d2

Support dexopting shared libraries.

Test: adb shell setprop dalvik.vm.boot-image /system/framework/apex.art && see compilations in dalvik-cache
Test: BackgroundDexOptServiceIntegrationTests
Test: DexoptUtilsTest
Bug: 119800099
Change-Id: I3445b8743d576e09c9a339602307ba3a219db1fc

Merge "Set NetworkStack targetSdk to 28"

Add euicc privileged actions and extras.

Bug: 122769846
Test: Test on phone
Change-Id: Ifcf6575c3c65cb38e8787e32bc04b0e4532f7170
Merged-In: Ifcf6575c3c65cb38e8787e32bc04b0e4532f7170

Merge "Remove ConnectivityManager and its usages from NetworkStatsService."

API tweaks round 2

Add nullability annotations and change a static constant to a method
that returns a value.

Fixes: 126702861
Fixes: 126699926
Fixes: 126698717
Fixes: 126699057
Fixes: 126701443
Fixes: 126700882
Test: compiles
Change-Id: I96396eb8890b8b5148cf74447e08c31983398e47

Set NetworkStack targetSdk to 28

Test: flashed, booted, WiFi works
Change-Id: I7bbd7eb563432a290358d16652c0ae69e9772002

Merge "add NonNull and Nullable annotation to APIs"

Merge "Add a Constant for CellInfo#UNAVAILABLE_LONG"

Make Uri.toSafeString() also handle 'rtsp'

Bug: 123669012
Test: atest CtsNetTestCases:android.net.cts.UriTest
Change-Id: I082bdbd55589f6b99a8cf5fc68c897477f2fa792
Signed-off-by: Eunyoung Moon <eunyoung.moon@lge.com>

Merge "Fix to hide phone number printed in the log"

Revert "Revert "Enable static overlay for SettingsLib resources""

This reverts commit 086c189e20b27209c07af08264050f6c7b60cf48.

Fixes: 116821732

Reason for revert: blocking issue in b/124370422 is resolved

Change-Id: I9ce03ce19b36f08640142c52d759386b83a2c55e

Merge "revert "HLS seeking: call readAt() on new thread""

Remove ConnectivityManager and its usages from NetworkStatsService.

NSS needed it for getting VpnInfo[], NetworkState[] and
activeLinkProperties which it used to query via ConnectivityManager.

For VpnInfo[], this was racy as NSS may ignore intermediate changes to a
VPN's underlying networks. See http://b/123961098 for more context.

It may also lead to deadlocks b/w ConnectivityService and
NetworkStatsService. See http://b/126245192 for more info.

This change will ensure that NSS is never contending on any of
ConnectivityService locks.

Bug: 123961098
Bug: 126245192
Bug: 120145746
Test: atest FrameworksNetTests
Change-Id: Id1da446b54d95ee68ed14079107b1a10318bcf8b
Merged-In: I57e117bb4e9efe491b19d6b5a479f2d58d1c58e6

Merge "Show mobile icons with left-to-right in order of slot index"

add NonNull and Nullable annotation to APIs

Bug: 126699430
Bug: 126699963
Bug: 126700457
Bug: 126700771
Bug: 126702453
Bug: 123939479
Test: Build
Change-Id: Ib429d2918e63f8d272fc7842b0abd83ffd5a9090

Check Android Runtime (Boot) device configuration flag `enable_apex_image`.

Test: core/jni/runtime_native_boot-flags-test.sh
Bug: 119800099
Change-Id: I10c6ed68eceebef14207bc38cab4e96fb2f75939

Refactor unit test for Android Runtime (Boot) device configuration flags.

This is in preparation of checking other device configuration flags.

Test: core/jni/runtime_native_boot-flags-test.sh
Bug: 72446017
Bug: 120794191
Bug: 123754583
Bug: 119800099
Change-Id: I14d6defe5f7e9fcf2f3dd27a4d0fae81f2aec596

Fix ImsMmTelManager API in accordance with the council

Bug: 121190195
Test: atest FrameworksTelephonyTests
Merged-In: I5d011e1875c082a4532747e30e8582ff3da4ee93
Change-Id: I5d011e1875c082a4532747e30e8582ff3da4ee93

Merge "Update statusbar color,nav divider color of settings theme"

Merge "Move ApplicationInfo to params in AppComponentFactory"

Merge "Set networking module APKs version to 10"

Merge "Apilint: Lint missing nullability annotations"

Merge "Fix broken javadoc links"

Merge "Update annotations for UiccCardInfo APIs"

Apilint: Lint missing nullability annotations

Also adds support for limited range blame parsing.

Bug: 124882145
Test: python apilint_test.py
Change-Id: Idbc7f67eeff27902c37ec482278f5888781c6e78

Move ApplicationInfo to params in AppComponentFactory

Tweak AppComponentFactory.instantiateClassLoader() API to accept
ApplicationInfo as an argument instead of an independent getter.

Bug: 124800390
Bug: 111342996
Test: atest CtsClassLoaderFactoryPathClassLoaderTestCases
Test: atest CtsClassLoaderFactoryInMemoryDexClassLoaderTestCases
Merged-In: I4a4d5af842446a5060083c88a0ac5bec014c0c55
Change-Id: I4a4d5af842446a5060083c88a0ac5bec014c0c55
(cherry picked from commit 33bd343596d6272913c47213e1a7d896ba42db00)

Merge "Apilint: Enforce final builders"

Apilint: Enforce final builders

Test: run apilint
Bug: 124883027
Change-Id: I7f2866e46bc40be60487846b2af9f8fd7edb7623

Merge "Add Os.connect(fd, SocketAddress) to public API"

Add Os.connect(fd, SocketAddress) to public API

1. There are already related socket API methods that return
   SocketAddress, such as Os.getpeername().
2. Having APIs that take SocketAddress are a closer match
   to the POSIX APIs that android.system.Os is exposing,
   and are more flexible than just taking InetAddress+port
   combinations since the latter only supports IPv4 and
   IPv6.

This was covered by tests in change:
Iadf9f10e26bbc650c5e1b9d03f32c5dafb6f21b4

And completes Ic496c1212dc2d97cf34db2a7c7d024b02790e8ed.

Test: m
Bug: 123062477
Change-Id: I885bfe1bf24a361bef2bb2f783622b934637ba99

Set networking module APKs version to 10

Up to now the version in output APKs was 28 (platform version by
default). This should make it easier to track and bump up the version.

Test: booted, WiFi works
Test: aapt dump badging shows version 10
Change-Id: I199eb55fd82e16ae748c8b011ce93cf48b505395

Merge changes from topic "nat64"

* changes:
  Catch ServiceSpecificException instead of IllegalStateException.
  Manage NAT64 prefix discovery lifecycle in the framework.
  Track NAT64 in the framework and start clatd iff NAT64 detected
  Change Nat464Xlat lifecycle.
  Minor improvements to verifyTcpBufferSizeChange.

Update annotations for UiccCardInfo APIs

Fixes: 124800040
Test: no change to behavior
Change-Id: I0644eb9bac417b9a8b0fd8eaf691728ed0784527

Merge "Added a filegroup for statsd config proto definition."

getConnectionOwnerUid: Clarify documentation

Test: build
Bug: 117573763
Change-Id: Ia899a541987b9c72a83287ed69fc47ab4d38f680

Merge "Implement isRemovable"

Merge "Add OBSERVE_ROLE_HOLDERS permission to shell."

revert "HLS seeking: call readAt() on new thread"

This reverts commit Ie527aeaff91e1b82c7e707a6feaf79548c7ac380

Bug: 119900000
Test: go/ag/5140159

Change-Id: Ia8b2f3efc2f3551e749c16c6c8dfbc070f8bb980

Merge "Expose DataSpecificRegistrationState as System API"

Merge "Add Documentation for ACTION_DEBUG_EVENT"

Merge "No need to pass shared library infos explicitly in performDexOpt."

No need to pass shared library infos explicitly in performDexOpt.

The method can just infer it from the passed package.

(cherry picked from commit 1158b87c6ea6d809c2ae665cc0db9f464515ced2)

Test: m
Change-Id: I676ee7d49600a36a8bd54893679d7dc900da2012
Merged-In: I676ee7d49600a36a8bd54893679d7dc900da2012
Exempt-From-Owner-Approval: approved in internal.

Merge "apilint: Correctly parse packages with annotations"

Ensure handleUpdateLinkProperties runs on the CS handler thread.

In its own change for ease of rollbacks due to the risk of
possibly crashing existing codepaths.

Bug: 65674744
Test: atest FrameworksNetTests
Test: builds, boots. Wifi, cell data, private DNS work
Change-Id: I2c0acc1c7b8367803f17b4a12c1df0fdfbc29691

Catch ServiceSpecificException instead of IllegalStateException.

The old NetworkManagementService methods to start clatd threw
IllegalStateException, but the new netd methods throw
ServiceSpecificException. Update the catch clauses so that if
starting clatd fails, the system doesn't crash.

Also stop logging stack traces but only include the exception
message itself.

Bug: 65674744
Test: atest FrameworksNetTests
Test: connecting/disconnecting to v6-only wifi in a loop does not crash
Change-Id: I4a9ec7f104712fbbe08f4f67e3288df03e8ed873

Fix broken javadoc links

Bug: 123683994
Test: Verify with javadoc
Change-Id: I4b1b936e3cf86104ace5fcb6829778710801f2d5

Merge "Use public APIs instead of Conscrypt ones"

Merge "Add @NonNull annotation to set proxy on VPN parameter."

Update statusbar color,nav divider color of settings theme

Update statusbar color, navigation divider color
of "Theme.DeviceDefault.Settings".

Bug: 126141701
Test: build & manual
Change-Id: Ib4ddb97c6473a91f3dd6c4971bb150d90a48f202

Merge "Revert "Update MbmsDownloadServiceBase to reflect implements IInterface""

Add @NonNull annotation to set proxy on VPN parameter.

Test: manual
Bug: 124883120
Change-Id: Ifb4bd98a69a006220b1a824033395fc6f12ea6ce

Add OBSERVE_ROLE_HOLDERS permission to shell.

This allows testing relevant APIs in CTS tests. The listener itself
only gives the changed role name and user affected.

Bug: 125404675
Test: build
Merged-In: I5c979a26dae103ea1b633c2119c59363d8953aa2
Change-Id: I29ca1827e597d318cbd74ba3e70796c5853a9dc4

Add Documentation for ACTION_DEBUG_EVENT

The debug event action lost out on the documentation
for DebugEventReporter when it migrated to TM. This
change adds documentation for ACTION_DEBUG_EVENT
EXTRA_DEBUG_EVENT_ID, and EXTRA_DEBUG_EVENT_DESCRIPTION

Bug: 126178766
Test: make offline-sdk-docs (docstring-only change)
Change-Id: I6073d8b181c1258dbd7b47e8d5a45732d619396a

Revert "Update MbmsDownloadServiceBase to reflect implements IInterface"

This reverts commit ddc496d62210fd06f7132e701fbc5df0b2c237f6.

Reason for revert: b/126374865

Change-Id: I53f87d15712a0f2acc5787efcf68c72021e9b831

Merge "Log the pid of the forked zygote child."

Added a filegroup for statsd config proto definition.

This is used in other parts of the framework for serializing some
statsd text proto config at build time. The filegroup allows aprotoc
to access the definitions elsewhere in the source tree.

Bug: 120569785
Test: Build is not affected.
Change-Id: I82a5f8869a2d05ea27d205f471b9c5b5b875fefc
Merged-In: I0e37a9b9a1f22a5dcebbf07ccc5cad98cf76024c

Expose DataSpecificRegistrationState as System API

  LteVopsSupportInfo is currently not accessible for telephony
  APK since DataSpecificRegistrationState is @hide api. This info is
  required to determine if current LTE network support IMS. This
  change exposes DataSpecificRegistrationState and LteVops info
  as System API.

Bug: 112194535
Test: atest

Change-Id: Iec5c42ccef88b66c3f8ef05f8f96b2a4133ca63f

Merge "Rename the RTT audio flag in ImsStreamMediaProfile"

Implement isRemovable

Add resource non_removable_euicc_slots which lists the slot indexes
which refer to non-removable eUICCs chips.

Test: EuiccCardTest
Bug: 122738148
Change-Id: I8836e25acf37527bbb067538902de056e1465b31
Merged-In: I8836e25acf37527bbb067538902de056e1465b31

Merge "Switch LocalSocket to android::base::{Send,Receive}FileDescriptorVector."

Log the pid of the forked zygote child.

Useful to link together the pids of the zygote and its children in logcat
output.

Test: Flash and boot, run "adb logcat"
Change-Id: I524b70aed7d1ce7a378df0e2b2063e23f1f9a454

Merge "Fix the build failure with enabling DEBUG_PSS"

Merge "Remove unused tests/utils/SleepUtils"