Add a Telecom command to stop the number blocking suppression that
happens after an emergency call.

Bug: 158798240
Test: atest CtsTelecomTestCases
Change-Id: I122de1962fb6aa48a3092c2aa323dcbdcbb8f42e

This change automatically enables the IPSEC_TUNNELS mandatory feature
for devices launching with or above Q.

This is for OEM convenience to ensure that they do not have to add the
feature for each device that they ship.

Bug: 156711979
Test: Removed feature from device, CTS tested
Change-Id: I9d1a0e9616cdc7089a9bef7c96ff3780de36ed8d
Merged-In: I9d1a0e9616cdc7089a9bef7c96ff3780de36ed8d

This is a Bluetooth game controller which supports Android, but the
thumbstick buttons had the wrong keycodes.

Test: manual test with Gamepad Tester app
Change-Id: Id7d8ccbf282e25fd076903cbf9f3d66d30941c3e

Test extra info sent to NetworkMonitor correctly if network
agent is created through new NetworkAgent constructor without
legacy network info taken as parameter.

Bug: 156173829
Test: atest FrameworkNetTests
Change-Id: I4f827664c528bea30cc957a0a617dd37693f4460

Those two classes are used by many tests using android.test.runner.
Making them as public APIs so that those tests can depend on the public
stub of android.test.runner and stop depending on its implementation
library.

Bug: 157007292
Test: m

Change-Id: I33b458f34ffd4c73430f538a85aac308baa6e010

FrameworkTestRunnerTests is the test for android.test.runner. The test
accesses various private APIs from the sdk library. Previously, that was
possible because depending on a java_sdk_library gave the implementation
lib of the java_sdk_library for linking. That will be changed after
I14cf217f21fd3534c920c3a6336cf2d14c02e60c where the stubs are by default
provided for linking.

Since this test actually uses some hidden APIs, work around the
restriction by explicitly depending on the impl library.

Bug: 157007292
Test: m FrameworkTestRunnerTests
Change-Id: I0792e9eb1c7c594a1609eb89b6694c1578c86e18

In dual SIM mode, if a SIM is unplugged, there's a chance that
notifyDisplayInfoChanged is notified to registered client incorrectly.

Bug: 157039855
Test: manual - in dual SIM mode, when data is on eSIM, pull off pSIM.
Verify the rat icon of eSIM does NOT disappear.

Change-Id: I305ff855f0d5595be13b6ee4e4480abd89ed258f
Merged-In: I305ff855f0d5595be13b6ee4e4480abd89ed258f

Some developers have been surprised by this limitation and had trouble
figuring out what the issue was. Add documentation to address this.

This also includes a drive-by removal of a duplicate check.

Bug: 149867479
Test: doc-only change
Change-Id: I5911d01984695550b6c9afe7a8eb535bf5e320a1

The original implementation of
KernelCpuUidBpfMapReader.removeUidsInRange did not work unless data
was present for the first & last UIDs in the range. Add a test case to
verify the fix for this issue & prevent regressions

Test: confirm test passes with fix & fails without it
Bug: 157887803
Change-Id: Icfabb7298dbfb1a7f1b89dadbe33bae0492922b6
Signed-off-by: Connor O'Brien <connoro@google.com>

* changes:
  Add back missing `tidy_checks`, since `tidy_checks_as_errors` isn't enough
  Use `tidy_checks_as_errors` list instead of `-warnings-as-errors=*`

config_mobile_hotspot_provision_app would be move out of framework and
only private for tethering only.
enforceTetherChangePermission is no longer needed because its only
caller PanService already gate by other privileged permission
(BLUETOOTH_PRIVILEGED).

Bug: 146918263
Test: m

Change-Id: I030871c2bc46bc09c4e52970b4995f98d31bb90e

Change-Id: I5e541138c35cdf67d18ce5475221baeb3e7faef4
Test: emulator
Bug: 158151004

Bug: 157380700
Test: manual
Change-Id: Ia5e5a84e1b17887b440c2c83894f0a8cfc50faf4
Merged-In: Ia5e5a84e1b17887b440c2c83894f0a8cfc50faf4

Bug: 144284873
Test: make checkapi
Change-Id: I99ed93e96f7465f1f7c9a32c3715a874055dc809
Merged-In: I99ed93e96f7465f1f7c9a32c3715a874055dc809

The BPF tethering coordinator listens to the forwarding rule
changes for updating data limit which is applied in the following
conditions.
- After adding the first rule on a given upstream, add data limit.
- After removing the last rule on a given upstream, clear data limit.
- The service applies a new data limit on current upstream.

The reason for relying on rule changes is because the Tethering and
IpServer objects have multi-internal state machines. It is hard to
synchronize all of their states.

Note that the data limit cleanup for stopping or switching upstream
relies on offload rules are all removed as well.

Bug: 150736748
Test: manual
Change-Id: I829d36339973f9473fe6b616c48aa288f18d1c46

DhcpResults should only be used inside the NetworkStack: modules are
expected to use DhcpResultsParcelable instead.

Add a compat utility so current users of DhcpResults can do the
conversion (wifi in AOSP in particular). The utility can be removed when
changes to stop depending on DhcpResults are merged in AOSP.

Bug: 149403767
Test: built, flashed, WiFi working
Change-Id: I5b85c1a541ecdf9dd3e9403b9fb1c2b5aba98dd8

I unfortunately missed in the original testing that
`tidy_checks_as_errors` isn't sufficient by itself. Instead we need to
have both places with the same variables. A future change will make
Soong expand `tidy_checks_as_errors` into both locations appropriately.

Bug: 119328308
Test: WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=*,-readability-*,-google-readability-*,-google-runtime-references,-cppcoreguidelines-*,-modernize-*,-llvm-*,-bugprone-narrowing-conversions,-misc-non-private-member-variables-in-classes,-misc-unused-parameters,-hicpp-*,-fuchsia-* mm
Change-Id: If69f69bf7861ba3569756270381f3f737366f2f0
Merged-In: If69f69bf7861ba3569756270381f3f737366f2f0
(cherry picked from commit 56af5939)

This fixes a build break due to new code from
http://ag/I96f970e82b5243be01b205ac2cb6ab249c6100bc that inadvertently
triggers some false positives with other clang-tidy flags. Builds that
use a wider set of tidy flags (as specified in the test below) will
break since `-warnings-as-errors=*` turns all warnings into errors,
rather than just the ones that were specified as part of this
Android.bp. The preferred way of checking this is to use
`tidy_checks_as_errors`

Bug: 119328308
Test: WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=*,-readability-*,-google-readability-*,-google-runtime-references,-cppcoreguidelines-*,-modernize-*,-llvm-*,-bugprone-narrowing-conversions,-misc-non-private-member-variables-in-classes,-misc-unused-parameters,-hicpp-*,-fuchsia-* mm
Change-Id: I941b3c89402e2cfa290b3829e2ac8ff7ea67ca3d
Merged-In: I941b3c89402e2cfa290b3829e2ac8ff7ea67ca3d
(cherry picked from commit 0bc64543)