Merge changes from topic "face-hidl"

* changes:
  Update framework together with HIDL onRemoved
  Update constants with HIDL

Merge "Improve error message when expecting a non-virtual autofill id."

Merge "Update resolver icon generation"

Merge "Only wait for animations to complete when injecting a DOWN event."

Merge "Fix issue where wrong icon would be shown"

Merge "Integrate FEATURE_TELEPHONY_IMS feature into APIs"

Merge "Allow touchable regions for assist UI"

Merge "Acivity to rethrow an exception from AMS."

Merge "Destroy remote service when CCService dies."

Acivity to rethrow an exception from AMS.

Bug: 128414194
Test: Check if existing tests pass. Should be no-op for normal cases.
Change-Id: Ib9428c34fba8315d5208fa9427e095066aeff4fa

Merge "[API feedback] fixed API for NR"

Merge "Include minute interpolation on hour hand angle."

Merge "Remove interface for AttentionManagerService."

Merge "Remove interfaces for runtime and runtime_native namespaces."

Merge changes from topics "string_suggestion_notification", "disconnect_blacklist", "api_modify"

* changes:
  wifi: Fix runtest.sh
  strings.xml: Modify suggestion notification strings
  WifiConfiguration: Add a new temp blacklist reason for disconnect
  WifiManager: Fix javadocs
  WifiNetworkSpecifier/Suggestion: Address API council comments

Merge "Make Paint default to filtering Bitmaps"

Merge "Modified 3 methods to private methods. They should not be public."

Merge "Add javadoc for IMM.showInputMethodPicker"

Merge "Revert deletion and deprecate few PopupWindow methods"

Merge "Add MANAGE_TEST_NETWORKS permission as @hide" am: 96f51fd970 am: 7a66b02e62
am: db34163526

Change-Id: Ic283c247bbb5b62f9816cf074abd1fbeb5778cd4

Merge "Changed navigation handle to be longer and removed assistant handle"

Update resolver icon generation

Adds support for launcher-like icons via a ported form of
Iconloaderlib. Only fetches resolved activity's app icons with a special
cutout based on permission.

Bug: 126568207
Test: manual, mostly port of known to work code
Change-Id: Iba2c6f9aa5e63c457d9fd47921817c6529776b7a

Merge "Add MANAGE_TEST_NETWORKS permission as @hide" am: 96f51fd970
am: 7a66b02e62

Change-Id: I1999d3c8c517f3f1369005b986f18105ef793dab

Improve error message when expecting a non-virtual autofill id.

Bug: 122959591

Test: atest CtsContentCaptureServiceTestCases
Test: mmm frameworks/base/:doc-comment-check-docs

Change-Id: I183771036e27bc770b9f895e1afd4a30b499b1a1

Only wait for animations to complete when injecting a DOWN event.

When running quick step launcher tests, the tests attempt to inject
input to start the recents animation. However, because the launcher is
animating, the inject input will wait until the animation is complete.
This causes very slow input injection when attempting to go to recents
with quick step.

Since most cases will inject DOWN before any other event, it's mostly
safe to only wait for animations to complete before the first injection.
This is because the windows will most likely be correctly set up by then
and all other injections can proceed normally.

The code makes an exception for events that are from SOURCE_MOUSE since
we do want to wait for an animation to complete when dragging around a
mouse pointer since focus change could affect behavior.

Fixes: 128333900
Bug: 128446223
Test: TaplTestsQuickstep#testBackground,testSwitchToOverview
Change-Id: I153130282a7cc824d260541b98b57c04e5f90f4f

Merge "Sharesheet - Scroll the content preview area"

Merge "Add MANAGE_TEST_NETWORKS permission as @hide"
am: 96f51fd970

Change-Id: Ia5e9f3e7ba7024151d0f4fa595bf66daaa0fc948

Merge "Add MANAGE_TEST_NETWORKS permission as @hide"

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

Fix issue where wrong icon would be shown

Fixes: 128395347
Test: manual
Change-Id: I5233121a9210f337944affb1784c7c91ec2c6a0b

Merge "Camera: Centralize OWNERS"

Remove interfaces for runtime and runtime_native namespaces.

Per API council feedback, we are making changes to include only the
namespace in the system API defined in DeviceConfig.java. Strings which
define property names should be defined in code local to the feature
instead.

Bug: 126411407
Test: atest FrameworksCoreTests:DeviceConfigTest

Change-Id: I73157c127a246719afb37eca65cabf2e3d24b38a

