[automerger skipped] Merge "Frameworks: Remove complex boolean"
am: 16eef1aad8  -s ours

Change-Id: I407b42d45738599955bca210b5f9896753e51224

Merge "persist.sys.boot.reason is cleared once read by bootstat"
am: 8ec0c6ba48

Change-Id: Ib52e362e07a81086e142ff1f120a266862f0a073

Merge "Frameworks: Remove complex boolean"

Fix a syntax error in SystemServiceRegistry

The syntax error was introduced by a bad conflict resolution
in http://ag/4324946.

Bug: 110097023
Test: Inspection only
Merged-In: I9e4eac70b944441f34491315cd1ce7fa2b9ae150
Change-Id: I546f2d010faa6f75033cf7e3e51ef7ed640854ed

Merge "persist.sys.boot.reason is cleared once read by bootstat"

Merge "Simple pass-through TimeDetectorService"
am: cb749e2e50

Change-Id: Ic2e3d3c3f9181ef4054f599baf4ac086ec21d8bd

Frameworks: Remove complex boolean

To allow an Errorprone warning to become an error.

Bug: 72451126
Test: m
Change-Id: I8733f0ee74de615e4ffc979fb9de38730c2506c6
Merged-In: Ia6856157e8813856268fba003e1e591d690cb26e

Merge "Simple pass-through TimeDetectorService"

Simple pass-through TimeDetectorService

This is sufficient to wire up time detection from telephony
to the new service without breaking time detection.

This cherry-pick contains a small change: to use
SystemClock.elapsedRealtime() instead of the newer
SystemClock.elapsedRealtimeClock() with Clock.millis().

Bug: 78217059
Test: atest FrameworksServicesTests:com.android.server.timedetector
Test: atest FrameworksCoreTests:android.util.TimestampedValueTest
Merged-In: Id7175878dc22e5272c31f3e478af4b0e4183b62b
Change-Id: Id7175878dc22e5272c31f3e478af4b0e4183b62b
(cherry picked from commit 24836bfb1564b1f98c322108a924965a8dbbee4f)

Merge changes I6abd2221,I2c149d29,I45d22857,I1f879b2c,If4986a25, ...
am: 6c0106b57f

Change-Id: I86bc83e2bd859b48017f12832e9ff0080ec162b8

resolve merge conflicts of 5e1d95b0cc05eb1a6bf25a13223601f2e8f8fccb to stage-aosp-master

BUG: None
Test: I solemnly swear I tested this conflict resolution.
Change-Id: Ic5527b29cbe8c84d2053be849cf09df5341e4790

Merge changes I6abd2221,I2c149d29,I45d22857,I1f879b2c,If4986a25, ...

* changes:
  [PT06] Move setGlobalProxy into ProxyTracker
  [PT05] Move sendProxyBroadcast into ProxyTracker.
  [PT04] Move PacManager into ProxyTracker.
  [PT03] Move some more code into ProxyTracker.
  [PT02] Move static methods to ProxyTracker
  [PT01] Introduce ProxyTracker
  Remove a useless parameter.
  Small cleanup of Network.
  Cleanup of LinkProperties.

[PT06] Move setGlobalProxy into ProxyTracker

Test: runtest
Change-Id: I6abd2221882db368a411b7174c66d8bd3b6b5110

[PT05] Move sendProxyBroadcast into ProxyTracker.

Test: runtest
Change-Id: I2c149d29d4b75d3978021b940b6bc58f677b8d17

[PT04] Move PacManager into ProxyTracker.

Test: runtest
Change-Id: I45d22857459fe1746d484ac04f8d5cd81fc61835

[PT03] Move some more code into ProxyTracker.

Add finals and annotations. Remove comments that have lost their
context (they were in the context of disabling a permission check
that had been added, but constituted an API change that would not
serve any real purpose).

Test: runtest
Change-Id: I1f879b2c105d2127072b88233d72097a0d78fe14

[PT02] Move static methods to ProxyTracker

No logic changes. Only changes are adding nullability annotations,
final modifiers, and adding an s in a comment.

Test: runtests
Change-Id: If4986a25bb36819de8ff459c4c0439c56d4e5a50

