Merge "Expose UIDs for NetworkStack permission checks"

Merge "Remove InterfaceController dependency on NMS"

Merge changes Ieef54d84,Ica3e88c2

* changes:
  Remove NetworkStack dependencies on hidden members
  Expose ServiceSpecificException as @SystemApi.

Merge "Add SystemApi utility for raw sockets."

Expose UIDs for NetworkStack permission checks

The UIDs are unlikely to change and added as public API.

Test: m
Bug: 112869080
Change-Id: I5477093a4c8593dd9ada7587b86c65352e628c34

Remove InterfaceController dependency on NMS

Test: atest FrameworksNetTests NetworkStackTests
Bug: 112869080
Change-Id: Ib3773068b087f58f4ac3394291cda132b00b2dcc

Merge "Deprecate JapaneseCalendar.CURRENT_ERA"

Merge "Add LinkProps and NetworkCaps methods to SystemApi"

Merge "apilint: correctly parse enum_constant"

apilint: correctly parse enum_constant

Change-Id: I1a243caa3a01837ff989d21926478f20d70372ae
Fixes: 120132045
Test: python apilint_test.py
(cherry picked from commit 373df11baab1fbbd9f6986e38a1a929abc901d63)

Merge changes I38145a51,Iac1fdabc,I90f18181,I76b979dd,Id2e17923, ...

* changes:
  ApiLint: Add operator keyword and property parsing
  ApiLint: Add Kotlin-style type support
  ApiLint: Performance improvements for verification
  ApiLint: Performance improvements for 2.0 signature format parsing
  ApiLint: Allow parsing 2.0 API signature files
  apilint: Fix API lint issues 2/2
  apilint: Fix API lint issues
  API Lint: Add support for base current.txt
  Guide towards Context.createPackageContextAsUser().
  Moar lint rulez!
  Update lint rules for graphics/text package layering
  Handle new current.txt format.
  Extend lint script to emit API statistics.

ApiLint: Add operator keyword and property parsing

Also fix up some issues with expression parsing, type use annotations, etc.

Test: python tools/apilint/apilint_test.py
Change-Id: I38145a51470ce6c3e5813a546d681489fd87fc19
(cherry picked from commit 403c8e35d8e7cc0f81a0a2c42d038c47e1b2703f)

ApiLint: Add Kotlin-style type support

Test: tools/apilint/apilint_sha.sh HEAD && python tools/apilint/apilint_test.py
Change-Id: Iac1fdabcbeffe57c8288d73b2359e8ce0b2bc3eb
(cherry picked from commit 7884d6b9090c586ac0d72abe0e6efab191a143a7)

ApiLint: Performance improvements for verification

Test: tools/apilint/apilint_sha.sh HEAD
Test: python tools/apilint/apilint_test.py
Change-Id: I90f18181cd0d3c43f176d7c9d1b198f6f5172390
(cherry picked from commit 7f8886a9933abbb6f1fdd97d920185b80d439339)

ApiLint: Performance improvements for 2.0 signature format parsing

Test: tools/apilint/apilint_sha.sh HEAD
Test: python tools/apilint/apilint_test.py
Change-Id: I76b979dd81702afce6468ac377230e589a25e08f
(cherry picked from commit a30d062775da3812589a6ce3e49be9d697ef0d6d)

ApiLint: Allow parsing 2.0 API signature files

Test: tools/apilint/apilint_sha.sh HEAD
Test: python tools/apilint/apilint_test.py
Change-Id: Id2e1792392b3626746f1ec99f481d0cb27e523a1
(cherry picked from commit d170961b02bf67189ddd14f358c9f263009df786)

apilint: Fix API lint issues 2/2

Fixes false positives that occur when a class in current.txt is faulty, and an
entry for that class is then added to system-current.txt.

This was so because when parsing the previous revison's system-current.txt, we
did not know about the class and thus didn't look for it in current.txt, and
thus never recorded that the error is preexisting.

To avoid that, we track all classes in system-current.txt with a matching entry
in current.txt in the current revision, and later use that to look up all classes we
may have missed when examining the previous revision.

Test: python tools/apilint/apilint_test.py
Change-Id: Ibe09f1159e351e56b35b8816ce0db760de4ef791
(cherry picked from commit 61e3730bc07e04181a01760d2eb1db834a854683)

apilint: Fix API lint issues

Fixes a bug where only the name instead of the fully qualified name was
considered when looking for a class, which lead to faulty results for inner
classes.

Test: python tools/apilint/apilint_test.py
Change-Id: Ib015669ed3faef21d2bdd16f1e27bc55c8669d70
(cherry picked from commit 2c5cacfd36128f43f5fab4f0665acf69ac049a44)