Merge "Remove deprecated Settings APIs."

Merge "Stop double-counting native memory in graphics classes"

Merge "Allow abandoning failed staged session"

Merge "Revert "Camera: Temporarily add @UnsupportedAppUsage to various Key things.""

Modified 3 methods to private methods. They should not be public.

Bug: 126366588
Test: Manually. Built and flashed on my test device.

Change-Id: I4d94b442baa6856c47c33bffb136164ee171fc60

Merge "Deep-link to notification categories page from the inline notification silencing controls."

Merge "Fix talkback on notification silencing buttons."

Merge "Face interrupt polish"

Revert deletion and deprecate few PopupWindow methods

After method rename procedure b/126354063 we discovered that
androidx-platform-dev already depends on that methods added during fixit
week, so revert deletion of these methods and deprecate them.

Bug: b/126354063
Test: PopupWindowTests
Change-Id: I17b985542ffd661358059740bdf620e1bfb5b550

wifi: Fix runtest.sh

Bug: 128603658
Test: ./frameworks/base/wifi/tests/runtests.sh
Change-Id: Ica4754d2ed4154a1292ff4f0b65ac34cb281bee1

Merge "Add @UnsupportedAppUsage annotations"

Merge "Using LangID to detect the language of the text and pass it to annotator"

Merge "Optimizing getUnsuspendablePackages"

Merge "Add nullable and nonnullable annotations to DeviceConfig."

Merge "Adding fadingEdgeLength dimension for TextView."

Merge "Check for theme change after attaching to window."

Allow touchable regions for assist UI

System vendors may require their system UI to show UI for an assistant.

This CL adds methods to AssistManager to communicate if the system UI
is showing UI for the assistant, as well as a region of the screen that
accepts touches for the assistant UI.

Test: Implemented subclass of AssistManager with real implementations
of the new methods. Verified touches were intercepted/not intercepted
as setShowingAssistUi(false/true) was called.

Test: verified Assist UI is presented/disappears as appropriate
Change-Id: I7702f80b75a231dc5718128e49335d34beeaccfd

Merge "Sharesheet - Remove pin/unpin actions"

Merge "Refactoring to extract method for better readability"

Sharesheet - Scroll the content preview area

Moved the content preview into the listview to be scrolled
with the rest of the content. Finally separated out each
content preview into its own layout file.

Bug: 120419296
Test: atest ChooserActivityTest, observe content preview area being scrolled
Change-Id: I2720ca5264a4db34abae178e653e4e91bc54500c

Merge "Adding "@defgroup" DOxygen tag to AMidi"

strings.xml: Modify suggestion notification strings

Bug: 120072329
Test: Manually verified the notification with CTS Verifier.
Change-Id: Ia4fea5db6affe70005685805a783d6e50ce8db59

WifiConfiguration: Add a new temp blacklist reason for disconnect

Also, added a constant to indicate the start of the permanent
blacklist reasons.

Bug: 126503889
Test: Compiles
Change-Id: I4ba1dc5377bffe34a9f236c0698c36cbf34969a9

WifiManager: Fix javadocs

Bug: 128536865
Test: make docs
Change-Id: I263b66e989b2d2240e25cac31c00fc3b8a3f5a3d

WifiNetworkSpecifier/Suggestion: Address API council comments

Modified builder methods to:
a) Add boolean input for all boolean setters.
b) Add a public value for unassigned priority and added IntRange
annotation.

Bug: 128317241
Bug: 128317186
Test: Compiles
Test: ./frameworks/base/wifi/tests/runtest.sh
Change-Id: Iaf9d26ed857fe9337d3e4006c9c894efbcfa32dd

Make Paint default to filtering Bitmaps

Bug: 77949917
Test: I5737be68dccdc5ebb38291bf3515ae25553dcfc2

In P, the switch to Skia HW rendering lost a feature of HWUI's old
renderer: using a Paint with a BitmapShader resulted in filtering the
Bitmap, regardless of whether FILTER_BITMAP_FLAG was set.

This somewhat restores that behavior, in that it results in filtering
BitmapShaders so long as the client doesn't explicitly turn it off. This
also means that it continues to be possible to get a pixellated drawing
if the client *does* explicitly turn it off.

Change-Id: I62c726a0ba281de239092f066a3df041ecbcee5b

Stop double-counting native memory in graphics classes

Bug: 121304803
Test: Infeasible

