Merge "Remove Gallery and Music role" into qt-dev

Merge "Add getTextClassifier perf test" into qt-dev

Merge "Fix lifecycle issue in CommonPool" into qt-dev

Merge "Ensure getParentSurfaceControl and getSurfaceControl return valid or null SC" into qt-dev

Merge "Adding a level of indirection. Dimens value should live in the framework/base and not the overlay." into qt-dev

Merge "Support profileable application action" into qt-dev

Merge "make libmedia_jni_utils depends on libgui" into qt-dev

Merge "wifi.proto: Add link probe event to StaEvent" into qt-dev

Merge "zygote: initialize ashmem early" into qt-dev

Merge "Only switch task focus to window's task that was tapped outside current focus" into qt-dev

Merge "EGLAttrib requires special handling in JNI" into qt-dev

Ensure getParentSurfaceControl and getSurfaceControl return valid or null SC

Many places (like SurfaceAnimator#reset) expect that these SC will be valid or
null, but various implementors of Animatable were not fulfilling this contract.
Fulfill the contract and update the java-doc. Since everyone follows function
documentation this will never happen again.

Test: Existing tests pass. go/wm-smoke
Bug: 129949355
Change-Id: I5acb6b3209158c4c5b258a06c5d60d9e06bb6db1

Merge "fix AudioAttributes matching rule in AudioProductStrategy" into qt-dev

Merge "Revert "Revert "Mute haptic channels by default.""" into qt-dev

Merge "Flag to configure model parameters" into qt-dev

Merge "Set system gesture exclusion rects for Bubbles" into qt-dev

Adding a level of indirection. Dimens value should live in the framework/base and not the overlay.

Bug: 129080865
Test: Manual
Change-Id: Ic6bc3b254c32afad4230351fe9dd33217a7a86a4

Merge "Catch ArrayIndexOutOfBoundsException in StateMachine.toString()" into qt-dev

Merge "Fix isExternalStorageSandboxed() bug." into qt-dev

Merge "Expose notification guts on menu shown" into qt-dev

Merge "Update icon for Calendar permission." into qt-dev

Merge "fix nullability" into qt-dev

Merge "Sharesheet - Limit EXTRA_INITIAL_INTENTS to 2" into qt-dev

Merge "Update theme icon packs." into qt-dev

Set system gesture exclusion rects for Bubbles

When there is at least one Bubble on-screen, the bounds
of the first (top) bubble are set to exclude system
navigation gestures.

Test: manually w/fully gesture nav enabled
Bug: 126360272
Change-Id: I47a3c69fdc89476805d48b5214a7f05f5d033306

Flag to configure model parameters

The objective is to allow us to push model parameters by using a settings flag,
without the need of pushing a new model file.

Settings.Global#TEXT_CLASSIFIER_ACTION_MODEL_PARAMS stores a comma
separated string that contains these three fields:
1. required_model_version
2. required_locales
3. serialized_preconditions

To ensure serialized_preconditions is applied to the target model file,
TextClassifierImpl only applies the serialized_preconditions when
required_model_version and required_locales are both met.

Test: atest frameworks/base/core/tests/coretests/src/android/view/textclassifier/
Test: adb shell settings put global text_classifier_action_model_params 'required_model_version=0,required_locales=en,serialized_preconditions=FAAAAAAADgAIAAAAAAAAAAAABAAOAAAAAAAAAA=='
      Observe that the flag is actually applied in the model.
Test: Ensure that finalize is called when the activity is dead.

BUG: 123616497

Change-Id: Ie42dcfeee705c83bbb693a5c1a0fedd0821df5e7

Merge "Stores serialized entity data to the extras" into qt-dev

Merge "Fix rollbacks in multi-user case." into qt-dev

Sharesheet - Limit EXTRA_INITIAL_INTENTS to 2

Will eventually merge with suggested apps

Bug: 129979509
Test: ChooserActivityTest
Change-Id: I0e5f5a42155c83c77c6f6e355289218bc88033e7

Fix rollbacks in multi-user case.

There are a few assumptions in the rollback manager that fail in the
multi-user case that need to be fixed:
* getAllSessions only returns sessions for the specific user.
* Session callbacks are only called on sessions associated with the
  registered user.
* getPackageInfo only returns info for the specific user.

Fix these issues so that rollbacks will work properly, in particular
when the initial install session is owned by a non-system user.

Bug: 129809507
Bug: 129397974

Test: On single user device: atest RollbackTest StagedRollbackTest
Test: On primary user of multi-user device: atest RollbackTest StagedRollbackTest
Test: On multi user device manually:
    adb install RollbackTestAppAv1.apk
    adb install --user 10 --enable-rollback RollbackTestAppAv2.apk
    -- verify the install succeeded --
    adb shell pm rollback-app com.android.tests.rollback.testapp.A
    -- verify the rollback succeeded --

Test: On multi user device manually:
    adb install RollbackTestAppAv1.apk
    adb install --staged --user 10 --enable-rollback RollbackTestAppAv2.apk
    adb reboot
    -- verify the install succeeded --
    adb shell pm rollback-app com.android.tests.rollback.testapp.A
    adb reboot
    -- verify the rollback succeeded --

(cherry picked from commit d81ff978667a776884362ef2461b3015c56117d1)

Merged-In: I1a7cf101b3bc3575421629c4bf0ff63418eb8731
Change-Id: I1a7cf101b3bc3575421629c4bf0ff63418eb8731

Merge "Fast follow-on unit tests for HashedStringCache" into qt-dev

Merge "Refactor PackageWatchdog explicit health checks" into qt-dev

Merge "Fix system_server watchdog timeout" into qt-dev

Merge "Fix NetworkStackTests." into qt-dev

Merge "Set sdk_version for PluginCoreLib" into qt-dev

Merge "Add secure setting to cache attention" into qt-dev

Merge "Restore gnss@1.1 HAL capability bits removed in gnss@2.0 (framework)" into qt-dev

Merge "Sharesheet - Fix portrait -> landscape switch" into qt-dev

Merge "Remove unused flags from bubble controller" into qt-dev

Merge "Fix enumerate corner case" into qt-dev

Merge "Update BiometricPrompt default title" into qt-dev

Merge "Add back render-ahead support" into qt-dev

Merge "Add draw props back to greylist" into qt-dev

Merge "Improved the debug message" into qt-dev

Improved the debug message

Improved the log readability by printed out data state
in string format instead of integer. Also added listen
local log.

Bug: 129631771
Test: Manual
Change-Id: If786d51b35380f606bc388e29a441bb24a2792e0

Merge "Adjust the physics params to values decided on in meeting w/ design." into qt-dev

Merge "Update shape for all bubbles" into qt-dev

Merge "Allow the WebView native library to have dependencies." into qt-dev

Merge "Fix mixed up variable in triggering logic" into qt-dev

Merge "notifyEnabledOnKeyguardCallbacks when user switches" into qt-dev

Merge changes from topic "face-profile-qt-dev" into qt-dev

* changes:
  Enroll for the correct user
  Do not updateActiveGroup on a handler
  Launch ConfirmDeviceCredential with actual userId
  Settings should be per-profile

Merge "Rename BluetoothManufacturerInfoReported to BluetoothDeviceInfoReported" into qt-dev

Add secure setting to cache attention

Fixes: 129912479

Test: Builds
Change-Id: I48979c65fc33bb5d70198f549a0ea863257e0247

zygote: initialize ashmem early

avoids dlopen of ashmemd_client in every app startup.

Test: boots, works, doesn't dlopen in app startup
Bug: 129543489
Change-Id: I76ba230af30cd007dbbca054e06660e8a30aca36

Merge "Camera: Clarify camera1 API behavior for getNumberOfCameras" into qt-dev

Merge "media: fix documentation for MediaFormat.KEY_PREPEND_HEADERS_TO_SYNC_FRAMES" into qt-dev

Merge "Change ISE to AssertException, which is more suitable." into qt-dev

Merge "WifiManager: Restore network pin behavior on enableNetwork" into qt-dev

Merge "Fixup App Compaction CL nits." into qt-dev

Merge "Don't show dnd suppressed icons on aod" into qt-dev

Refactor PackageWatchdog explicit health checks

Improvements:

1. Queuing PackageWatchdog requests to startObserving packages:
When observing packages with the watchdog, we needed to get
the packages supporting explicit health checks so we can decide if a
package should be passing or not. This prevents us from receiving
requests to monitor packages during early boot, before third party
packages are ready. In this change we don't depend on ExtServices to
be up to startObserving, we initially treat all package as failing a
health check and lazily syncRequests to request or cancel explicit
health checks based on the currently observed packages. When we receive
onSupportedPackages, we mark the packages that don't support health
checks as passing.

2. Lazy binding to the explicit health check service:
We were always bound to the explicit health check
service regardless of whether we are expecting requests or not, we need
to be able to bind and unbind dynamically to improve device resource
usage. In this change, we bind as soon as we make a request and are
expecting results, we unbind otherwise.

3. Fixed Races:
There were a couple of potential races that could lead to exceptions
that could bring the system server down, e.g when the service is
transitioning between disconnected and connected state (maybe it
crashed) or when ExtServices is being upated and is down or early
boot requests when third party apps are not ready. This change fixes such.

4. Logging:
We improved the logging wording and order and made it more consistent

Bug: 120598832
Test: Manual tests. Stress tested behavior by killing extservices and
making requests simultaneously

function killproc {
    while true
    do
        local pid=$(adb shell pidof $1)
        if [[ ! -z $pid ]]
        then
            echo $pid
            adb shell kill $pid
        fi
    done;
}

adb install-multi-package -i com.android.shell --enable-rollback \
NetworkStack.apk ModuleMetadataGoogle.apk

Also switched between enabled and disabled states to verify packages
are handled correctly. Will automate these tests in later cl

atest PackageWatchdogTest
Change-Id: Iafaef553e95d107f700109f9a8328950a5e2bf71

Merge "Introduce a result class for IStatusBarService#registerStatusBar()" into qt-dev

Update theme icon packs.

Mostly Settings, framework permission group icons and some SystemUI
volume dialog icons.

Bug: 129677355
Bug: 128354921
Test: make; install all overlays and change themes
Change-Id: I7099a050c6e4521e54fd258126e800dab3f66c3e

Merge "Update systemui wifi icons to GM2." into qt-dev

Fix lifecycle issue in CommonPool

Destroy things more faster

Fixes: 129250875
Test: hwui_unit_tests
Change-Id: I7e060fcb61f5321dd9e68a3ee4a01868033b3fc3

notifyEnabledOnKeyguardCallbacks when user switches

The setting is per-user and the source of truth is in BiometricService.
Keyguard's cache needs to be updated when user switch occurs

Fixes: 5268106

Test: Builds
Change-Id: I8a5678375989221e5b502fa683697cf4ca664def

Enroll for the correct user

Bug: 110907543

Test: Builds
Change-Id: I637809cd18d84402918c95d1fc010dacc14f8785

Merge "Fix for NetworkStats/Telephony deadlock" into qt-dev

Rename BluetoothManufacturerInfoReported to BluetoothDeviceInfoReported

* Rename this atom to match the atom's field name

Bug: 112969790
Test: make
Change-Id: Idb54766703f47a5eb6cd7f0cdff1c41e1a92cfb1

Fix mixed up variable in triggering logic

When this code got refactored the check for severe warnings
accidentally had the time remaining being compared against the percentage
value of the severe warning threshold. Unfortunately this meant that
it was impossible to mark the severe level warning as shown and could
result in multiple triggerings. This change makes it so everything
is compared with the appropriate type of value again.

Test: PowerUI Tests pass
Bug: 129730217
Change-Id: Ic8bdabf1c70e49d117210bb80f1ca0a0d7684617

Adjust the physics params to values decided on in meeting w/ design.

Fixes: 129981373
Test: manual
Change-Id: Idfa840d6aade8ac975bd89bb0ba155f21e9f3ed8

Update shape for all bubbles

After theme change.

Test: manual
Bug: 123829494
Change-Id: I9eca25f19646607d85957f5113178619b7927dfe

Update systemui wifi icons to GM2.

Remove unneeded NeutralGoodDrawable.

Bug: 124106839
Test: make; used sysui demo mode
Change-Id: Ib3e903d83be140b00410e3adbb31f217b562d372

Change ISE to AssertException, which is more suitable.

Bug: 129139241
Bug: 129721058
Test: Treehugger
Change-Id: I12e97f37a58e1d88731a9bddd933abeb8c1169c1

Support profileable application action

Fixes: 129358330
Test: make -j16 aapt2_run_host_unit_tests && cat out/target/product/walleye/obj/PACKAGING/aapt2_run_host_unit_tests_intermediates/result.xml | grep "failure message"
Change-Id: Id9ec59447a99a5f6e0bfe47c0b8ce3553bf751c7

Fix NetworkStackTests.

NetworkStackTests dlopens library which indirectly
uses libcgrouprc. Add it to jni_libs.

Test: atest
Fixes: 129902619
Change-Id: I0d60d7a317fa60607d6fdaea15f61b458aefd2f0
Merged-In: I0d60d7a317fa60607d6fdaea15f61b458aefd2f0

Do not updateActiveGroup on a handler

This method must not return to the caller until complete, otherwise
weird race conditions may occur.

Bug: 110907543

Test: Builds
Change-Id: Ia82671b34fac3662928012d8c709143efdf92266

Launch ConfirmDeviceCredential with actual userId

Fixes: 129905318
Bug: 110907543

Test: Set up work profile, use BiometricPromptDemo to allow device
      credentials. Work profile demo app now requests work profile
      password.

Change-Id: Iec53966c6ad87021fdaa1808a1814a4abfbf89bb

Settings should be per-profile

Fixes: 129905061

Test: Builds
Change-Id: If1b3e92da61c051c5f38a425da17f98cdfc3336d

Catch ArrayIndexOutOfBoundsException in StateMachine.toString()

StateMachine.toString() calls mSmHandler.getCurrentState(). In the
rare case when mStateStackTopIndex is -1 due to the StateMachine being
not yet initalized, this results in an ArrayIndexOutOfBoundsException.

Bug: 124858416
Test: none

Change-Id: Id558be95b6e12d8a923d325bca5f6f6d7f942b22

media: fix documentation for MediaFormat.KEY_PREPEND_HEADERS_TO_SYNC_FRAMES

Bug: 32746212
Test: builds
Change-Id: I38ecfa879091bc4016fbae04eab77ed6fd9eb1c1

Sharesheet - Fix portrait -> landscape switch

Calculating the reserved height was using the wrong figures. Expose
the correct reserved alwaysShow height, and use it to determine the
offset for scrolling.

Bug: 120419081
Test: manual, rotate device
Change-Id: I21fd9fa8494a258d17f6002f79967aacc0efde34

make libmedia_jni_utils depends on libgui

libmedia_jni_utils only has dependencies on libgui,
not entire libmedia_omx, it's linking to libmedia_omx
to get to libgui/bufferqueue headers.

bug: 129272021
test: builds

Change-Id: I48453cb53d36e6ecb2a6f6dd2b09f9a99e8479f0
(cherry picked from commit a527e6d94b77b74545e09bd617b0060e2075eee2)

Fast follow-on unit tests for HashedStringCache

Unit tests for HashedStringCache that was commited in earlier CL
ag/6867725 . This is testing the various inputs and expected outputs.
Testing also revealed some vulnerability for invalid input so added
validation in the code under test.

Bug: b/129870147
Test: This is the test file
Change-Id: I7387f808df87a869f81339cd4aea99b23dfc06bd

Set sdk_version for PluginCoreLib

Bug: 129423457
Test: m Launcher3 && m Launcher3QuickStep

Change-Id: Iea69efb67d4aec124f29eb3cc956b96f5d06f1e7

Introduce a result class for IStatusBarService#registerStatusBar()

This is a preparation to fix an issue that IME cannot enable light
navigation bar on direct-reply.

The initial state of StatusBar client is provided as return values of
IStatusBarService#registerStatusBar().  However, the way for the
client to receive the return values is to rely on "out" parameters in
the Binder method as follows.

    void registerStatusBar(IStatusBar callbacks,
            out List<String> iconSlots,
            out List<StatusBarIcon> iconList,
            out int[] switches,
            out List<IBinder> binders,
            out Rect fullscreenStackBounds,
            out Rect dockedStackBounds);

The issue is that if I want to add a new parameter into
StatusBarManagerService to fix Bug 122439339, then it needs to be
notified back to the client via one of these out values, e.g. a new
element in "switches", which is hard to understand and maintain.

It'd be better to introduce a dedicated return object as follows.

    RegisterStatusBarResult registerStatusBar(IStatusBar callbacks);

This is a purely mechanical code clean-up.  Hence there should be no
behavior change.

Bug: 122439339
Test: presubmit
Change-Id: Ib1c0ae8f591ca09d0bce7a39f85ba57aad386e47

wifi.proto: Add link probe event to StaEvent

Add link probe event type to StaEvent in order to correlate link probe
success/failure with other StaEvents (e.g. disconnection, score breach).

Bug: 112029045
Test: compiles
Change-Id: I2a18abc6f19179b164cd4c007597ba6cb69bf889

fix nullability

Bug: 126701951
Test: It builds, it runs
Merged-In: I43e02551ced8a6aa51b84dd8df7e57e75e1d58a9
Change-Id: I43e02551ced8a6aa51b84dd8df7e57e75e1d58a9

Revert "Revert "Mute haptic channels by default.""

This reverts commit e6243996efb90d6e9dd0b855aa47c8795cf4ba9f.

Bug: 129721367
Test: change volume in Play Music
(cherry picked from commit ab3a3f8c02503f8a53a14f6b3f413e0cf739c957)

Change-Id: Id6e63e87265717130dde0e1b46e28edbb0b53edb

fix AudioAttributes matching rule in AudioProductStrategy

The matching rule on audio attributes flags must be that ALL flags
set in the reference attributes are present in the compared attributes
and not just some flags.

Bug: 129721367
Test: change volume in Play Music
Change-Id: I462fd73fb9b78b094411c6f3b52004349f49bcf3
(cherry picked from commit 9b0bb840cd082822487e6cfe8c4470ce928d7531)

Add draw props back to greylist

Bug: 129957634
Test: none
Change-Id: Iaf2d7f8dc0f5d6f1b0dc12025210820bf8255f67

Expose notification guts on menu shown

This change adds a few methods to NotificationMenuRowPlugin so that a
menu row can tell the stack scroller to expose the notification guts
when the menu is fully exposed.

TODO: fix the menu close animation so it happens after the guts open,
and if you close the guts by swiping you can trigger a flicker.

Also there are some things to make lint happy and lots of null checks.

Bug: 127998765
Test: swipe RTL on any notification
Change-Id: I46f7c7dc90032a1914f7b5513bc57e089887c722
(cherry picked from commit 9272fd2f57e57475cb7ce9967e60946d0770d879)

Add back render-ahead support

Currently only supported in the EGL path.
Vulkan support Coming Soon

Bug: 127822449
Test: trace of hwuimacro
Change-Id: Iac2b039e11d964aab5b8ca1bdf2a5430b187e2ea

Remove unused flags from bubble controller

- ENABLE_BUBBLES_AT_TOP (default false)
- ENABLE_BUBBLE_FOOTER (default false)

Test: manual
Bug: 129546118
Change-Id: I6aaa33975d65fb20ba9b82a378e00c2100fd3164

Update BiometricPrompt default title

Fixes: 117466084

Test: manual

Change-Id: I3b82396694a26d63a79334e7e86c4e9121a370f6
Merged-In: I3b82396694a26d63a79334e7e86c4e9121a370f6
(cherry picked from commit 87fbeab12f708b3e987f89ad0c547f2315baac2a)

Only switch task focus to window's task that was tapped outside current focus

The previous logic would check what Task was at the coordinates that the event
receiver got. This was an issue if a window that was tapped didn't have a Task,
for example, system windows. The Task focus would get switched to the Task that
was behind the tapped window.

This change leverages the new API onPointerDownOutsideFocusLocked. When a down
outside focus occurs, WindowManager will check if the window has a Task + the
check that existed previously, before determining whether the new Task should
get focus.

Fixes: 129571534
Test: Steps from bug
Test: Resize in freeform continues to work.
Change-Id: I73286e03f505cdf37f30fa5914cf24f843a6bc7c

Fix enumerate corner case

Do not post reset runnable when another client is requested while
cleanup is happening. The runnable will be "pending" and started once
cleanup finishes instead.

Fixes: 129534163

Test: Based on logs in the bug, this is reasonable
Test: reboot device, fingerprint still works
Change-Id: I766cc04be151abaae60a0092f71fd7a3acf3397a
Merged-In: I766cc04be151abaae60a0092f71fd7a3acf3397a
(cherry picked from commit ea80d17cb65e69457e169b0595094893ebabd494)

Fix isExternalStorageSandboxed() bug.

It needs to use the same STOPSHIP logic for consistency with the
two other enforcement sites across the OS.

Bug: 129487770
Test: atest android.appsecurity.cts.ExternalStorageHostTest
Change-Id: I7a3fa836e0795912c264aae58a55472ffae3d8c3

Restore gnss@1.1 HAL capability bits removed in gnss@2.0 (framework)

In the IGnssCallback.hal@2.0 introduced in Android Q, the
capability bits in IGnssCallback.hal@1.1 that represent sub-HAL
interfaces have been removed as they are derivable from the
existing getExtensionXXX() family of methods in the IGnss.hal
interface.

These need to be restored back as the synchronous nature of the
getExtensionXXX() methods called by the framework has an impact on
partner implementations that need to communicate with the modem to
get the capabilities.

Additionally, the capability bit MEASUREMENT_CORRECTIONS needs to be
added for the new optional measurement_corrections@1.0 sub-HAL
introduced in gnss@2.0.

Fixes: 129870126
Test: Verified through cuttlefish default implementation and VTS tests.
Change-Id: I30c34361882e0722240feb3deba7d80136cd4bb3

EGLAttrib requires special handling in JNI

EGLAttrib is defined for a C / C++ interface and is
intended to be the size of a pointer. That can change
depending on which abi the code is built for but Java doesn't
have a way of dealing with such types. Java defines EGLAttrib
as a jlong, so need to convert jlong to 32bit when running on
a 32bit device.

Bug: 124382141
Test: atest --all-abi CtsGraphicsTestCases:EGL15Test
Change-Id: I966f6a3716b2710e17d10e0d0cb58314853675e0