Refactor DhcpErrorEvent error codes

Addresses review comments on aosp/930843. The new format is more
concise.

Resulting constant values are unchanged as demonstrated by the absence
of change in the API file.

Test: m
Change-Id: I2e086c0411bce1d5b528de50b59dcf51e390681f

Merge "Modify SettingsLib for partial connectivity"

Modify SettingsLib for partial connectivity

Add partial connectivity related settings in SettingsLib.

Bug: 113450764
Bug: 128489091
Test: 1. Build pass.
      2. Fake partial connectivity case for testing.

Change-Id: I0733ecc9070e6c4281fdff40f432605d139355ec
Merged-In: I4791abe9351c62d9549ada383aa4cc8aa34ddaf2

Merge changes from topic "iwlan_api_review"

* changes:
  IWLAN API review changes
  Unhide methods required for Parcelable class
  Added annotations to the API

Merge "Fix build due to no explanatory comment."

Merge changes I2e8c2ad9,Ic4f4c7d2

* changes:
  Move the networkstack AIDL interfaces to services/net.
  Remove the framework dependency on networkstack-aidl-framework.

Merge "Add "from" extra for ACTION_TOGGLE_SUBSCRIPTION_PRIVILEGED"

Merge "Fix the assumption in GetEuiccProfileInfoListResult."

Merge "Expose CellInfoTdscdma"

Fix build due to no explanatory comment.

Test: manual build
Change-Id: Ibfb3a114c74d673eb94dd2fa1fc9b9918283e9db

Fix the assumption in GetEuiccProfileInfoListResult.

Under error cases, LPA can return a GetEuiccProfileInfoListResult
with either null profiles or empty profiles.

Bug: 127376266
Test: build
Change-Id: Id29c8f00183660d2195c58c96eb4154454623eb6
Merged-In: Id29c8f00183660d2195c58c96eb4154454623eb6

Merge "Deprecate SystemApi that was accidently made public in P"

IWLAN API review changes

Made several API changes per API council
suggestions.

Test: Manual + unit tests
Bug: 127812471
Bug: 128607082

Merged-In: I0f93c07207b10b566e9f858361e55b18dc0881a0
Change-Id: I0f93c07207b10b566e9f858361e55b18dc0881a0
(cherry picked from commit 806b64228db64e25bd04ab5da607094932561b36)

Unhide methods required for Parcelable class

Those methods were already public in the base class.

Test: Build
FIXES: 126702898

Merged-In: Ibce8f9bf5d1a3ebf9225fa46422dc3c78e29b1f6
Change-Id: Ibce8f9bf5d1a3ebf9225fa46422dc3c78e29b1f6
(cherry picked from commit 654bb21f703f943d24586323c9eb822fc06a0857)

Added annotations to the API

Added @NonNull and @Nullable to APIs.

Test: Build
Bug: 126702898

Merged-In: I697298837858829bb0851aba389159701fa0fc5e
Change-Id: I697298837858829bb0851aba389159701fa0fc5e
(cherry picked from commit 1eb3f0cdaed5a7156e9dcb11223f7f4ac82f1360)

Merge "Modified MultiSim APIs"

Add "from" extra for ACTION_TOGGLE_SUBSCRIPTION_PRIVILEGED

Bug: 126242076
Test: Test on phone to ensure the extra is received by LPA
Change-Id: I1367fa988dbc71eb30631ebf9903309eba3ef96d
Merged-In: I1367fa988dbc71eb30631ebf9903309eba3ef96d

Merge "Require the ANDROID_TZDATA_ROOT env var"

Merge "Update documentation"

Merge "Fix Automated API Review issues."

Merge "Temporarily disable flaky test."

Move the networkstack AIDL interfaces to services/net.

Nothing in the framework uses these, so there is no point in
having the build be in frameworks/base/Android.bp.

Fix: 128804404
Test: builds, boots
Test: atest FrameworksNetTests
Change-Id: I2e8c2ad9ea8bfcd3668a83e4c044de10389a8a22

Remove the framework dependency on networkstack-aidl-framework.

This dependency is unused because the only file it contains is
only used by services.net, which already depends on it via the
networkstack-aidl-interfaces rule.

In its own change because it conflicts in internal master.

Test: m
Bug: 128804404
Change-Id: Ic4f4c7d2f7de06f75344946345684f26f1e05932
Merged-In: Ic4f4c7d2f7de06f75344946345684f26f1e05932