Previously, the NativeAllocationRegistry needed to report how much
native memory was being held onto by the Java class in order to get a
more accurate count of how much memory was in use. But recent changes
allow the system to have an even more accurate view of the native memory
with mallinfo(). Further, the AOSP change
Idccb8b50d26c8c3e93cc34040d784f74dfcdbf51 introduces new APIs that allow
distinguishing between large native malloc allocations, which should
cause more frequent mallinfo() checks, and whose sizes need be available
to ahat etc, and smaller native malloc allocations.

Bitmap and AnimatedImageDrawable use the version for large native malloc
allocations. The rest pass an implied size of 0. Note that many of the
old Registries used somewhat arbitrary sizes, reinforcing the fact that
the new way of keeping track of this is better.

Add Bitmap::pixelStorageType to differentiate between types of memory
that should be accounted for by the Registry.

Update Bitmap::getAllocationByteCount to report the actual size of
ashmem allocation.

Fix a typo in LineBreaker.java, discovered while looking to find
existing callers of Bitmap's constructor.

Change-Id: I57c407258450aeaf08b47df32432466639d9faed

Merge "DPM API review: Remove "ForUser" method"

Merge "Visual changes in OngoingPrivacyChip"

Merge "Add missing XmlUtils.skipCurrentTag(parser); to "bugreport-whitelisted" case"

Merge "API Review: Private DNS host can be null"

Merge "Pull system ion heap size."

Using LangID to detect the language of the text and pass it to annotator

1. Use LangID to detect the language tags, and pass the result to
   the native side.
2. Have feature flags to turn off "detectLanguageTagsFromText" and
   foreign action for classification respectively.

Test: Select a non-english word, no more "define" action.
Test: atest frameworks/base/core/tests/coretests/src/android/view/textclassifier/

BUG: 123705564
BUG: 128413589

Change-Id: I6a68caca1e0709e63319907acbf8d776894f555b

Add missing XmlUtils.skipCurrentTag(parser); to "bugreport-whitelisted" case

