Merge changes from topic "Porting-ONSAPIChanges"

* changes:
  Should not turn on validation for data
  Provide network bands in AvailableNetworkInfo

Merge "Update feature flag mapping owners"

Merge "Integrate testNetworkService and Manager with Connectivity stack"

Merge "Fix ServiceState.hashCode() bug."

Fix ServiceState.hashCode() bug.

ServiceState currently overrides hashCode() incorrectly. It hashes an
Array field without using Arrays.hashCode(), meaning that the default
Object.hashCode() implementation is used (which is based on the address
of the Array field). This allows equivalent ServiceState instances to
produce different hashes.

Bug: 128609620
Test: atest CtsTelephonyTestCases:ServiceStateTest
Change-Id: I40a7d0b7ebab6d2a1be47ea63a6d223c4c13700b

Merge "Bugreport Manager service clear binder identity"

Merge changes I39e84219,I8d0e12cb

* changes:
  Bugreporting API: Take ownership of fds.
  Don't manage dumpstate lifecycle from system server

Merge "[KA02.5] Use binder thread and executor to invoke callback"

Merge "Remove SMSDispatcher.sendData() from greylist as it's params changed"

Merge "Initialize the NetworkStack before WiFi"

Merge "API review: Telecom outgoing call Call redirection API"

Merge "Rename some failed causes"

Update feature flag mapping owners

Test: rebuild

Change-Id: I288e039ffaf88ce271ba17bcc26f0e038314b354

Initialize the NetworkStack before WiFi

The NetworkStack needs to be marked as start requested before any client
can use it from the system server, so it knows to queue requests until
it actually comes up.

This fixes a bug introduced in commit:
I93315ad31925bd436e21d1eabb21d5967e6aae60
where a client trying to use the network stack in the system server
before ActivityManager.systemReady() could block and eventually timeout,
resulting in a null INetworkStackConnector.

Test: booted, WiFi working
Bug: 128620654

Change-Id: Ieca2657373eba78195b387fb266fa0424fce8ada

Merge "Avoid IllegalArgumentException when bluetooth is disabled"

Merge "Explain the difference of isValid, isActive, isUsable"

Remove SMSDispatcher.sendData() from greylist as it's params changed

Test: none
Bug: 120403639
Merged-in: Ib70dd739d8067ca2e37f0dbc6c48a2d625935a4b
Change-Id: I0218764fc02d3f6b9f12b3480aba9742a52b8413

Bugreport Manager service clear binder identity

Binder identity of the caller needs to be cleared before making a call
to ensureIsPrimaryUser() and is later restored for the rest of the code
to execute.

Test: Builds
Change-Id: Ida781b633efbea57a9442fb9a8a5f823f9acbe8b
Merged-In: Ida781b633efbea57a9442fb9a8a5f823f9acbe8b
(cherry picked from commit 99766179567464860585feeb669eb01619598a50)

Explain the difference of isValid, isActive, isUsable

Bug: 118675257
Test: compile
Change-Id: Ibec1bf7929991ca0e2910684f5216ab8372d4e95

Merge "Add MANAGE_TEST_NETWORKS permission as @hide"

Bugreporting API: Take ownership of fds.

BUG: 126434607
FIXES: 127649051
Test: manual
Change-Id: I39e8421925c53061b6bc2954dffe3bccb7b3314d

[KA02.5] Use binder thread and executor to invoke callback

Currently, client side of keepalive event handling rely on a
newly created thread, looper, messenger and handler per object.

However, by creating oneway AIDL interface with the executor,
the callbacks can be invoked on the binder thread with user
specified context, which not only greatly simplify the design
but also reduce the cost of current thread modeling.

Bug: 114151147
Bug: 123969871
Test: 1. atest FrameworksNetTests --generate-new-metric 10
      2. atest-deflake.sh

Change-Id: I27504074cd28d5b5eb94a7ec0e97ebaaaaa1ae3d

Merge "Add a whitelist to control packages that can use Bugreporting API"

API review: Telecom outgoing call Call redirection API

Bug: 126360354
Test: Treehugger
Change-Id: Id5b0b4aecb33ba4dc9629cc6bd2bb8836aedc653

Merge "fix the issue that setPreferredNW send wrong RAT to HAL"

Should not turn on validation for data

Should not turn on validation for data switch every time.
Provide result as a callback