Merge "Fix Automated API Review issues."

Merge "Define DhcpErrorEvent constants at compile time"

Merge "Replace TcpSocketInfo with similar structure"

Temporarily disable flaky test.

Test: None
Bug: 128426024
Change-Id: I6fa12e50decbafe9d934030865c5a8f20d42b86f

Merge "Fix ODR issue in viewcompiler"

Fix Automated API Review issues.

These API's argument/return value must be marked either @NonNull
or @Nullable.

Bug: 126701148
Bug: 126699090
Bug: 126701058
Bug: 126700772
Bug: 126699941
Bug: 126701299
Bug: 126700007
Bug: 126700900
Test: atest FrameworksNetTests
Change-Id: Id030a9f1116178b96aa3d4614b10969a537b2fc4

Merge "Restrict scope of switch Repair Mode"

Modified MultiSim APIs

Capitalize S in Multisim.
isMultiSimSupported should return three states, to describe the three possible cases
(hardware not supported, hardware supported but carrier restricted, available)

Bug: 128524079
Test: compilation
Change-Id: Icbd36f714c577a965d39336d54d7224c8c2c7cd2
Merged-In: Icbd36f714c577a965d39336d54d7224c8c2c7cd2

Merge "Cleanup NetworkManagementService for Netd commnads binder migrartion"

Fix Automated API Review issues.

1. Some API's argument/return value must be marked either
@NonNull or @Nullable.
2. Change some system APIs to public APIs.
3. Modify the method name

Bug: 126700123
Bug: 126702339
Bug: 126699682
Bug: 118296575
Bug: 126699216
Bug: 126699675
Bug: 126699429
Bug: 126699193
Bug: 123586045
Test: atest FrameworksNetTests
Change-Id: Iaa2832cdcf83758ed0fec81b954a0c63bc5a7bf6

Fix ODR issue in viewcompiler

libz is already linked as a static library through
libdexfile_static_defaults, don't also link it as a shared library.
Fixes:
=================================================================
==43189==ERROR: AddressSanitizer: odr-violation (0x55d1ed723a60):
[1] size=48 'inflate_copyright' external/zlib/src/inftrees.c:11:12
[2] size=48 'inflate_copyright' external/zlib/src/inftrees.c:11:12
These globals were registered at these points:
[1]:
 #0 0x55d1ed769a0e (/buildbot/src/android/master/out/soong/host/linux-x86/bin/viewcompiler+0xf3a0e)
 #1 0x55d1ed991b1b (/buildbot/src/android/master/out/soong/host/linux-x86/bin/viewcompiler+0x31bb1b)

[2]:
 #0 0x55d1ed769a0e (/buildbot/src/android/master/out/soong/host/linux-x86/bin/viewcompiler+0xf3a0e)
 #1 0x7f5dff20a09b (/buildbot/src/android/master/out/soong/host/linux-x86/bin/../lib64/libz-host.so+0x3e09b)

Test: m SANITIZE_HOST=address viewcompiler && viewcompiler
Change-Id: I4ebdf7ef4ec1641488fbc25275dbf6aa0cdbe2eb

Merge "Improve partial connectivity"

Merge "Document type of euicc privileged action extras."

Merge "Document permissions for euicc privileged actions."

Expose CellInfoTdscdma

Expose CellInfoTdscdma and CellSignalStrengthTdscdma
which are both returned by existing public APIs.
This corrects an oversight whereby the values returned
by the public API could not be used by callers.

Existing CTS tests already validate these structures as
though they were public (which was the intention).

Note, this was also requested as feedback in the public
beta.

Bug: 128880490
Test: cts - atest CellInfoTest; atest SignalStrengthTest
Merged-In: Ia75ea9af52796729e26b85ca04fd97b6c9b80f8e
Change-Id: Ia75ea9af52796729e26b85ca04fd97b6c9b80f8e

Merge "Bluetooth: Add event_value to BluetoothClassicPairingEventReported"

Merge "Support starting gsid lazily"

Merge "Dist aapt2 protos"

Merge "API review feedback changes"

Document type of euicc privileged action extras.

Bug: 128317614
Test: Test on phone
Change-Id: Ib7978807376c7987d3aa8a7e3d1dc4bc6c814c6a
Merged-In: Ib7978807376c7987d3aa8a7e3d1dc4bc6c814c6a

Merge "Add Prefix per Carrier requirement"