API Lint: Add support for base current.txt

Allows specifying a base current.txt and previous.txt file when linting
system-current.txt and test-current.txt to avoid false positive error
messages due to public API members not being duplicated in the respective
non-public APIs

Test: python tools/apilint/apilint.py --base-current=api/current.txt api/system-current.txt
Change-Id: I306a99b1423584ef3fcdc9272a83cb5eacc37227
(cherry picked from commit 7690d0d4eea0ffa429351b0b1caa34cdb3e0d37f)

Guide towards Context.createPackageContextAsUser().

It's a better alternative that should be used instead of adding
new "ForUser" or "AsUser" methods.

Bug: 115654727
Test: manual
Change-Id: I8742c2ef42d743ef69f8f7a91378f498fdc81e43
(cherry picked from commit 86445841ac90e04941dbc8dad34f2a893a2e0f8b)

Moar lint rulez!

Bug: 37534642, 116675691, 116798271, 72059458, 111790177
Test: manual
Change-Id: Ib079ae580a827f225be08f90dbdddeee7d341c48
(cherry picked from commit daac37f229f84ed844adcf1ffd3432b235524d9b)

Update lint rules for graphics/text package layering

Test: N/A
Bug: 77347886
Change-Id: I907d99b4a1dee0c69b914e55a033d3d9c4eebe56
(cherry picked from commit e23aeb802409cb337ad66454049c9e3b6af0db9a)

Handle new current.txt format.

We're starting to see "@interface" show up, so handle them like any
other interface.  We're also seeing more details argument lists
with names and annotations; ignore them for now, since all our
existing lint checks work on the "real" data type.

Verified that it handles new support library current.txt files
without causing any regressions against existing framework
current.txt files.

Test: manual inspection
Bug: 111555356
Change-Id: Id11c3561edd317e4ba1a9b43993fd96d8243e00d
(cherry picked from commit bd2611916990b0d18a36483060365207fdd94c13)

Extend lint script to emit API statistics.

Bug: 77588754
Test: manual
Change-Id: I240dba5fae1a8635a4265a1af903517f00dec54c
(cherry picked from commit daa7cf3a365c24e3a81d0df178c188b12e6d3a91)

Remove NetworkStack dependencies on hidden members

 - DhcpClient only shares its handler with IpClient, and NetworkMonitor
   has its own handler: remove Protocol.BASE_DHCP,
   Protocol.BASE_NETWORK_MONITOR
 - Remove dependency on Network.netid in NetworkMonitor
 - Remove dependency on Sets.newArraySet in DhcpServingParams
 - Remove dependency on formatDuration() in DhcpClient
 - Replace isMetered() with hasCapability() in NetworkMonitor
 - Use WifiManager.isScanAlwaysAvailable instead of reading setting

Test: atest FrameworksNetTests NetworkStackTests
Bug: 112869080
Change-Id: Ieef54d847ddc081fb33cbad0b050b06d2e52548e

Expose ServiceSpecificException as @SystemApi.

This class is useful as a system API because:

- It is one of the few exceptions that is natively Parcelable.
- It is directly supported by native code without using JNI,
  using Status::fromServiceSpecificError.
- Unlike other natively parcelable exceptions, it contains both
  an error message and an error code.

Test: m
Bug: 112869080
Change-Id: Ica3e88c2b6877e429a61b053d75f69b9f19fc74a

Add SystemApi utility for raw sockets.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 112869080
Change-Id: Id228bf3317b6933314174571697ee256b2f18452

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

Add LinkProps and NetworkCaps methods to SystemApi

Added methods are used by the NetworkStack.

Test: atest FrameworksNetTests NetworkStackTests
Bug: 112869080
Change-Id: I8822d43c1a4981ca39289525685b3f975fb0e8d5

Merge "Add NetworkStack metrics to system API"

Merge "Fix lint errors for moving IpClient to NetworkStack"

Merge "Properly expect all callbacks in testStackedLinkProperties."

Fix lint errors for moving IpClient to NetworkStack

Test: checklint at upload, m
Bug: 112869080
Change-Id: I82fb9c9037cc0a9d3c80958d64552ff1ef0a90ab

Merge "Connect IRadioConfig 1.1 HAL interfaces with RadioConfig.java"

Merge "Add NetworkMonitor constants to SystemApi"

Merge "Add startActivityAsUser to SystemApi"

Properly expect all callbacks in testStackedLinkProperties.

Currently testStackedLinkProperties accounts for some of the
NetworkCallbacks that are fired, but not all of them. The test
passes because it only ever expects LinkProperties changed
callbacks (and not, e.g., lost callbacks), because it uses
waitForIdle(), and because at the end, when the callback is
unregistered, it does not call assertNoCallback().