Test: verified manually
Bug: 126264455
Merged-In: I5df2311d91fdab34563148b4466163696e52faf9
Change-Id: I5df2311d91fdab34563148b4466163696e52faf9

Merge "Add me to owners for batterystats."

Provide network bands in AvailableNetworkInfo

Provide network bands in AvailableNetworkInfo

Test: verified using CTS and make
Bug: 123292899
Merged-In: I0d0a180eb4da3edf50a94ba621505ad95c7a262c
Change-Id: I0d0a180eb4da3edf50a94ba621505ad95c7a262c

Merge "Should return the number of supported SIM cards"

Merge "Fix issues associated with hold failure"

Merge "Use jarjar rules for framework.jar."

fix the issue that setPreferredNW send wrong RAT to HAL

Bug: 126805258
Test: Manual test to set LTE only mode
Change-Id: I2e88d3bc22ca8d1e626bf381840cc80f832a8b3f

Add me to owners for batterystats.

I do solemnly swear that other powers I get incidentally will be
used carefully.

Test: none
Bug: 123732427
Change-Id: I672f5f10b32646b8beb3e9692ed131019c391593

Merge "DO NOT MERGE Let isLocationEnabledForUser() return true location setting"

Merge "Add alanstokes as owner of DynamicCodeLoggingService."

Merge "Annotate Parcelable CREATOR fields as @NonNull"

Add a whitelist to control packages that can use Bugreporting API

Test: checked SecurityException is thrown for my custom app
Test: whiltelisted my custom app, checked no SecurityException is thrown
Bug: 126541701
Change-Id: Id0b61ccc1adf40bcb455d3b59b640f4b160bdd84
Merged-In: Id0b61ccc1adf40bcb455d3b59b640f4b160bdd84
(cherry picked from commit 07964b49ac81b9955d9ecf5cf60c0bb1fc32c501)

Merge changes from topic "hidl-updater-system"

* changes:
  pm: AndroidHidlUpdater: system only
  pm: PackageBackwardCompatibility -> applyPolicy

Merge "Add Nullable annotation to getDecorView"

Don't manage dumpstate lifecycle from system server

Dumpstate now exits correctly when it's finished. So we don't have to
manage its lifecycle from system server.

BUG: 123571915
Test: manually verified dumpstate service dies
Change-Id: I8d0e12cb607cda74b5cc36e26306ea106da67ba3

Merge "Remove support for event-based handovers."

Merge "Clean up bugreport API."

Merge "Improve documentation around HostnameVerifier"

Add alanstokes as owner of DynamicCodeLoggingService.

As suggested by agampe on I69f415929785845ede8f2d1341f4b70f97b4f480.

Test: None.
Change-Id: Ibaac6f5d96ad9b9a343ba8940345bc7d0c8126d9

Merge "Refactor OWNERS file."

Clean up bugreport API.

Make BugreportManager class final.

Add FloatRange annotation for BugreportManager.onProgress() function's
parameter.

Make @IntDef in BugreportMode hidden.

Bug: 126434607
Test: builds
Change-Id: I95f004106ba97bc9e99859a8b23b96a86b6a43f4
Merged-In: I95f004106ba97bc9e99859a8b23b96a86b6a43f4

Should return the number of supported SIM cards

When multiple SIM is enabled, getSimCount() in TelephonyManager.java
just returns 2 and it is not consistent with getPhoneCount(). For
devices which support more than 2 SIMs, it will cause
ArrayIndexOutOfBoundsException in the getInstance() function of CatService.java
when thress SIMs are present. To fix this issue, we need to return
the phone count in getSimCount() to synchronize both APIs.

Bug: 126292874
Test: check if JE is fixed on a device which supports 3 SIMs
Change-Id: I227f7d05349c1d0cf4c8bfa46c4e3b5f9bd42be9

Merge "continue read file when returned count > 0"

Merge "Support partial connectivity probe result option"

Merge "Call notifyStartupCompleted from reportFullyDrawn"

Call notifyStartupCompleted from reportFullyDrawn

ART will now know when startup is startup is finished.

Test: manual
Bug: 120671223
Bug: 123377072

(cherry-picked from commit 63e14c10b3f7b2c9709b98cb9f128a2a817d4968)

Merged-In: I1134f66792bddc9ab33606fa4c2807b3918d5261
Change-Id: I1134f66792bddc9ab33606fa4c2807b3918d5261

