Merge "Changes to APIs for CarrierRestrictionRules class"

Changes to APIs for CarrierRestrictionRules class

Add @NonNull annotation where applicable.
Rename isCarrierIdentifiersAllowed into areCarrierIdentifiersAllowed.
Make Builder class final.

Bug: 126375826
Bug: 126699079
Bug: 126700761
Test: Compilation
Change-Id: I1a02bc007f147597df97eaa55c00fd31a952870b

Merge "Adding new APN type for Mission Critical Services."

Merge "Fix javadoc HTML escaping."

Merge "Revert "Revert "Revert "Correct nullability and final constraints on new APIs.""""

Revert "Revert "Revert "Correct nullability and final constraints on new APIs."""

This reverts commit b255be1307aeae47ca7e8ee72ab4da670a1c49f1.

Reason for revert: the commit being reverted fixed git_master but broke aosp-master. This revert is aimed to fix aosp-master while not touching git_master.

DO NOT MERGE: This change is aimed at aosp-master only and should not be merged into git_master.

Change-Id: Ie5aaabb211affe2d21daaf87809641276d9d4707

Merge "Use new libcore filegroup variable"

Merge "Revert "Revert "Correct nullability and final constraints on new APIs."""

Revert "Revert "Correct nullability and final constraints on new APIs.""

This reverts commit 8462c66bd52f3132c3fa03475fcab2af30618f67.

Reason for revert: Looks like this change broke the build.

Change-Id: I4cff1ec1211e278d5e5734f1594c096722293662

Merge "Add @Nullable to new method"

Merge "Return not supported if no necessary socket option"

Merge "Revert "Correct nullability and final constraints on new APIs.""

Revert "Correct nullability and final constraints on new APIs."

This reverts commit b6940c3ea6779bc6d062a430d9eebe192943f09a.

Reason for revert: There appears to be a downstream issue with how the test API definitions are created; until I can figure out how to account for this I'll revert in AOSP (this is in internal master and its fine there).

Change-Id: I840c3ed82afa18323408e161913a84858f7a1539

Merge "Correct nullability and final constraints on new APIs."

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

Adding new APN type for Mission Critical Services.

Test: build
Bug: 124447962
Change-Id: I9f608badd363703394432640dc892a4465660b6d

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"

Correct nullability and final constraints on new APIs.

Per API review, correct nullability and final constraints on new Telecom
APIs.

Test: Build API.
Fixes: 126700293
Fixes: 126699447
Fixes: 126700465
Fixes: 126702895
Fixes: 126702527
Change-Id: Icec15403493de596194a61b27bb3e1031fdc1099
Merged-In: Icec15403493de596194a61b27bb3e1031fdc1099

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

Add @Nullable to new method

Add @Nullable to a newly-added method, as requested by API council.

Bug: 126698875
Test: make
Change-Id: I2e72ea457b5adaceea70d9908bb8e2cac2e0d963

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"

Fix javadoc HTML escaping.

HTLM special characters ('<', '>' and '&') don't need to be escaped
inside {@code}, even though they do elsewhere in javadoc. And in fact
they shouldn't be - because if they are escaped then the escaped
version shows up verbatim in the developer docs.

See for example:
https://developer.android.com/reference/android/graphics/drawable/GradientDrawable
https://developer.android.com/reference/android/support/v13/view/inputmethod/InputConnectionCompat

This change just removes the escaping to fix the public documentation.

Test: N/A (only comment lines changed)
Change-Id: I131b3b3e26547e30cff7b67514d177c48926bb26

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"