Document permissions for euicc privileged actions.

Bug: 128317064
Test: Test on phone
Change-Id: I285597e48e97f1b4ffe4b4eb4ac68010905f4e1f
Merged-In: I285597e48e97f1b4ffe4b4eb4ac68010905f4e1f

Add Prefix per Carrier requirement

Some carrier requires if users dial an emergency number address with
some prefix, the combination of the prefix and the address is also a
valid emergency number to dial.

Bug: 122429213
Test: Treehugger
Change-Id: Ib9b96bb0a210aba64a8652984d22f3564813fa9d
Merged-In: Ib9b96bb0a210aba64a8652984d22f3564813fa9d
(cherry picked from commit 69f48c5e906512bf9ac56b315daa6d4dfdfb7600)

API review feedback changes

Made the follwing changes based on API council review
feedback

1. Made the class and its Builder class final
2. Removed methods that should not be in the builder
3. Changed the return type of available services
   from array to list.

Test: Unit tests
Bug 127657991

Merged-In: I3c2d406fed3ddaf62dee193c7657caf8c5bc13d2
Change-Id: I3c2d406fed3ddaf62dee193c7657caf8c5bc13d2
(cherry picked from commit 20d71403b8a1950bb44ee183138dc7a83c673472)

Update documentation

Fixes: 128919950
Test: no change to behavior
Change-Id: If303386de0ce8bdc215eb42ed8ad313cbb91ca44

Merge "Make ImsException final"

Merge "Remove telephony-common classes."

Merge "[API feedback] fixed API for NR"

[API feedback] fixed API for NR

This CL included the follow change:
1. Change getPreferredNetworkTypeBitmap to
getPreferredNetworkTypeBitmask [b/126552419]
2. Change setPreferredNetworkTypeBitmap to
setPreferredNetworkTypeBitmask [b/126552419]
3. Add @IntRange annonation to CellIdentityNr.getNrarfcn() [b/126552419]

4. In javadoc, explain the acronym NR to say: (New Radio 5G)
[b/126552559]

5. Mark CellInfoNr.getCellSignalStrength and CellInfoNr.getCellIdentity
    as @NonNull [b/126699572]

6. Mark CellIdentityNr.getMncString() and CellIdentityNr.getMccString()
    as @Nullable [b/126702744]

Bug: 126552419
Bug: 126552559
Bug: 126699572
Bug: 126702744
Test: build
Merged-In: I133075a3688c3fa7550c8a2f43bc13d8b25999b2
Change-Id: I133075a3688c3fa7550c8a2f43bc13d8b25999b2

Deprecate SystemApi that was accidently made public in P

ImsFeature.Capabilities was accidently made public in P
due to a reference to it in another @SystemApi method.
Marking as @Deprecated in Q and then moving back to @hide
in R.

Bug: 115639348
Test: manual
Change-Id: Iab0d0c9227798b9f4e153d1c5a0a4e16353ee87c

Merge "pm: SharedLibraryInfo: toString closing bracket"

Merge "improve mism support for phonestatelistner"

improve mism support for phonestatelistner

1. throw exception if reuse the same listner for different sub
2. subId should come from TM instance not phonestateListener.
3. refactor settings to apply phonestatelistner with TM instances

Bug: 117555407
Test: Manual
Change-Id: I1af07798d7982575cccc27462af179182c033409
(cherry picked from commit 0ebc9b67e96d0c4a625837ce7a32dbea4afa44ab)
Merged-in: I1af07798d7982575cccc27462af179182c033409

Replace TcpSocketInfo with similar structure

Replace TcpSocketInfo with TcpKeepalivePacketDataParcelable
because their structures are very similar.

bug: 128882321
Test: -build, flash, boot
      -FrameworksNetTests

Change-Id: Iafb4031a64ba4775a495c156e2c997d890c6b261

Restrict scope of switch Repair Mode

Bug: 123969339
Test: -boot, flash
      -atest FrameworksNetTests

Change-Id: Ie0fb685be5f7a2d06544065d67c605d87a19ff2f

Cleanup NetworkManagementService for Netd commnads binder migrartion

remove dead code used for netd UNIX sokcet

Bug: 65862741
Test: atest NetworkManagementServiceTest FrameworksNetTests
Change-Id: I5db61cb86325b71ac8a8fce7b15dbf569abc5935

Bluetooth: Add event_value to BluetoothClassicPairingEventReported

