Use sampled value in hidden api logger

The hidden api logger interface also receives the actual sampled value.
This is an intermediate step before a new logging method is added.

Test: m
Bug: 119217680

(cherry picked from commit 31bde163932dab75a8301e43b2adcb684f0470ec)

Change-Id: I3345bdcdf57e1f60970c85572e503d7b33af7148
Merged-In: I707a2624a254a80391cdae277dd59daa503d055f

Merge "Adding validate before switch feature."

Merge "Add support for Telecom logging to obfuscate string phone numbers."

Merge "Move getPhoneCount implementation back to TelephonyManager."

Adding validate before switch feature.

In CBRS DSDS mode, switching data to CBRS network will require
validation first. Adding a component of CellularNetworkValidator
to trigger the validation and pass the result back to PhoneSwitcher.

Bug: 118348832
Test: manual
Change-Id: I689aa494f031834b5cee76906922ebdafa9c77ed
Merged-In: I689aa494f031834b5cee76906922ebdafa9c77ed

Merge "Turn on enforcing mode for Q-based location APIs"

Merge "Add 'hasKnownUserIntentEmergency' into ImsCallProfile Parcel"

Move getPhoneCount implementation back to TelephonyManager.

Moving it inside iTelephony creates problems. Some components call
getPhoneCount during initialization while iTelephony service is not
even running, which will cause phone process to crash.

Bug: 123667461
Test: manual
Change-Id: I51199eaf0bd87055c21fe61af50f4c353137d621
Merged-In: I51199eaf0bd87055c21fe61af50f4c353137d621

Add 'hasKnownUserIntentEmergency' into ImsCallProfile Parcel

Test: Treehugger
Bug: 123999640
Change-Id: Ie68a02a7e0f9aabe5dbc37806a495169feb63765
Merged-In: Ie68a02a7e0f9aabe5dbc37806a495169feb63765
(cherry picked from commit 0c86fbf491f4781ef3be68ba51aa3bfdb0bd7bc0)

Merge "[DO NOT MERGE] Support trace command on Window Manager dumpsys"

Merge "Lazely initialize AccountManager debug table."

Merge "Convert Android.mk file to Android.bp"

Merge "Add dariofreni to APEX-related files owners."

Merge "Add a dalvik.vm.boot-image option for zygote and installd."

Add dariofreni to APEX-related files owners.

Test: n/a
Change-Id: I3a26033f52d98e5edcde852fff007ba33ae90856

Merge "Fix authority parsing test"

Fix authority parsing test

The test has been failing because http://r.android.com/793302 changed
how the port separator was found, only scanning ASCII digits until it
hits a colon, and previously it would scan for the colon and then
percent-decode the rest of it.  The new behavior actually better
matches the WHATWG URL parsing algorithm [1], which specifies that
ports only include ASCII digits.  It does mean that some edge cases
that previously parsed as host "foo", port 42 will now parse as
host "foo:42", no port, but those URLs wouldn't be accepted by
browsers so they should be exceedingly rare.

The behavior per the WHATWG spec would be to fail to parse in the case
of a percent-encoded character in the port section, but this class is
specifically documented to accept garbage, so lumping the mis-encoded
port as part of the hostname (which will result in a hostname that's
invalid and impossible to resolve) seems like the best option.

[1] https://url.spec.whatwg.org/#port-state

Fixes: 124360078
Test: atest android.net.UriTest
Change-Id: I1c788cb7703c821ae74b542b74d89e10cba5a546

Merge "Fix for testLifecycleOnMoveToFromSplitScreenRelaunch"

Merge "Force blacklist on VMRuntime.setTargetSdkVersion*()"

Merge "Revert "Remove legacy-test and legacy-android-test targets""

Revert "Remove legacy-test and legacy-android-test targets"

This reverts commit 3b0be60307cc6063795f8cd32356745ee371e757.

Reason for revert: Broken build