Add the missing callback expectation and expect a LOST callback
at the end. Also add an assertNoCallback, even though it's not
strictly necessary. Remove all the waitForIdle statements, which
are no longer necessary because every assertion happens after
waiting for a callback.

Making the expectations more precise in a separate CL will help
ensure that upcoming changes to this code don't break anything,
and will make said changes easier to review.

Bug: 65674744
Test: atest FrameworksNetTests
Test: ConnectivityServiceTest#testStackedLinkProperties passes 400 times
Change-Id: I73cda0dd029a0e6546f7272beedf841ce5a4c0dc

Merge "Add IIpClient API for IpClient"

Deprecate JapaneseCalendar.CURRENT_ERA

In the future, if the era changes in Japan, CURRENT_ERA
on old Android devices will be incorrect.

Thus, Android app developer should not use this API.

Bug: 77850660
Test: m droid
Change-Id: I9ee640bccc83a73c94ac76138597f09f05af8796

Merge changes I5a6174a4,Idfbfdf54

* changes:
  Acquire lock to write NetworkStackConnector
  Allow Bluetooth to bind to NetworkStack

Add startActivityAsUser to SystemApi

Normal applications do not have permissions to use this method.

Test: m
Bug: 112869080
Merged-In: I693b3bf56f3be71f0790776e3aad5694717786ef
Change-Id: Ibe91c1bc4c94883a313e799cb0a37db7696fc62e

Add NetworkStack metrics to system API

Test: atest FrameworksNetTests NetworkStackTests
Bug: 112869080
Change-Id: Ie52c55f248c173e2a5ee603ecd004fbac9004ac1

Merge "Add fields used by NetworkStack to the API"

Merge "Remove mistake of calling iSub twice in getSelectableSubscriptionInfoList"

Merge "Add helper API to hide subscriptions when getting active subscriptions"

Add helper API to hide subscriptions when getting active subscriptions

Adding a hidden API that only return user visible active subscriptions.

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

Connect IRadioConfig 1.1 HAL interfaces with RadioConfig.java

IRadioConfig 1.1 getPhoneCapability and setPreferredDataModem
are defined. Here we hook the calls into RadioConfig.

Bug: 121354686
Test: sanity
Change-Id: I2a198d446962510ef9a3019d3aa30b40e796ebe0
Merged-In: I2a198d446962510ef9a3019d3aa30b40e796ebe0

Remove mistake of calling iSub twice in getSelectableSubscriptionInfoList

Don't call getAvailableSubscriptionInfoList twice in
getSelectableSubscriptionInfoList.

Test: build
Change-Id: Ie5821b1d8a01834228401ce6bb1e9ae251168760
Merged-In: Ie5821b1d8a01834228401ce6bb1e9ae251168760

Merge "Clean up config of tabs and trailing whitespace"

Merge changes from topics "coreapi-flag", "coreapi-stubs-csv"

* changes:
  hiddenapi: Support 'core-platform-api' flag
  hiddenapi: Expect public/private API as CSV

Acquire lock to write NetworkStackConnector

As Java reference writes are atomic, the previous code should not have
issues, but since mConnector is marked @GuardedBy it could trigger
static analysis.

Test: m
Change-Id: I5a6174a4f32c349e593d0a16d58293d608a487c7

Merge "Add OsConstants networking constants to public API"

Add fields used by NetworkStack to the API

Test: m
Bug: 112869080
Change-Id: I59f6d5ae8a601b7496548b87f137a577f2365a37

Merge "Replace uses of libcore.io.SizeOf with X.BYTES."

Clean up config of tabs and trailing whitespace

Test: whitespace only change
Change-Id: Ib2d8132e04b3bf604e2174deb1a54a074d8299d8

Add NetworkMonitor constants to SystemApi

NetworkMonitor lives in the NetworkStack which must only use system
APIs. This includes constants used by NetworkMonitor and captive portal
login.

Bug: 112869080
Test: m
Merged-In: I8b5f0bb61c3ef0d2f33af7dd67b72ac6c5a1d1b5
Change-Id: I4879568d3fbf9435767c8d4d0ab5198d421a2f4f

Add OsConstants networking constants to public API

Test: m
Bug: 123062477
Change-Id: Iac5c0c6acf8d24c17fb4780bcb97164ee21cca8f

Replace uses of libcore.io.SizeOf with X.BYTES.

libcore.io.SizeOf is an internal API and is going away soon.