* Add an event_value field to log status value such as
 - encryption enabled state
 - simple pairing mode
 - secure connection host support
 - delete all flag

Bug: 128966402
Test: make
Change-Id: I512b7a1a7fba73ea30bca028566d43d353c56613

Improve partial connectivity

Improve the design and fix some nits.

Bug: 113450764
Test: 1. Build pass
      2. atest FrameworksNetTests
      3. atest NetworkStackTests
      4. Change captive_portal_https_url to https://invalid.com
      to simulate partial connectivity.
Change-Id: Ia56645841d00d2ed8406cfeacb86a4a27fd58650

Merge changes from topic "framework-no-networkstack-aidl"

* changes:
  Remove the framework dependency on netd_aidl_parcelables-java.
  Move TcpKeepalivePacketData out of the framework.
  Stop using netd parcelables in the framework.

Define DhcpErrorEvent constants at compile time

Addresses API review comments.

Bug: 126700007
Test: m
Change-Id: I8c32a0e8bcf2d63eaf33c9ac884afd4fdbe0fdd3

Merge "Fix the problem of CaptivePortalLogin app cannot show SSL error page"

Merge "Renamed NetworkRegistrationState to NetworkRegistrationInfo"

Merge "Add documentation for CaptivePortal APP_RETURN_*"

Merge "Convert tests/**/Android.mk files to Android.bp, part II"

Merge "Add Parcelable implementation for ApfCapabilities"

Remove the framework dependency on netd_aidl_parcelables-java.

In its own change because it conflicts in internal master.

Test: m
Bug: 128804404
Change-Id: I74531ef11d504b232d4f377532d5e0812dbc88db
Merged-In: I74531ef11d504b232d4f377532d5e0812dbc88db

Move TcpKeepalivePacketData out of the framework.

This class is not used in framework.jar, only in services.jar.
Move it out of the framework so framework.jar can stop depending
on the NetworkStack stable AIDL interfaces.

This should really under com.android.server as well, but that's
a cross-project change to be done in another CL.

Test: m
Bug: 128804404
Change-Id: Ib2805f7fcc516a5f6989c252365d1c95b8042703
Merged-In: Ib2805f7fcc516a5f6989c252365d1c95b8042703

Stop using netd parcelables in the framework.

The only actual users of these classes are in services.jar, not
in frameworks.jar. The only reason the framework depends on them
is that the code that converts to and from stable parcelables is
currently in the framework. Move that code to services and cut
the dependency.

These classes aren't used in the networkstack app so they don't
need to be in shared. They also can't be in shared because the
classes are not in the SDK. So put the conversion functions
directly inside their only user (NetworkManagementService).

Also remove the jarjar rules that rename the classes for use by
the NetworkStack app.

This does not actually remove the dependency from the build file,
that will be done in a future CL.

Bug: 128804404
Test: builds, boots
Test: atest FrameworksNetTests android.net.cts.ConnectivityManagerTest HostsideVpnTests
Change-Id: I027d50ba56091f5558f45e6e08f32e5912b2a82a
Merged-In: I027d50ba56091f5558f45e6e08f32e5912b2a82a

Merge "Add class comment to avoid extending ServiceState class."

Make ImsException final

Bug: 128902253
Test: atest FrameworksTelephonyTests
Change-Id: I88058f8382978163718de8194249fad87ae52d10

Merge "Zygote: Load system server code early"

Merge "Add uid information in PackageListObserver"

Merge "Notify caller when unlockUser() bails out"

Zygote: Load system server code early

In an effort to allow loading integrity-checked artifacts from
the dalvik-cache, attempt to create and cache the system server's
classloader early, while still being in the system_server_startup
selinux domain.

The advantage of this approach is that allowances for loading
from the cache are restricted to startup.

Bug: 128688902
Test: m
Test: Device boots, picks up /system artifacts
Test: Device boots, picks up integrity-checked /data artifacts
Merged-In: If4a75fa106db09f1bd666d6d8df7ac3ac3e35a8c
Change-Id: If4a75fa106db09f1bd666d6d8df7ac3ac3e35a8c

Support starting gsid lazily

The aosp/914714 made the gsid to start when needed.
The DynamicAndroidService must support this behavior as well.