Change-Id: I71270c7b47163ab7147029f2d3961d1848fdd9eb

Merge "Specify no resource_dirs for SystemUI"

Merge "Remove legacy-test and legacy-android-test targets"

Force blacklist on VMRuntime.setTargetSdkVersion*()

The methods could be used to bypass the restrictions. Force them
onto the blacklist.

Test: builds
Change-Id: I77d1fb8ca7339bec5e933f87a3b672096e2e559e

Merge "Add OWNERS for ExifInterface"

Specify no resource_dirs for SystemUI

The res directory is already included via the SystemUI-core
library. The default for resource_dirs is ["res"] if it exists,
so explicitly clear it to avoid depending on it twice.

Bug: 124035856
Test: m SystemUI
Change-Id: Ia1b06cd9eff4e614684efde34288cf9ebd895870

Merge "Replace TcpSocketInfo with TcpKeepalivePacketDataParcelable in ApfTest."

Add OWNERS for ExifInterface

Adding specific owners for ExifInterface in order to make sure
AndroidX code is changed when framework code is changed.

Bug: 122276930
Test: N/A
Change-Id: Icf0d74ad915d4b322a70c74762753411ad96bbb1

Merge "Rename NetworkStack package"

Merge "Check intent action in OMS.PackageReceiver for secure coding"

Merge "Fix the bad link due to typo"

Merge "Sign networkstack with its own certificate"

Turn on enforcing mode for Q-based location APIs

Also fix some logging.

Bug: 116258458
Test: manual, unit
Change-Id: I370aae906760365d9cd2a35b456d8ff8522c0902

Merge "Fix framework and NetworkStack classes conflicts"

Merge "zygote: use async signal safe log to record child process death"

Fix the bad link due to typo

Replaces the link with the new API name, createInsecureL2capChannel.

Bug: 70683224
Test: Compile
Change-Id: I9580d90722f8b0c0146a902bb5fcace4ef58d421
Merged-In: Ia06e1fffd814671289a1caebd5962aedc18a28d7

Lazely initialize AccountManager debug table.

Prevent crash if the database cannot be opened.
Guard not thread safe SQLiteStatement.

Test: atest com.android.server.accounts
Change-Id: Idab30703facacb469a568361998c59c039d3dd2a
Merged-In: I2af9074e50bddc5a1d18ea781cc6f56934f21302

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

Merge "Populate 'hasKnownUserIntentEmergency'"

Merge "Adding framework check for 3DES key size"

Convert Android.mk file to Android.bp

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

Bug: 122332340
Test: treehugger
Change-Id: I89cb31814f1442c91c31e5b867501d768679230e

Merge "No-op flag to keep tests working in this branch."

Merge "Move RCS table creation flag to contract class"

Merge "Add new file to owners"

Merge "Fix loop iteration bug."

No-op flag to keep tests working in this branch.

Bug: 124507818
Test: manual
Change-Id: I65ae1a55c420690e68f833674d4d09452a826302
Merged-In: I7973b123cf4fc08e8ce2b05bd4c23fa41b1cdcdf

Add new file to owners

Test: None
Change-Id: I69f415929785845ede8f2d1341f4b70f97b4f480

Merge "Update VPN capabilities when its underlying network set is null."

Merge "Move NetworkStack to services.net"

Replace TcpSocketInfo with TcpKeepalivePacketDataParcelable in ApfTest.

Test: atest NetworkStackTests
Change-Id: I95fde70c07b398a2a2f734f35165870284a3959c

Remove legacy-test and legacy-android-test targets

Fixes up the android.test.legacy target to stop using sources directly
now that the issue with android.test.runner-minus-junit not being able
to be built against the SDK has been resolved.

Also, fixes a couple of comments.

Bug: 63127064
Test: make checkbuild
Change-Id: If38fa0d87b36343896e97efb2269ac070810db84

Sign networkstack with its own certificate