[PT01] Introduce ProxyTracker

The goal of this is to simplify ConnectivityService by reducing
the amount of code it contains. This is small enough to be obviously
correct and followup changes will move code into this class.

Test: runtest
Change-Id: Ic5ab19b521e98ae397c9bf657856820304362fbb

Remove a useless parameter.

This argument is useless because all its callers pass the same
value : false.

Test: runtest
Change-Id: Id921caa4ffadef535a5bbcfea052283a07320b28

Small cleanup of Network.

Test: runtest
Change-Id: I56dbd37bb8f890938d360f45835de72be4beb91a

Cleanup of LinkProperties.

Test: runtest
Change-Id: I7299de93a79901635ce755a2d933666ee43767d5

Merge "Minimum viable TimeDetectorService"

[automerger skipped] Merge "Update Docs to clarify using a Message across IPC"
am: 0be0692691  -s ours

Change-Id: I1d3887c7d4a868a1f1b56c0d2e4f824eef29fb3a

Merge "Update Docs to clarify using a Message across IPC"

Merge "Framework: Disable native-bridge for non-zygote"
am: 05c09faad2

Change-Id: I2aea1bd9c520ca6bb42bf80bd445da0e86db8136

Merge "ScanRecord.getServiceData NPE fix"
am: d62f74e806

Change-Id: I11a254cf633dc5cf9a7c53c992bf49b00f3d274e

Merge "Framework: Disable native-bridge for non-zygote"

Merge "ScanRecord.getServiceData NPE fix"

ScanRecord.getServiceData NPE fix

Bug: 109662587
Change-Id: I0e9653d9cd7e4b748870bacb6fbcecae535ebb2a

Minimum viable TimeDetectorService

This is a do-nothing TimeDetectorService that can be
populated in following commits. A temporary method has been
added so the service has one method.

Unit tests can be run with:

atest FrameworksServicesTests:TimeDetectorServiceTest

Test: build / boot
Test: See above
Merged-In: I9e4eac70b944441f34491315cd1ce7fa2b9ae150
Change-Id: I9e4eac70b944441f34491315cd1ce7fa2b9ae150
(cherry picked from commit feeee682a23f25760c4a1aa07c00945901911ba3)

Merge changes Ifd7d73ad,Id79096fa
am: bfe3b1a691

Change-Id: I263f51ba2aafdf8e19e020b2a4f6ff1b8238fa3c

Merge changes Ifd7d73ad,Id79096fa

* changes:
  Bugfixes in LinkProperties#toString
  Cleanup of ConnectivityManager.

Bugfixes in LinkProperties#toString

We should stop trying to use a human brain to write code

Test: manual
Change-Id: Ifd7d73ad65de0f671466080beacbd97287e17bfe

Cleanup of ConnectivityManager.

This is not exhaustive, but still an undeniable improvement.

Test: runtest frameworks-net
Change-Id: Id79096faf20eec80de7c17b261ef338b895af1e0

Merge "Use ParcelFileDescriptor's C++ implementation in libbinder"
am: 20e243d627

Change-Id: I0bc6158d1c37ee8a02450dee55ca6732d878fde3

Merge "Use ParcelFileDescriptor's C++ implementation in libbinder"

Merge "No-op cleanup of ConnectivityService."
am: a190a41b51

Change-Id: I9ec9633a85b8b2298f90419da2a73c540b6e6805

Merge "No-op cleanup of ConnectivityService."

Merge "Copy clat modified linkProperties into new one."
am: 33b5abb6f8

Change-Id: Ia1f9ab3d5c3b889beddb4dda56ad3389c4967275

Merge "Copy clat modified linkProperties into new one."

Merge "Disable the AppOp Restriction for IpSec Tunnels" into stage-aosp-master

Merge changes from topic "audio-package-update"
am: 2d5156733c

Change-Id: I051fc7af9663a90ff5b7e080dd44001646d35089

Merge "Check for null path in getInternalPathForUser"
am: 0559afdbcc

Change-Id: I0669f429a389655e04f26ab4cd19fbf69f3dbef1

Disable the AppOp Restriction for IpSec Tunnels