Somehow missed it in the original CL. :(
CL with CP into aosp already has this fix:
aosp/ee4d7be21948307a0fedc6ab75a21d41660aa0c5.

Test: m
Bug: 126541701
Change-Id: Ib5a0f0c7e07699e99f618fc70f4b3cba3abd235d

[automerger skipped] Merge "Add a whitelist to control packages that can use Bugreporting API" am: 17492d0691 am: e54c0344de
am: 36abd0a7e8 -s ours
am skip reason: change_id Id0b61ccc1adf40bcb455d3b59b640f4b160bdd84 with SHA1 07964b49ac is in history

Change-Id: I4aa30f899c83e80d07837adc2921b9f9b6261707

Merge "Fixed text of manage notification button for a11y is confusing"

Merge "Add a whitelist to control packages that can use Bugreporting API" am: 17492d0691
am: e54c0344de

Change-Id: I8545743ebce6206b2b8d747eb252a31282bc86cf

Merge "Add a whitelist to control packages that can use Bugreporting API"
am: 17492d0691

Change-Id: If224c6522c89c53cb1d5e72a55d5aa051a4e0d02

Merge "Don't tie getPasswordComplexity to secure lock screen."

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

API Review: Private DNS host can be null

Mark getGlobalPrivateDnsHost as @Nullable since it can return null
(in the case of a host not set).

Bug: 126699845
Test: That it builds
Change-Id: I4680a8fa59815f62a74c14c149124d4cda567cca

DPM API review: Remove "ForUser" method

Replace the setProfileOwnerCanAccessDeviceIdsForUser with
setProfileOwnerCanAccessDeviceIds that takes the DPM's user ID.

Per automated API review, new methods that target a specific user (by
taking in a UserHandle) are discouraged.

Instead, the caller should get the user-specific service from the user
context.

Bug: 126700461
Bug: 126699845
Test: Manually, flashed a device and set-up TestDPC on it straight to
work profile, then verified it can do Device ID attestation.
Test: atest FrameworksServicesTests:DevicePolicyManagerTest

Change-Id: I13f20f4a434394f266a3dc2373152cb1b9ea2d2b

Merge "Prevent readTrainInfo from crashing statsd."

Merge "Keep ContentCapturePerUserService alive while the package is being updated."

Merge "Add StorageManager checkpoint APIs."

Prevent readTrainInfo from crashing statsd.

copy_n in readTrainInfo needs an input iterator and an output iterator.
When we read the size of the trainName, we were passing an
outputIterator that was a pointer to size_t. The goal was to fill up the
size_t with the length of the train. However, we need to do this byte by
byte, and by passing the pointer to the size_t, we increment by size_t
bytes instead of one bytes when we increment the iterator.

The change now is to use linux read, which is the mirror to how we write
the file. This code is cleaner and less error prone than reading the
file to a string.

Another thing this cl does is implement some sanity checks to make sure
we dont read pass the string input buffer, and to try to safeguard
against devices that upgrade from beta 1 to beta 2 and have a trainInfo
from beta1 stored on the device (since they are not backwards
compatible.) Another option here is to create a separate file for
everything besides the experiment ids, but that might not be deasible in
the time we have.

Test: storage manager unit tests pass
Test: debug phonesky apk to trigger binary push, manually inspecting the
file created to ensure the write works, and using adb shell cmd stats
pull-source to verify the read works.
Test: statsd testdrive 10051

Bug: 128460940
Change-Id: I3c08d946db9c43d77e5efca2eb1088adbb4a3ce1

Keep ContentCapturePerUserService alive while the package is being updated.

Test: manual verification (cannot be tested using CTS because it would kill the test process)
Test: atest CtsContentCaptureServiceTestCases

Bug: 126266412
Fixes: 128466656

Change-Id: I73e89f41b58615070c38103fa2f1fa04ac015dca

Merge "Revert removal of GpsStatus APIs"

Merge "Fix NPE for auto-bubble notifications"

Merge "Support PointerLocation on secondary display"

Merge "Changed ContentCapture session lifecycle so session is kept alive when service died."

Merge "Fix autogrant of location permissions for use-open-wifi apps."

Merge "fix the issue that setPreferredNW send wrong RAT to HAL" am: 4837dbe536 am: 3a720c6739
am: c7fb8914b6

Change-Id: Ibc3e52b0a109843808a6042a3e41530f1fdf5a17

Merge "fix the issue that setPreferredNW send wrong RAT to HAL" am: 4837dbe536
am: 3a720c6739

Change-Id: I2245e9a0fd7afac8835dbf1ead499153efa39e36

Merge "fix the issue that setPreferredNW send wrong RAT to HAL"
am: 4837dbe536

Change-Id: I33e98b0f4ac9b0d7130b9e6b25fec5b76409e021

Merge "Mark return value of onBind() as @Nullable"

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

Merge "Addressed deferred CL feedback comments."

Revert removal of GpsStatus APIs

Bug: 126698813
Test: manual
Change-Id: I174d3f4cd21259e59381fc79990005c644380065

Merge "Get flaky test out of pre-submit and move non-flaky test into it."

Fix NPE for auto-bubble notifications

Test: manual - have notif based bubbles, dismiss
Fixes: 128525044
Change-Id: I94cc9c44c8ee2ae70a8f8aa2e1c98710f10e5358

Merge "Change target SDK to '28' am: b0103b5ebb am: dc65b65666"

Change target SDK to '28' am: b0103b5ebb
am: dc65b65666

Change-Id: Ifb9d9fa0df3b49bbf827120d211b8c8480cbb5e2

Merge "Update ringer setting API visibility per council feeedback"

Addressed deferred CL feedback comments.

This CL addresses several bits of feedback from previous CLs:
* ZygoteState.mABIList => ZygoteState.mAbiList
* Adding appropriate whitespace
* Finalizing variables where appropriate
* TEMP_RETRY_FAILURE around write
* Zygote.getSystemProperty => Zygote.getConfigurationProperty

Test: m
Test: Treehugger
Change-Id: I5ae4c8dfc336c7511bee375a80966abb1ead275e

Change target SDK to '28'
am: b0103b5ebb

Change-Id: If4e9acb9fef17a38013d532196a06091446f5bb2

Fix autogrant of location permissions for use-open-wifi apps.

For apps that are targeting pre-Q will get COARSE and BACKGROUND
location permissions.

For apps that are targeting Q+ will get FINE and BACKGROUND location
permissions.

Bug: 124237492
Test: flashed onto walleye and confirmed permission granted when WFA is
enabled.
Change-Id: I1ce73f76c6ebda61f95a3843a77d0055a44b98eb

Face interrupt polish

Let's try to authenticate more often, especifically when the user
pulls down quick settings.

Transient messages on the keyguard also be cleared if there's a
new authentication attempt.

Fixes: 128034459
Fixes: 128037051
Bug: 111414690
Test: manual with multiple biometrics on multiple device types
Change-Id: I4b231579565e6cce4604caebefc45ce20a19e903