Merge "Rename entitlement callback to adhere to API guidelines."

Merge "Only return one mobile subscription per group in Settings."

Only return one mobile subscription per group in Settings.

Right now getSelectableSubscriptionInfoList returns all subscriptions
that are visible and available. But we should only show one per
subscription group. So updating getSelectableSubscriptionInfoList
to make sure of it.

Bug: 126438122
Test: manual - group Fi primary subscriptions and make sure only
one Fi subscription is shown in Settings UI.
Change-Id: I867296b4f8efbdf2e9885ef34dd45806fa0abbca
Merged-In: I867296b4f8efbdf2e9885ef34dd45806fa0abbca

Merge "Add @NonNull annotations to LE CoC APIs"

Add @NonNull annotations to LE CoC APIs

Added NonNull annotations to the 4 LE Connection-oriented channel API in
BluetoothDevice and BluetoothAdapter.

Bug: 126701988
Bug: 126701989
Test: Compile only
Change-Id: I2d4dc8fbd06e30c782123e01f8481d249e40ee02

Integrate testNetworkService and Manager with Connectivity stack

This change adds TestAPIs for tests to retrive an instance of
ConnectivityManager, allowing it to build test TUN interfaces, as well
as test networks.

This also integrates the TestNetwork types with ConnectivityManager,
creating virtual networks if the network agent is a test agent.

Bug: 72950854
Test: Compiles, CTS tests using this passing correctly
Change-Id: Ic1a04aa66014d1c66a74e65dbace3218437403ae
Merged-In: I741ef9cdf4bd4125d9129af3a030edf32f438e4f

Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"

Merge "Tell the factory it is already serving a request."

Merge "Temporarily disable flaky test."

Merge "Schematize Crypto state system property"

Merge "[KA13]: add unit test for exposed TCP socket keepalive API."

Merge "add NonNull annotation to API"

Add MANAGE_TEST_NETWORKS permission as @hide

This commit adds a new permission for test networks, granted to the
shell permission identity.

Bug: 124519473
Test: In-progress CTS tests
Change-Id: I995b93a66d283b9e37381b616843dd44dbafe319

Temporarily disable flaky test.

Test: None
Bug: 128426024
Change-Id: Id88bf452f41f512703d5831e357d9ba72dc5b911

Merge "Don't return hidden subscription in getActiveSubIdList"

Merge "Don't return hidden subscription in getActiveSubscriptionInfoList."

Merge "Revert^3 "Delay start of bootanimation until after the Runtime APEX is mounted.""

Revert^3 "Delay start of bootanimation until after the Runtime APEX is mounted."

This reverts commit a6bb8916b057bb619aab2aae5cb07e190e97c79d.

Bootanim doesn't need to be delayed because the APEXes from the system partition are now
activated very early in the boot sequence (far before bootanim is started).

See https://android-review.googlesource.com/q/topic:%22apex_earlymount_base%22+(status:open%20OR%20status:merged)

Bug: 125549215
Test: bootanim is started without being delayed
Change-Id: I367b1a5c6bf08dad770cd2bf7f6d67dbf1a813e6

Merge "[API Feedback] Rename DebugEventReporter"

Avoid IllegalArgumentException when bluetooth is disabled

When the corresponding profile service is not available on device,
bluetoothA2dp, bluetoothHidHost and bluetoothPbap will report
IllegalArgumentException when bluetooth is disabled. This change avoid
that exception.

Bug: 127359897
Test: local test and observe there is no IllegalArgumentException when
bluetooth is disabled

Change-Id: I457e5225b8710dd5d02c405ef558892e99d1736f
(cherry picked from commit ee8371007b82ff8c6d26e8ac90c6e3d3cf6bf2a8)

DO NOT MERGE Let isLocationEnabledForUser() return true location setting

* In API 28, LocationManager#isLocationEnabled() and
  LocationManager#isLocationEnabledForUser(UserHandle) always
  return false when a device has neither LocationManager#GPS_PROVIDER
  nor LocationManager#NETWORK_PROVIDER.
* Instead of letting setLocationEnabled() and isLocationEnabled()
  depend on whether device has GPS or NETWORK location provider, this
  CL changes these two APIs to directly use LOCATION_MODE setting to
  store the location enable state
* Hence when LOCATION_PROVIDERS_ALLOWED contains a provider, the provider
  may be used, but it is not guaranteed to exist