This CL temporarily removes the AppOp restriction
that disallows creation of IpSec tunnels due to
the lack of the appropriate AppOp in AOSP/master.
When the relevant framework merges out to master,
this CL should be reverted.

Manually merging this due to skipping a merge
conflict at ef9f2740a790feceaa47a24b1a179e93c4ffb5e6

Bug: none
Test: compilation
Change-Id: Ic06c193f85f6bcdd0ead4238825c1add78703cde

Merge pi-dev-plus-aosp-without-vendor into stage-aosp-master

Bug: 79597307
Change-Id: I2ec319e9fc92e959580a4a3cfe955ae91996dc96

Skip ef9f2740a7 in proposed pi-dev-plus-aosp-without-vendor merge.

Bug: 79597307
Change-Id: I8ac04fdd94b069434e2f00b0bfab63eb916f6129

Merge changes from topic "audio-package-update"

* changes:
  Include InCallNotification.ogg in AllAudio.mk file
  Add ChargingStarted + InCallNotif to AudioPackage

Merge "Check for null path in getInternalPathForUser"

Merge "API updated for ICU4J 61.1"
am: c2791389a5

Change-Id: I15f620499d0cc69dae06e6d986f8e2d9bcde26a6

Merge changes from topic "ipsec-sync-with-pi"
am: b2749539c0

Change-Id: Ic91ed8f496163c560c6f1fd306bc74e5fecb61c2

Merge "Adds a private field to parceled data."
am: 7db23752ab

Change-Id: I9d0bde31a8aeb67b62e202b3b8946b066b81d186

Merge "Unhide @SystemApi Constructor"
am: 442b546e2a

Change-Id: Ie6281283f6605b7125882698333644843838e2ef

Check for null path in getInternalPathForUser

In some situations, path could be null resulting
in a crash.

Test: no crash
Bug: 109730998
Change-Id: I2ce0410162d1327905d690331f461f9187e20906
(cherry picked from commit 6f6154bf0493cf66628c8f2418827fe54679b1eb)

Merge "API updated for ICU4J 61.1"

Merge changes from topic "ipsec-sync-with-pi"

* changes:
  Disable the AppOp Restriction for IpSec Tunnels
  Rework Exception Handling for IpSecManager
  Update IpSecManager to use InetAddress and prefixLen
  Add AppOps Checks for MANAGE_IPSEC_TUNNELS
  Add MANAGE_IPSEC_TUNNELS Permission

Merge "Adds a private field to parceled data."

Merge "Unhide @SystemApi Constructor"

Merge "Build dark greylist from public + light grey API"
am: 60229832de

Change-Id: I7a0077ed63f17e4a60482fad3f39ee1ab2d89461

Merge "Build dark greylist from public + light grey API"

API updated for ICU4J 61.1

Bug: 77634530
Bug: 109837373
Test: make
Test: cts-tradefed run cts-dev -m CtsIcuTestCases -t android.icu.dev.test.format.MeasureUnitTest
for CTS coverage
Change-Id: I721fbef5c79df1310e144a48ab2f18929b10bdfa

Merge "Opportunistic typo fix"
am: 6b03498a54

Change-Id: Id6dc12172a9fb1980a598ba070af1b10b1738849

Merge "Opportunistic typo fix"

No-op cleanup of ConnectivityService.

Funny how these things accumulate. Not exhaustive of course, but
still an improvement.
- Remove unused imports.
- Remove unused variables and members.
- Replace members with locals where applicable.
- Remove useless type parameters and explicit unboxings for Java 7.
- Conversely add the diamond operator for auto-genericity for
  Java 6.
- Reduce visibility of members where possible.

Test: runtest frameworks-net
Change-Id: I13586aee09b4cd1c87c525fafb5eee44dedb5360

Copy clat modified linkProperties into new one.

updateLinkProperties copied the linkProperties in networkAgent,
but the clat fix-up function did not copy stacked link into new one.
This caused an incorrect clat iface removal, and the IPv4 network
 to be unreachable.

Clean cherry-pick of ag/4260470

Bug: 80261579
Test: 1. ping after ipv6 only data call with 2g voice call on/off
      2. runtest frameworks-net
      3. run cts -m CtsNetTestCases