Note: The files touched by this CL have some other unused
imports which I did not touch. Therefore I had to use
"repo upload --no-verify" to upload this CL to gerrit.

Bug: 123174577
Test: Manually checked that this CL applies cleanly in internal
      master and that no other uses of SizeOf then remain in
      internal master's frameworks/base
Test: Treehugger
Change-Id: Ib29a195f3dda2f21ebbe2f7073e0af4dbb524b19

Delay start of bootanimation until after the Runtime APEX is mounted.

It depends on libdexfile_external and libnative{bridge,helper,loader} which
are provided by the Runtime APEX.

Test: flash & boot
Test: atest CtsJdwpTestCases
Bug: 113373927
Change-Id: Id9a78f9adb4c928ab4c91fda0ee3ea4b13eed592

Merge changes I2ddfef0c,I49bee0c9

* changes:
  [MS08] Read back attributes and blobs.
  [MS07] Implement storeNetworkAttributes and storeBlob.

Add IIpClient API for IpClient

The API will be used to interact with IpClient once it is moved to the
NetworkStack process.
This change keeps compatibility with the previous API so that clients
can be migrated independently.

Test: atest FrameworksNetTests NetworkStackTests
Bug: b/112869080
Change-Id: I57d8164afae849230bbc0e25f17859c0b22d0f29

Allow Bluetooth to bind to NetworkStack

Test: flashed, verified bluetooth tethering gets IP address
Bug: b/112869080
Change-Id: Idfbfdf54754fea46eb0099b9b9a3bdc29dd241e0

Merge "Add stable AIDL parcelables for IIpClient API"

Merge "fix getIfaceStats and getTotalStats bug"

[MS08] Read back attributes and blobs.

Test: New tests in IpMemoryStore
Bug: 113554482

Change-Id: I2ddfef0c2ed37459c038f75d1dfc92fdefbf58f5

[MS07] Implement storeNetworkAttributes and storeBlob.

Test: New tests in IpMemoryStore
Bug: 113554482

Change-Id: I49bee0c903247e77ab93517efbe44548313cf1a4

Add stable AIDL parcelables for IIpClient API

Test: atest FrameworksNetTests NetworkStackTests
Bug: b/112869080
Change-Id: Ia4ac1eb482850a06c4dc1b6c1fe55d33d4087388

Merge "Remove dead KeepaliveInfo if it cannot be started"

Merge "Update signature files to the new format"

Merge "Adjust to new OpenNativeLibrary API."

hiddenapi: Support 'core-platform-api' flag

Libcore class members annotated with @CorePlatformApi now generate
a new hiddenapi flag. This is the first of "domain API" flags which
can be used in conjunction with API list flags. Therefore modify
the 'generate_hiddenapi_lists.py' logic to treat them differently.
Specifically, the script marks otherwise unassigned class members
blacklisted. A class member with 'core-platform-api' may still not
be assigned an API list and should be blacklisted.

Bug: 119068555
Test: m appcompat
Change-Id: I2b67e7a619677e853c87bc2da934410458ce4d14

hiddenapi: Expect public/private API as CSV

Refactor of `hiddenapi` changed the output format of public/private API
lists to a single CSV file. Change API list generation accordingly.

In order to avoid special-casing this CSV file, it is treated the same
as the CSV files produced by `class2greylist`. The merging rules are
relaxed so that signatures in CSV files are not checked against
a pre-initialized set of all signatures (previously generated from the
public/private API files). This should not lead to build errors as the
CSV files are always auto-generated, and a missing/extra signature will
be caught by `hiddenapi`.

API lists in frameworks/base/config are processed later and checked
that they are a subset of the signatures in CSV.

Bug: 119068555
Test: compiles, hiddenapi-flags.csv unchanged
Change-Id: I33f2cbaa15f2d423a75e6ca64abe1c5b0c40c86f

Update signature files to the new format

The new format is described in tools/metalava/FORMAT.md.

Fixes: 116589402 Switch signature files over to the v2 format
Fixes: 112626813 Drop "synchronized" modifier from the signature format
Fixes: 122358225 Omit overriding methods that only differ in final [...]
Fixes: 73088715 API Review: Need doclava to output nullability
Fixes: 79739773 API Review: Metalava enforcing constants are constant
Exempt-From-Owner-Approval: Large-scale tooling change
Test: make sdk, make update-api, make checkapi
Change-Id: I8314f4e7099fa92e4f8ed7d283ccf836cc9a84a0
Merged-In: Ia248aece5250e84e47c815c601133b698bf644c2

Merge "Pass in calling UID and package to dumpstate"

Merge "Trigger PhoneStateListener onCallAttributesChanged"

Merge "Remove the eng tag from invoke_mock_media_player"