The NetworkStack module is comprised of 2 updatable APKs
(NetworkStack.apk and CaptivePortalLogin.apk), and a stub APK to define
the signature permissions for the APKs (NetworkStackPermissionStub.apk).

Test: flashed, WiFi works, captive portal login works
Bug: 124033493
Change-Id: I772d1d63363a3eb6f817c21906498bd2385ab02b

Fix framework and NetworkStack classes conflicts

Classes that are used in framework.jar cannot be linked in NetworkStack,
as the framework takes precedence in the classpath. This prevents the
networkstack from using these classes due to the hidden API usage
detection.

Do the following:
 - jarjar any shared source file between framework and NetworkStack, so
   the version in the NetworkStack uses a different package.
 - Move any shared class not used in the NetworkStack to services.net

The CL  uses jarjar on the app copy and not the framework classes, as
the framework cannot be updated without an OTA, and non-network stack
specific classes should not be renamed because of the network stack.

Test: atest FrameworksNetTests NetworkStackTests
Test: flashed svelte build, WiFi works
Bug: 124033493
Change-Id: I85d888b756adc28c36638913632bfdfdbf0e0486

Merge "Add NetworkStackTests to device-tests suite"

Merge "Remove NetworkMonitor dependency on ICaptivePortal"

Merge "Fix testApfFilterKeepaliveAck."

Add NetworkStackTests to device-tests suite

It's needed for the test to run in a Test Mapping suite.

Bug: 123544631
Test: local build
Change-Id: I0678a3e8f9beed7c637c1b11594369aa5d3c87bf

Move NetworkStack to services.net

NetworkStack is only used in services.net or clients of services.net. It
cannot stay in framework.jar because it needs to depend on AIDL
interfaces, which would conflict with app implementations if they were
in framework.jar.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 124033493
Change-Id: Ib1d08a3669983640119d008db7e2990fa798724f
Merged-In: I501b125a388c1100c2182bde4670944c2f0d7a02

Remove NetworkMonitor dependency on ICaptivePortal

ICaptivePortal is used in the framework and cannot be used as a
dependency in NetworkMonitor, as the framework class takes precedence
when linking.
Also fix NetworkMonitorTest that was not verifying the right
startCaptivePortalApp call.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 124033493
Change-Id: I8e7bb79e50650ae182a2e4277fb49abf5fb6d910
Merged-In: Ib6a89e54312628662b130fbeec18d11e139f09fa

Merge "Remove deps from framework on netd interfaces"

Update VPN capabilities when its underlying network set is null.

Previously, they were only updated when underlying network set was
non-null.

This change also ensures that all the calls b/w ConnectivityService and
Vpn that leads to updating capabilities are on ConnectivityService
handler thread.

Additionally, it also ensures that capabilities are propagated after VPN
enters connected state. This was previously done from establish which
could potentially lead to race between VPN getting connected and
capabilities getting updated.

This change also updates VPN capabilities inline from
ConnectivityService handler thread. Previously, there was an additional
loop where Vpn would update capabilities via NetworkAgent thru
AsyncChannel which posts back to CS handler thread, which could
potentially lead to delays in updating VPN capabilities.

Bug: 119129310
Bug: 118856062
Bug: 124268198
Test: atest FrameworksNetTests
Test: manual - verified VPNs capabilities are getting updated and
DownloadManager is working correctly.

Change-Id: Id0abc4d304bb096e92479a118168690ccce634ed

Merge "Prevent 3rd party app from turning on speaker during the system call Test: manual"

HLS seeking: call readAt() on new thread

Bug: 119900000
Test: go/ag/5140159
Change-Id: Ie527aeaff91e1b82c7e707a6feaf79548c7ac380

Merge "SMS: Add a compatible API getRecipientAddress for 3GPP2 SMS"

Merge "Fixed a typo in an OWNERS file."

Fix loop iteration bug.

This patch fixes a loop iteration bug in the capabilities calculation
code of the Zygote.  This bug was introduced in aosp/804219.