* Settings.java has two workarounds that actually convert:
   - putInt(LOCATION_MODE, LOCATION_MODE_HIGH_ACCURACY)
       => putString(LOCATION_PROVIDERS_ALLOWED, "+gps");
       => putString(LOCATION_PROVIDERS_ALLOWED, "+network");
   - getInt(LOCATION_MODE, LOCATION_MODE_HIGH_ACCURACY)
       => getString(LOCATION_PROVIDERS_ALLOWED):
         - "gps,network" - LOCATION_MODE_HIGH_ACCURACY
         - "gps"         - LOCATION_MODE_SENSORS_ONLY
         - "network"     - LOCATION_MODE_BATTERY_SAVING
 - others        - LOCATION_MODE_OFF
* Hence this is NOT a new behavior

Bug: 121040693
Bug: 118242060
Bug: 127359153
Test: CTS tests,
android.bluetooth.cts.BluetoothLeScanTest#testBasicBleScan,
android.bluetooth.cts.BluetoothLeScanTest#testScanFilter,
android.location.cts.LocationManagerTest,
android.location2.cts.LocationManagerTest
Change-Id: I7972d8f97f4ca82c58c29641a081ef73fdcb106c

pm: AndroidHidlUpdater: system only

Backwards compatibility takes a slight performance hit. Only apply this
rule to outdated system apps as they are the only ones that use these
libraries.

Bug: 86259915
Test: boot Pixel 2
Test: boot Pixel 3 and systrace app startup
Test: atest android.content.pm.AndroidHidlUpdaterTest

Change-Id: I2d2e2ca79266a92ba0e4efcf25e31d15daee87a1

pm: PackageBackwardCompatibility -> applyPolicy

This moves PackageBackwardCompatibility from parsePackage to
applyPolicy.

The motivation of this is to expose partition information to these
files. Specifically, to expose whether the app is a pre-installed
(or updated pre-installed app) for certain backwards compatibility
rules that should only apply to these cases.

Bug: 86259915
Test: boot Pixel 2
Test: Pixel 3: atest -p cts/tests/signature
- all passed except for CtsSystemApiAnnotationTestCases test which
  failed (known issue b/124210145)
Change-Id: Ic824f06489781233dc98bf3dc0182681d32f7142

Don't return hidden subscription in getActiveSubIdList

Now getActiveSubscriptionInfoList will not return hidden
subscriptions. And getActiveSubIdList should be consistent
with it. In the mean time, internal SubscriptionController
API should have a way to specify if caller wants hidden
subscriptions or not.

Bug: 124690005
Test: manual
Change-Id: I46ccae88167c2988d71a6e07da2937bf9644a1f0
Merged-In: I46ccae88167c2988d71a6e07da2937bf9644a1f0

Merge "Correct registerNativeAllocation type"

Merge "Change summary text of Wi-Fi calling"

Remove support for event-based handovers.

Public APIs have been available for 2 releases; removing support
for the EXTRA/EVENT-based handovers.

Test: Manual
Bug: 110846969
Change-Id: I4747a509a9fd4d41c161febb78fbb7e4e11699f2

Don't return hidden subscription in getActiveSubscriptionInfoList.

Bug: 121396526
Test: manual
Change-Id: I3a9f3a9a2628a8e2ad1c401f6fe5e4c5866a0047
Merged-In: I3a9f3a9a2628a8e2ad1c401f6fe5e4c5866a0047

Rename entitlement callback to adhere to API guidelines.

The method for OnTetheringEntitlementResultListener needs to be
onTetheringEntitlementResult in order to meet our API guidelines:

"Use Listener when there is a single callback method and there will
never be any others. It should be an interface whose name is the same
as the callback method plus Listener."

Bug: 126392011
Change-Id: I9a883765541ae853c4eb3bd52c9d93a7af16e661
Test: Manual build

Merge "Add carrier's option to cancel CS reject cause notification"

Add carrier's option to cancel CS reject cause notification

Some carrier still want to cancel CS reject notification automatically
when it gets service. So, we need a way to set carrier's option for it.

Bug: 79889307
Test: Manual
Change-Id: If4c696e8116094d0cb555fe87c92243af1f34cec
Merged-In: If4c696e8116094d0cb555fe87c92243af1f34cec

Annotate Parcelable CREATOR fields as @NonNull