Bug:126622385
Test: adb shell am start-activity \
-n com.android.dynandroid/com.android.dynandroid.VerificationActivity  \
-a android.content.action.START_INSTALL    \
--es KEY_SYSTEM_URL file:///storage/emulated/0/Download/system.raw.gz  \
--el KEY_SYSTEM_SIZE $(du -b system.raw)  \
--el KEY_USERDATA_SIZE 8589934592

Change-Id: Ifb9d642d183ed3e7181ed643f9415850178ef58d

Merge "Add unit tests related to data accounting for VPNs with one underlying network."

Merge "Extend light-greylist of some TelephonyManager APIs until Q."

Merge "Revert "Rename some failed causes""

Merge "Mark product apks as unbundled"

Add class comment to avoid extending ServiceState class.

ServiceState implements Parcelable and all 'Parcelables' are required to
be final, per the Android API Guidelines (go/android-api-guidelines).
This won't be changed for historical reasons, but clients should avoid
extending this class.

Bug: 128865231
Test: non-functional change that won't impact tests.
atest CtsTelephonyTestCases:ServiceStateTest
Change-Id: Ib369f1449a3e085f404fff1c706fda0b76d42890

Renamed NetworkRegistrationState to NetworkRegistrationInfo

API review changes. The class covers more information other than
registration state.

Test: Build
Bug: 127657991
Merged-In: I8af36b388182ab08257b64c37078f156d4821709
Change-Id: I8af36b388182ab08257b64c37078f156d4821709
(cherry picked from commit e27d3fde88d37332cc9970c79b25060c003e9f17)

Merge "Move the IpMemoryStore to the network stack."

Fix the problem of CaptivePortalLogin app cannot show SSL error page

This commit is trying to fix the limitation of older design:
1.There is no guarantee that onReceivedSslError() will be called
after onPageStarted(). So, onReceivedSslError() may compare the
host of SSL error url with mHostName which is unset.
2.Some WIFI APs may redirect to the probing url(mUrl) along with
special parameter first then redirect to the real log-in page,
it will make onPageStarted get unexpected hostname.
Above 2 situations will make captive portal app cannot show SSL
error page and result in captive portal app is blank.

The new design is trying to get the main frame url and compare
it with SSL error url. If the SSL error url is equal to main
frame url, then show the SSL error page.

Bug: 123208856
Bug: 122991421
Test: build pass
Change-Id: I27be7258436ccb833ff75ebcfed3af811467fa47

Add documentation for CaptivePortal APP_RETURN_*

This fixes API review requests.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 126883311
Change-Id: I0632a98675e65c5e722e81a2c1e0cf1403b54395

Merge "Fix API for SocketUtils"

Merge "Replace Handler with Executor for async DNS API"

Merge "Refactor answer callback for async DNS query JAVA API"

Dist aapt2 protos

Create aapt2-protos.zip containing the aapt2 proto interface and
dist it for SDK builds.

Bug: 128878015
Test: m PRODUCT-sdk_phone_armv7-win_sdk sdk_repo dist
Change-Id: I13c774c8a2262464dcc4731c999497f6d0dc9095

Revert "Rename some failed causes"

This reverts commit 1c405b73b096c5d20c610fb97012f7576bc0cf4e.

Reason for revert: <INSERT REASONING HERE>

Change-Id: I4af7c87d8a8a988c2f1c3dc6662027c6fcd34088

Merge "Fixing engineGetCertificateChain exception"

Merge "Remove deprecated entitlement API"

Add uid information in PackageListObserver

The uid information of a removed package cannot be retrieved by the
packageName anymore once it is removed. So it would be useful to provide
the uid of removed package in the onPackageAdded and onPackageRemoved
method of the PackageListObserver. This modification helps simplify the
design in PermissionMonitor.

Bug: 125396053
Test: dumpsys netd trafficcontroller

Exempt-From-Owner-Approval: Already merged in internal master
Merged-In: I2bd4bdf924687960a4fa3a47235bae68d885e445
Change-Id: I2bd4bdf924687960a4fa3a47235bae68d885e445

Merge "Address API review comments for VPN changes."

pm: SharedLibraryInfo: toString closing bracket

before:
SharedLibrary[...

now:
SharedLibrary{...}

Switched from square to curly braces because curly braces appear to be
the standard in this directory.

Bug: N/A
Test: manual
Change-Id: I02c63dd594fde68e645348cb1227f8594e2a7d3e

Merge "MediaHTTPConnection: move connection states into an inner class"

Merge "Emergency API review"

Merge "Add 5G icon to System UI"