Bug: 124268515
Test: Built
Change-Id: Ic4e3e736c0aa0e2ecaed87ca6f12367be41715f3

Merge "Add Test Network and Transport types"

Add a dalvik.vm.boot-image option for zygote and installd.

Bug: 119800099
Test: adb shell setprop dalvik.vm.boot-image "/system/framework/boot.art"

Change-Id: Iccafc56b5c95e848efca2699e16306a26d856478

Merge "Add :frameworks-base-api-current.txt filegroup"

Fix testApfFilterKeepaliveAck.

Apf program for TCP keep alive ACK also compares protocol field in IPv4
header and the total length of TCP keep alive ACK packet. Besides,
should set the ACK flag in the the TCP header of keep alive ACK packet.

Bug: 123992564
Test: atest NetworkStackTests
Change-Id: I997cd6807a33f7e1af25566226d803dcc2e257dd

Remove deps from framework on netd interfaces

If included in framework.jar, the interfaces conflict with any app that
needs to depend on them, including the NetworkStack.

Bug: 124033493
Test: atest FrameworksNetTests NetworkStackTests
Change-Id: I2db9f87b7154130726d4700b241d55b041635d98
Merged-In: I0ecae20d514bf888f3a80331f19369ceb1c52aa3

Merge "Use ServiceState to get data cell ID"

Merge "Don't run doc-comment-check-docs for update-api"

Merge "Add "close" socket option for immediately close socket after finish check captive portal."

Populate 'hasKnownUserIntentEmergency'

This is only used to specify when the dialed number is ambiguous,
identified as both emergency number and any other non-emergency number;
e.g. in some situation, 611 could be both an emergency number in a
country and a non-emergency number of a carrier's customer service hotline.

Test: Manual
Bug: 123999640
Change-Id: I7b1a8832ab86b3e7d8f9af24eb4f7bde7d566612
Merged-In: I7b1a8832ab86b3e7d8f9af24eb4f7bde7d566612
(cherry picked from commit 6d9e7457dadaf683b2706ba1a41bd8385f25c9a2)

Merge "Fix flaky testNattSocketKeepalives"

Fix flaky testNattSocketKeepalives

The test did not wait for the WifiNetworkAgent to disconnect, and there
are reports of flakes where it is still connected in
connectKeepaliveNetwork.

Test: atest FrameworksNetTests
Bug: 124319553
Change-Id: I85968806885c8853ec4f4dbea356366c1af0ea1c

Merge "Call e.rethrowFromSystemServer() to exception of setDefaultSmsSubId"

Merge "Add build rule of data stall proto"

Add build rule of data stall proto

Bug: 121185319
Bug: 120452078
Test: Build and test against data stall code
Change-Id: I3b243ab947464a431ef205c5f2f30edc883e2963

Add "close" socket option for immediately close socket after finish check captive portal.

Unexpected packets are observed after captive portal check. Due to the packets, modem can't fall-back to idle state (modem remained as active state).
It makes unnecessary power consumtion.
To prevent it, "close" option is added in sendHttpProbe()

Test: check socket close.
Change-Id: I5b6987071ad915081846504adbb209eb371fd1c3

Merge "Clear all lingering notifications when network is disconnected"

Use ServiceState to get data cell ID

ServiceState now contains a CellIdentity that can be obtained
specifically from the data subscription, and for PS on cell
only.

This should be used for connectivity level network monitoring
info.

Bug: 123893112
Test: 1. atest NetworkMonitorTest
      2. Verify if cell id is correct

Change-Id: Ieb00de84a0d518e3197e3884f946b872e998304a

Don't run doc-comment-check-docs for update-api

update-api would be fast except that it spends ~4 minutes doing
doc-comment-check-docs.  doc-comment-check-docs isn't really
related to update-api, so lets just remove it.

Fixes: 124386818
Test: m update-api
Change-Id: I0f5311e98063786671c0d8b6a8db52351d9fcfd7