Change-Id: Ide47a3b0680dddfcf3e2e759a59b19aee3605050

Merge "pm: Exposing Vulkan 1.1 does not guarantee AHB extension support" into pi-dev
am: 551328eaaa

Change-Id: I4f9a420dcefb53e767a745f595787b71fcacd228

Merge "pm: Exposing Vulkan 1.1 does not guarantee AHB extension support" into pi-dev

Merge "Don't access keyguard from instant apps." into pi-dev
am: e7bd4e7e8f

Change-Id: I5d5d8ea0c7b5756143b5140e4cf16ae584147345

Merge "Don't access keyguard from instant apps." into pi-dev

[automerger skipped] Merge "Add field to set A2DP device volume on connect (1/2)" am: 1b9de57f38
am: fe96bd9c72  -s ours

Change-Id: Ife1a1762bba30d0a24919498c8b5c06b5639ac92

Merge "Merge "Revert "Let sendShowMessage called from hide to show"" into pi-dev am: eaf3590eda" into pi-dev-plus-aosp

Merge "Revert "Let sendShowMessage called from hide to show"" into pi-dev
am: eaf3590eda

Change-Id: I569a2caf105c4b6cf60b81026567ef4af7ca31c9

Merge "Merge "Add filter for NoPreloadHolders" am: 88ee5c93bb am: 202e61f127" into pi-dev-plus-aosp

Merge "Add filter for NoPreloadHolders" am: 88ee5c93bb
am: 202e61f127

Change-Id: Id4967cc4eb918ef81be118d6c03707b4e4e1983e

Merge "Animate owner info during wake-up transition" into pi-dev
am: c28e772042

Change-Id: I24a01e8e9ea9d82275a7434fc8dfdc5d1f4a30ad

Merge "Revert "Let sendShowMessage called from hide to show"" into pi-dev

Merge "Animate owner info during wake-up transition" into pi-dev

Don't access keyguard from instant apps.

Test: cts-instant-tradefed run cts-instant -m CtsViewTestCases

bug:109652548

Change-Id: I6f8bb714bd98fd53068b2662633a9155d64b274c

Merge "Revert "Allow instant apps to send broadcasts to themselves"" into pi-dev
am: 97995da6e1

Change-Id: I014e8de99a78a489f1f32738429d1cb0cebc4076

Merge "Add field to set A2DP device volume on connect (1/2)"
am: 1b9de57f38

Change-Id: I128bf69bfdd9efaaf186ed3b9278d7ef1479b19e

Merge "Revert "Allow instant apps to send broadcasts to themselves"" into pi-dev

Merge "Docs: noted that setting proxy hostnames on an ApnSetting requires an internet connection for DNS." into pi-dev
am: 0f5f09844d

Change-Id: I6c19f03debc6d15eece445dfd7fb0f5336d90407

Merge "Add filter for NoPreloadHolders"
am: 88ee5c93bb

Change-Id: I4e6b539a4fabf1584673100e48146f22a929f3b3

Merge "Docs: noted that setting proxy hostnames on an ApnSetting requires an internet connection for DNS." into pi-dev

Merge changes from topic "volume-change-flags" into pi-dev
am: a88cf78009

Change-Id: Idbcaa08b7255c8d2dfd7d2bae42fe1daf3c12bf8

Merge "Add field to set A2DP device volume on connect (1/2)"

Merge changes from topic "volume-change-flags" into pi-dev

* changes:
  Only show the volume dialog if the AudioManager.FLAG_SHOW_UI bit is set
  Comply with the volume callback change

Disable the AppOp Restriction for IpSec Tunnels

This CL temporarily removes the AppOp restriction
that disallows creation of IpSec tunnels due to
the lack of the appropriate AppOp in AOSP/master.
When the relevant framework merges out to master,
this CL should be reverted.

Bug: none
Test: compilation
Change-Id: Ic06c193f85f6bcdd0ead4238825c1add78703cde

Rework Exception Handling for IpSecManager