Bug: 126726802
Test: Builds and existing tests pass
Change-Id: I243073dc559e93fa6302d6b69468088ca5960ab5

Improve documentation around HostnameVerifier

Explain usage of HostnameVerifier / HttpsURLConnection.

Bug: 126851476
Test: make docs / visual inspection
Merged-In: I6bc86f8e7efc47521316d41e07ef546ea995061c
Change-Id: I6bc86f8e7efc47521316d41e07ef546ea995061c
(cherry picked from commit 9f5e6f22bdc086ee0396f01556e3367c621737c0)

Tell the factory it is already serving a request.

This is a cherry-pick of ag/607226 that has been rebased on
top of four years of changes and with comments addressed.

Gives each factory a serial number and propagates it to every
NetworkAgent so when a score comes back indicating a request is
being handled the factory can account for it properly.

Without this, a new request that's already handled by a network
offered by a factory will not cause an increment of the factorys
ref count. Concretely this results in issues like the RAT icon
not being displayed in spite of the network actually being up
and usable.

This will be ported to AOSP as soon as possible, but immediately
some master-only WiFi tests need to be adjusted with this change
which would not let me submit to AOSP.

Bug: 18637384
Bug: 29030667
Test: manual
Test: atest frameworks/opt/telephony/tests/telephonytests
Test: atest frameworks-net
Test: atest CtsNetTestCases CtsHostsideNetworkTests
Change-Id: I597ac588f76dd507512ff02868fd1310b7e63f7e
Merged-In: I597ac588f76dd507512ff02868fd1310b7e63f7e

Refactor OWNERS file.

This makes it easier to read the per-file exeptions and to make
further changes.

Specifically:
- Collapse multiple lines so we only have one per-file line per file.
- Remove per-file owners who are already global owners of all files here.

See https://gerrit.googlesource.com/plugins/find-owners/+/master/src/main/resources/Documentation/syntax.md.

This reverts many of the changes in
b662614ca624c3d79827470fafb522b0d9b7f48f but I believe does not
actually change the effect of it at all.

Test: None.
Change-Id: I1bef6e806c17344c57377a8ab28109fa40e9ecf2

Merge "[CS] Support "instant failure" from factories"

Support partial connectivity probe result option

Bug: 124629963
Test: atest NetworkStackTests
Change-Id: I1c22bad6ba94d23892e8907d6e7b28d074922339

Merge "Add product apk support from libnativeloader"

[KA13]: add unit test for exposed TCP socket keepalive API.

Bug: 123987272
Test: atest FrameworksNetTests
Change-Id: I543d8f00ae539b706592a149981e3e8992999284

Merge "Check if network has partial connectivity"

Check if network has partial connectivity

In some networks, network validation may only get success
result for http probe but fail result for https probe.
For this kind of network, it may still work at some websites
or apps, but user didn't know about that. In order to fix this
issue, we will check if network has partial connectivity and
notify user to make a choice if they want to use this partial
connectivity or not.

Bug: 113450764
Test: 1. Build pass.
      2. Fake partial connectivity case for testing.
      3. atest FrameworksNetTests
      4. atest NetworkStackTests

Change-Id: I69ed00ac4850904ff708c9fef22e148879a10e92

DO NOT MERGE - Merge PPRL.190305.001 into master

Bug: 127812889
Change-Id: I7c5ada5c5afe38848be733616cbb1568440ab1c5

Merge "Modify API return type and add callback."

Merge "Move NetworkStack start to AM.systemReady"

Merge "Update nullability of api"

Merge "Improve keepalive offload documentation and error handling"

Move NetworkStack start to AM.systemReady

In practice the NetworkStack was not started after
ActivityManager#systemReady, since ActivityManager was only scheduling
the start (mPendingStarts).
Requesting the start too early causes issues for the PackageManager to
install updates/rollbacks.

Test: flashed AOSP and internal builds, WiFi working
Bug: 128331883
Change-Id: I93315ad31925bd436e21d1eabb21d5967e6aae60

Rename some failed causes

Rename failed cause to avoid using min/max in the word.

Bug: 126699870
Test: Build pass.
Change-Id: I7fb7b0a2de23e75e873ef3c023738d0e15f52834

Fix issues associated with hold failure

Add new connection event to indicate hold failure

Fixes: 128275255
Test: manual
Change-Id: I2678cda12d7703eba37d3feca3a8d98b21c050b1