Merge "Enable DOM storage API for CaptivePortalLoginActivity"

Call e.rethrowFromSystemServer() to exception of setDefaultSmsSubId

Test: Treehugger
Bug: 123927534
Change-Id: Ica15965cb904b758e078b8439e5e9d3f39b9c0a2
Merged-In: Ica15965cb904b758e078b8439e5e9d3f39b9c0a2
(cherry picked from commit 04e94fcb0a41a50dce82f20a603b71614c3f5ed0)

Merge "Add UNINITIALIZED and rename INVALID"

Merge "Implement more location checks"

Adding framework check for 3DES key size

Previously the framework would accept any key size that was a multiple
of 8 for the KeyGenerator.

Bug: 117509689
Bug: 122274787
Test: atest cts/tests/tests/keystore/src/android/keystore/cts/KeyGeneratorTest.java
Change-Id: I60b52f6062a41ae52486bae0ae36616f4b532b37

Add support for Telecom logging to obfuscate string phone numbers.

Test: Manual
Bug: 117472746
Change-Id: I48e29fd9a6ab1d41fdefb55393c7c326ec72a87f

Merge "Manually merge 885aca1bbaa076b11914b9e4cb0d2a6d0c6fab3a to aosp-master"

Merge "Add getSlotsMapping to TelephonyManager"

zygote: use async signal safe log to record child process death

Bug: https://issuetracker.google.com/issues/119713191
Test: open/close apps, observe the child process death log
Test: run monkey
Change-Id: Ia121f44c00f715367446dfe97de6df1f7fa6bdfe
Signed-off-by: wangmingming1 <wangmingming1@xiaomi.com>

Manually merge 885aca1bbaa076b11914b9e4cb0d2a6d0c6fab3a to aosp-master

Bug: None.
Test: NA.
Change-Id: I04ebbaa3c2f939b139f0b8b6f25777383fd42375
Merged-In: Ifcc0c80f1da45c2ff89f7c88da6407ba777473e2

Merge "Revert "RESTRICT AUTOMERGE: Manually merge 885aca1bbaa076b11914b9e4cb0d2a6d0c6fab3a to aosp-master""

Fixed a typo in an OWNERS file.

Test: None
Change-Id: I7f758f20fb2e5bfb9c2ba3193e55080046902bdc

Revert "RESTRICT AUTOMERGE: Manually merge 885aca1bbaa076b11914b9e4cb0d2a6d0c6fab3a to aosp-master"

This reverts commit a738111eeea622a25cb50ed7bcae5cbf0218b137.

Reason for revert: mistakenly used "RESTRICT AUTOMERGE" on the title.

Change-Id: I7ec9917887763912114405ba89eb5f0fac328d97

Merge "Update documentation for SubscriptionManager listener"

Merge "Improved wipe reason information for RecoverySystem"

Implement more location checks

Make the following modifications to APIs and required permissions.
Move the following APIs from checking coarse permission to fine
permission (except for apps targeting pre-Q):
TelephonyManager.getCellLocation()
(deprecated) TelephonyManager.getNeighboringCellInfo()
TelephonyManager.getAllCellInfo()
TelephonyManager.requestCellInfoUpdate()
PhoneStateListener#onCellLocationChanged
PhoneStateListener#onCellInfoChanged

Start checking for fine permission on the following APIs:
(Note -- these will only be logged, not denied until a future CL)
TelephonyManager.requestNetworkScan()
TelephonyScanManager.requestNetworkScan()
NetworkScanCallback#onResults

Start checking for coarse permission on the following APIs:
(Note -- these will only be logged, not denied until a future CL)
TelephonyManager#getAvailableNetworks
TelephonyManager#getServiceStateForSubscriber and #getServiceState
PhoneStateListener#onServiceStateChanged

Bug: 116258458
Test: manual via testapps
Change-Id: I45326ddacbcf538da50ca544d67eed12cf0c0861