Merge "Fix potential NULL dereference errors."

Merge "Carrier restriction enhancements for Android Q"

Merge "[b/109850442]Fail to adjust volume when connect Ampplifier.]"

Merge "Quiet SNaN inputs in Half.toFloat()"

Merge "Cellular data stall recovery based on networkStatus"

Cellular data stall recovery based on networkStatus

This CL introduces cellular data stall recovery based on
network validation result. It still re-uses the existing
recovery steps but the triggers are based on networkStatus
from NetworkMonitor when the network is bad. Each recovery
step is spaced by a configurable grace period (default 3 mins)
to avoid back to back recovery.

This feature is controlled by the following settings. If this
feature is enabled then the legacy data stall detection based
on TrafficStats will automatically be disabled.

Settings.Global.DATA_STALL_RECOVERY_ON_BAD_NETWORK
1 => Enabled (default)
0 => Disabled

Settings.Global.MIN_DURATION_BETWEEN_RECOVERY_STEPS_IN_MS
3 minutes is default value

Bug: 113916551
Test: atest FrameworksTelephonyTests:com.android.internal.telephony.dataconnection.DcTrackerTest

Change-Id: I738fc7ed08803de1114f7a3d79bf278449e94ea8

Remove the eng tag from invoke_mock_media_player

The test using this has been suppressed for >9 years since we moved
testing from eng devices to userdebug.

Test: treehugger
Change-Id: Ie91f3b97a33a8e8e5be49ea2098f163567e2496d

Carrier restriction enhancements for Android Q

Add support for dual SIM to carrier restrictions.
Add support to exclude specific carriers in the list of carrier
restrictions.

Bug: 120313541
Test: Created test application to verify correct functionality.
Test: Added test cases for CarrierRestrictionRules.
Change-Id: I2723fb07db13efdafbc74471a87182e2d2a32718
Merged-In: I2723fb07db13efdafbc74471a87182e2d2a32718

Merge "Avoid calling ParcelFileDescriptor.fromSocket on new Sockets."

Merge "Fix maindexclasses use of proguard"

Merge "Add SOONG_HIDDENAPI_FLAGS as hiddenapi_flags.csv dependencies"

Merge "Add thread ID reuse to API contract"

Merge "Remove LTE Rsrp Boost from SignalStrength"

Merge changes from topic "EnableAirplaneModeInSafeMode"

* changes:
  Add import of android.provider.Settings
  Enable airplane mode when booting in safe mode

[b/109850442]Fail to adjust volume when connect Ampplifier.]

[Description]
Fix this issue : Don't suppress adjustments for single volume device

Test: Android TV: press vol button once
Change-Id: I2d37ce60c3e6928e00b036a805f01f281437f4ac

Add SOONG_HIDDENAPI_FLAGS as hiddenapi_flags.csv dependencies

Hiddenapi build logic has moved to soong and list of flags.csv files
is now being passed to make files through SOONG_HIDDENAPI_FLAGS
variable. Adjust the build rule for hiddenapi_flags.csv accordingly.

Test: aosp compiles
Change-Id: I930f918fb36ce3532478035e83c2632559fd07b6

Remove dead KeepaliveInfo if it cannot be started

If KeepaliveInfo was created but cannot be started, current
design will only notify apps but not remove it from local
hashmap, which will cause unexpected behavior if someone
tries to iterate records and fire callbacks.

Bug: 122636283
Test: atest FrameworksNetTests
Change-Id: If4cb25a881dd3555f569c729f88741675f7f3bdf

Adjust to new OpenNativeLibrary API.

Bug: 122874359
Test: m
Change-Id: Ia018fd334eaf77b690ee6f7b5852743d1efa0e2f

Merge "Split applyNewState into enable/disable APN"

Fix maindexclasses use of proguard

Fixes: 122988865
Test: OUT_DIR=$HOME/out mmma frameworks/base/core/tests/hosttests/test-apps
Change-Id: I7f1f22dbe016182d2502643e45e095a5d8f60e41

Merge "Update sharedPreference to false when exits new device provisioning"

Merge "[CS] Add an option to block sensitive network specifier"

Merge "Add Dynamic tiles for Settings"

Split applyNewState into enable/disable APN

This is the preliminary work for adding handover
type for enable/disable APN.

Test: Telephony sanity tests + unit tests
Bug: 73659459

Merged-In: I51fd01d798cdf8d04fd41455126d017ac97f5f3f
Change-Id: I51fd01d798cdf8d04fd41455126d017ac97f5f3f
(cherry picked from commit ade6d1a66d0cc4ef9260c80c760093a2386679d8)