In order to properly support EOPNOTSUPP this CL
applies a consistent approach to handling Exceptions.
Hereafter, all exceptions that aren't of a special
method-specific type (such as SpiUnavailableException)
will all be returned to the calling process unchanged.
At the API call site, the ServiceSpecificException,
which is really an Errno, will be inspected and either
converted to an unchecked exception for types we know,
or it will be converted to an IOException in cases where
that method can return a checked exception. In cases
where we do not expect an errno, we will simply throw
a generic RuntimeException. This means all API calls
will now properly throw UnsupportedOperationException
and may be CTS tested accordingly.

Bug: 72420898
Test: runtest frameworks-net
Merged-In: I4a00e221618896223fcdb4b4279fb14cd14e34d8
Change-Id: I4a00e221618896223fcdb4b4279fb14cd14e34d8
(cherry picked from commit ddeb90aa9db108d4a2e5aadc778a726b65e5c921)

Update IpSecManager to use InetAddress and prefixLen

LinkAddress constructors are currently @hide; this change updates
IpSecManager to use InetAddress and prefixLen, and then construct a
LinkAddress internally. LinkAddress is used over the binder interface to
IpSecService to ensure validity.

Bug: 77528639
Test: CTS, Java unit tests ran on walleye
Merged-In: I19e124adef6d9f4992d8293db3190bcf74c95848
Change-Id: I19e124adef6d9f4992d8293db3190bcf74c95848
(cherry picked from commit 3f2c54b7820c3a568890700479230966fdbcf0a4)

Add AppOps Checks for MANAGE_IPSEC_TUNNELS

Adds support for a new AppOp to permit services to
use IpSec tunnel mode. The IpSecService now needs
a context so change the service mode to a cached
service rather than a static service.

Bug: 66955045
Test: runtest frameworks-net
Merged-In: I17a4a286225b432c3e15ea1587d946189931b4f4
Change-Id: I17a4a286225b432c3e15ea1587d946189931b4f4
(cherry picked from commit 592dadbd43fcb7c5d67e737adb34d07923da90c4)

Add MANAGE_IPSEC_TUNNELS Permission

Add a new MANAGE_IPSEC_TUNNELS permission and
protect all IPsec Tunnel mode APIs with it.
This permission is only granted to the system or
through an AppOp.

Bug: 66955045
Test: compilation
Merged-In: I0f618373b500c493ef2211bece681f74652a1833
Change-Id: I0f618373b500c493ef2211bece681f74652a1833
(cherry picked from commit 159788455c79fa47847d0c40fcee7aceff2551ae)

Change Quick Settings to account for branded VPNs.
am: 10422896ed

Change-Id: If9b181b5d4175f9418b4c8b0e4a8938b0bc98811

Merge "Disable REQUEST_LOCATION from GNSS HAL by default" into pi-dev
am: a364b3111a

Change-Id: I4ee77d8ae3417e76c330486205320077b23ea941

Merge "Remove usage of deprecated DayNightStyle values." into pi-dev
am: a0d475a43f

Change-Id: I603568a7168fb3b8a4a946d937b754f13b3d917c

Merge "Move nulling of nextFrameCallback" into pi-dev
am: e4e9ee7951

Change-Id: If988240d948266118a9441ee356ff384b4ff53b5

Only show the volume dialog if the AudioManager.FLAG_SHOW_UI bit is set

Bug: 109841417
Test: manual

Change-Id: Icc803a4b22ceccff7d3e788b9164975a8e395a3b

Change Quick Settings to account for branded VPNs.

Screenshots: https://photos.app.goo.gl/FrnK69TXR86npFmY2
Test: runtest --path frameworks/base/packages/SystemUI/tests/src/com/android/systemui/qs
Bug: 109837832
Change-Id: Ie447e6598ff593ff5d14ec8787db82ba39c9ea54

Merge "Disable REQUEST_LOCATION from GNSS HAL by default" into pi-dev

pm: Exposing Vulkan 1.1 does not guarantee AHB extension support

Bug: b/109875843
Change-Id: Ib53fb1a4f7a5a6c055b575b4d8775f8bd62db7a8

Revert "Allow instant apps to send broadcasts to themselves"

This reverts commit 7b40358fa316038ce6a3243b8706c6f5b7a49536.

Reason for revert: False positive - this works and patch not needed.

bug:109583877

Change-Id: If708350b03292cfa3bf9841199739e06c82f6c4f