Fix index error

Change-Id: I93cf18b35060a09a3b8f4be8e0b9b4682d390b25
Fixes: 76101440
Test: runtest systemui-notification

Merge "Document Job Scheduler internal "calling" vs "source" package" into pi-dev

Merge "Ignore non-existent displays in #topRunningActivityLocked." into pi-dev

Merge "Added Locking for MmTelFeature" into pi-dev

Merge "Fix issue with slippery state for quickstep button case." into pi-dev

Merge "Don't add stopping apps in relaunch based on stopped flag." into pi-dev

Merge "Add metric ACTION_ANOMALY_TRIGGERED" into pi-dev

Merge "Renamed config_dozeAfterScreenOff" into pi-dev

Merge "Update material_deep_teal_500 to #008577." into pi-dev

Merge "Category message is sufficient for message notif" into pi-dev

Merge "Preserve letterbox while dragResizing" into pi-dev

Merge "Include additional information in the dumpsys" into pi-dev

Merge "WebView Tracing API: remove LARGE_BUFFER, make RECORD_CONTINUOUSLY the default mode" into pi-dev

WebView Tracing API: remove LARGE_BUFFER, make RECORD_CONTINUOUSLY the default mode

This patch:
- remove the RECORD_UNTIL_FULL_LARGE_BUFFER tracing option, because
  this does not appear really useful in the production setting as
  the buffer size limit exceeds the device memory and can grow
  uncontrollably large.
- make RECORD_CONTINUOUSLY the default mode for tracing, to make
  sure the least memory is used by default (typically 4x less than the
  previous default RECORD_UNTIL_FULL).
- add some missing IntDef values (to make them match with the constants)

BUG: 63750258
Test: CTS

Change-Id: Ib3698273dee776ebc7a8388fc1bd8c80255e3b63

Merge "Make autofill save UI RTL compliant" into pi-dev

Merge "Some Slices API refinement" into pi-dev

Merge "Added implementation for VTI add/remove address" into pi-dev

Merge "Add support for auth-only transforms" into pi-dev

Merge "Merge transactions to parent if layer is going to be removed." into pi-dev

Merge "DisplayCutout: Add back always resource" into pi-dev

Merge "Report native allocation size of AnimatedImageDrawable" into pi-dev

Merge "Add new hidden API uses to light greylist" into pi-dev

Merge "Get rid of upperbound on wireless charging anim" into pi-dev

Merge "Update logging util to not crash on null" into pi-dev

Ignore non-existent displays in #topRunningActivityLocked.

Previous changes aimed at driving display creation through
ActivityManagerService. However, there are still cases where
WindowManagerService will encounter the display first (such as add
window). A subsequent changelist enabled this out of band creation.
As a result the WindowManagerService and ActivityManagerService can
still have different lists of displays. This mismatch leads to
#topRunningActivityLocked processing null ActivityDisplays.

This changelist skips over displays not yet encountered by
ActivityManagerService in #topRunningActivityLocked. Such displays
have no stacks and therefore no activities that could be the top
running activity.

Test: atest FrameworksServicesTests:com.android.server.am.ActivityStackSupervisorTests#testTopRunningActivityLockedWithNonExistentDisplay
Fixes: 74593710
Change-Id: I316e5305f7ce233a04ecf18d605eebdf723b0920

Merge "UsageEvents for slices pinning" into pi-dev

Merge "Ensure rotate disable2 flag can be set on boot" into pi-dev

Merge "Update IME/keyboard switcher navbar icon to GM" into pi-dev

Merge "Distinguish between bringing stack to front and delivering to top." into pi-dev

Merge "Add unit test for ActivityStackSupervisor#testApplySleepTokensLocked." into pi-dev

Add new hidden API uses to light greylist

From common apps.

Bug: 64382372
Bug: 74430927
Test: make
Change-Id: I81c38a78fcdc7c33f54cd25823b301ba29638019

Merge "Restore state if password change fails" into pi-dev

Merge "Update QS customizer add tiles string." into pi-dev

Merge "Intercept touches in the emulated DisplayCutout" into pi-dev

Merge "DO NOT MERGE Update references to backup/restore agent timeouts" into pi-dev

Merge "More APIs on hidden API light greylist / vendor list" into pi-dev

Merge "Rename persist.traced.enable to sys.traced.enable_override" into pi-dev

Merge "Animate clock after unlocking" into pi-dev

Category message is sufficient for message notif

If a notification is marked with category message, then
it is sufficient enough to be deemed a "message" notification.
However, to be considered an important message, we still
check if the message is from the default messaging app and has
category = message.

Change-Id: I4f2b502634b805919bdf8b82e3bdf475c0992bdd
Fixes:76019310
Test: AttentionManagementVerifierActivity
Test: atest services/tests/uiservicestests/src/com/android/server/notification/NotificationComparatorTest.java

Get rid of upperbound on wireless charging anim

Previously, if the battery percentage was above 95%, then
wireless charging wouldn't turn the screen on nor play the
wireless charging animation. Now the charging
animation will trigger even if battery percentage is above 95%.

Test: manual
Change-Id: I531334548d41cba8a1a4c28dff484d3f2dfefe01
Fixes: 75029514

More APIs on hidden API light greylist / vendor list

New scan after updates.

Bug: 64382372
Bug: 76062566
Test: make
Change-Id: Ia2c67222bddccb0b7f607ee0ce8951611a668dc3

UsageEvents for slices pinning

Test: atest frameworks/base/services/tests/servicestests
Bug: 73455786
Change-Id: I6a37f5525c29d3f47aa37e262c8834840309ba44

DisplayCutout: Add back always resource

Temporarily adds back "always" value for layoutInDisplayCutoutMode
until all usages have been migrated to "shortEdges".

Bug: 74983661
Test: make droid
Change-Id: I27189d17129a8dd6e5e9ac3aaf997967df21e8ef

Rename persist.traced.enable to sys.traced.enable_override

Allowing for both a default in the system
image and an override form the server requires at least two properties.
Given that we're using two properties anyway we can change the
server override to match the pattern of the other flags and not be
a persisted setting.

Test: Manual.
Bug: b/76077784
Merged-In: I98932085430cdeaf4ccd036a0751f119c0cdeb3c
Change-Id: I98932085430cdeaf4ccd036a0751f119c0cdeb3c

Merge "Correct exiting pip animation state." into pi-dev

Report native allocation size of AnimatedImageDrawable

Bug: 73641604
Test: infeasible

Fix nNativeByteSize's return value to be jlong, instead of long.
Add up the bytes used by the SkAnimatedImage and SkPictures and store
them on the AnimatedImageDrawable for registration.
Note that this is an approximation, and it assumes it will be drawn to a
hardware canvas and animated.

Merge "Amend package whitelist." into pi-dev

Merge "DisplayCutout: make dock divider cutout aware" into pi-dev

Preserve letterbox while dragResizing

Fixes an issue where the letterbox would disappear while entering
and leaving dragResizing, because this clears the hasDrawnState.

To fix this, also treat preserved surfaces as ready to show for
letterboxing purposes.

Change-Id: I4d6e1b1a27d99542370fde8924f126eca1ea80d5
Fixes: 76008154
Test: Enable double cutout, enter split screen, go home, touch slider verify the letterbox does not flicker.

Merge "Add RecoverySession importKeyChainSnapshot method" into pi-dev

Merge "Use the new root cert file under the core/ folder" into pi-dev

Restore state if password change fails

Bug: 73299306
Fixes: 73299306
Test: Call LockPatternUtils.clearLock() with wrong password,
      make sure device still unlocks after reboot
Change-Id: I640fc62cbe0c0c57e980455d4f499df02dee0832

Merge "Add API WebView.getWebViewLooper()" into pi-dev

Update QS customizer add tiles string.

Change-Id: Iddea2dd8bd48b980e083fac447e905616031b365
Fixes: 27961533
Test: visual

Merge "Enable hidden API blacklist" into pi-dev

Merge "Add hidden API uses to light greylist" into pi-dev

Add RecoverySession importKeyChainSnapshot method

This imports the keys directly into the keystore of LockSettingsService,
allowing them to be accessed via the RecoveryController getKey method.
This is better as it does not expose raw key material to any app.

Bug: 74345822
Test: runtest frameworks-services -p \
      com.android.server.locksettings.recoverablekeystore

Change-Id: I4991b0cff1d2fa2e5bd0b53a71c096499e93e98b

Merge "Temporarily whitelist platform test packages for hidden API" into pi-dev

Amend package whitelist.

Now that Antons cleanup of all packages is complete, we can generate this
whitelist from those packages that specify

  LOCAL_PRIVATE_PLATFORM_APIS := true

This is the resulting list of packages. This change also includes fixes to
the method; previously, packges that didn't specify any certificate may
have been exluded.

Test: Verify that device boots.
This is a cherry-pick of change I1f578322135274b80708d4bb7664f7732ac33cc7
from AOSP.

Bug: 64382372
Change-Id: Iada043cb85f9b3281893ed9a2828771b8f1ef045

Merge "Add new Intent start a package's Open By Default page directly." into pi-dev

Use the new root cert file under the core/ folder

This CL also adds an alias param to the RecoverySession#start method.

Bug: 76033708
Test: runtest frameworks-services -p \
      com.android.server.locksettings.recoverablekeystore

Change-Id: I870f4f89bd6e319e1687a981aa04af0d23f3c922

Merge "Allow statsd to be given empty config." into pi-dev

Merge "Create task record with window container controller when adding task" into pi-dev

Merge "Stop creating PrecomputedText in StaticLayout" into pi-dev

Merge changes from topic "SoundTriggerJobService-steps" into pi-dev

* changes:
  Throttle number of sound trigger operations / day
  Add SoundTriggerDetectionService
  Expose SoundTriggerManager and SoundTrigger APIs

Merge "Fix confusing Log.e message in SpellCheckerSession" into pi-dev

Merge "Screen off animation" into pi-dev

Merge "Get current and default brightness configs." into pi-dev

Animate clock after unlocking

Change-Id: I660e493e3909bf74a9d6b8f60009899ebf5dffe5
Fixes: 74586970
Test: visual

Merge "Fix docs for LocationRequest#setLowPowerMode()" into pi-dev

Merge "Use AsyncTask.SERIAL_EXECUTOR as the default executor for scan" into pi-dev

Stop creating PrecomputedText in StaticLayout

No performance regressions are expected

android.text.PrecomputedTextPerfTest:
  create NoStyled Hyphenation                  : 17,849,966 -> 17,858,570: (+0.0%)
  create NoStyled Hyphenation WidthOnly        : 17,814,338 -> 17,801,205: (-0.1%)
  create NoStyled NoHyphenation                :  7,123,449 ->  7,068,308: (-0.8%)
  create NoStyled NoHyphenation WidthOnly      :  7,108,169 ->  7,074,908: (-0.5%)
  create Styled Hyphenation                    : 12,179,203 -> 12,131,020: (-0.4%)
  create Styled Hyphenation WidthOnly          : 12,112,347 -> 12,241,311: (+1.1%)
  create Styled NoHyphenation                  : 11,870,126 -> 11,880,442: (+0.1%)
  create Styled NoHyphenation WidthOnly        : 11,836,742 -> 11,860,292: (+0.2%)

android.text.StaticLayoutPerfTest:
  create PrecomputedText Balanced Hyphenation  :    697,713 ->    691,148: (-0.9%)
  create PrecomputedText Balanced NoHyphenation:    517,113 ->    498,106: (-3.7%)
  create PrecomputedText Greedy Hyphenation    :    468,243 ->    455,015: (-2.8%)
  create PrecomputedText Greedy NoHyphenation  :    479,514 ->    461,617: (-3.7%)
  create RandomText Balanced Hyphenation       : 17,183,044 -> 17,049,811: (-0.8%)
  create RandomText Balanced NoHyphenation     :  7,183,745 ->  7,025,070: (-2.2%)
  create RandomText Greedy Hyphenation         :  7,130,841 ->  6,995,785: (-1.9%)
  create RandomText Greedy NoHyphenation       :  7,122,398 ->  7,037,074: (-1.2%)

  draw PrecomputedText NoStyled                :    520,306 ->    551,465: (+6.0%)
  draw PrecomputedText NoStyled WithoutCache   :    545,773 ->    566,956: (+3.9%)
  draw PrecomputedText Styled                  :    826,044 ->    838,979: (+1.6%)
  draw PrecomputedText Styled WithoutCache     :    829,958 ->    841,749: (+1.4%)
  draw RandomText NoStyled                     :    537,079 ->    545,428: (+1.6%)
  draw RandomText NoStyled WithoutCache        :  6,473,166 ->  6,445,194: (-0.4%)
  draw RandomText Styled                       :    995,033 ->  1,015,913: (+2.1%)
  draw RandomText Styled WithoutCache          :  2,725,313 ->  2,770,604: (+1.7%)

android.widget.TextViewPrecomputedTextPerfTest:
  newLayout PrecomputedText                    :    754,311 ->    718,130: (-4.8%)
  newLayout PrecomputedText Selectable         : 17,716,239 -> 17,484,046: (-1.3%)
  newLayout RandomText                         : 16,657,952 -> 16,511,625: (-0.9%)
  newLayout RandomText Selectable              : 17,675,222 -> 17,520,653: (-0.9%)
  onDraw PrecomputedText                       :  1,307,123 ->  1,280,009: (-2.1%)
  onDraw PrecomputedText Selectable            : 17,613,031 -> 17,404,379: (-1.2%)
  onDraw RandomText                            : 17,369,256 -> 17,295,363: (-0.4%)
  onDraw RandomText Selectable                 : 18,207,392 -> 18,077,660: (-0.7%)
  onMeasure PrecomputedText                    :    748,537 ->    739,128: (-1.3%)
  onMeasure PrecomputedText Selectable         : 17,842,953 -> 17,784,459: (-0.3%)
  onMeasure RandomText                         : 16,633,454 -> 16,549,182: (-0.5%)
  onMeasure RandomText Selectable              : 18,022,286 -> 17,873,919: (-0.8%)
  setText PrecomputedText                      :    120,769 ->    119,496: (-1.1%)
  setText PrecomputedText Selectable           :    162,411 ->    150,809: (-7.1%)
  setText RandomText                           :     11,096 ->     10,956: (-1.3%)
  setText RandomText Selectable                :     48,852 ->     48,593: (-0.5%)

Bug: 72998298
Test: atest CtsWidgetTestCases:EditTextTest
    CtsWidgetTestCases:TextViewFadingEdgeTest
    FrameworksCoreTests:TextViewFallbackLineSpacingTest
    FrameworksCoreTests:TextViewTest FrameworksCoreTests:TypefaceTest
    CtsGraphicsTestCases:TypefaceTest CtsWidgetTestCases:TextViewTest
    CtsTextTestCases FrameworksCoreTests:android.text
    CtsWidgetTestCases:TextViewPrecomputedTextTest

Change-Id: I3af758ecc5a15975c4e59c6378faf7c14c3bd65b

Include additional information in the dumpsys

Test: adb shell dumpsys dbinfo system
Bug: 64262688
Change-Id: I43d76e505e4d5598994c9af3e615c572017f120c

Fix confusing Log.e message in SpellCheckerSession

With this CL, calling SpellCheckerSession#close() multiple times will
no longer show the following error message in logcat, which was quite
confusing.

  E SpellCheckerSession: ignoring processOrEnqueueTask due to
    unexpected mState=TASK_CLOSE scp.mWhat=TASK_CLOSE

Note that with a recent CL [1], we now more often see the above
message.  A typical scenario is:

  1. App creates SpellCheckerSession x.
  2. App calls x.close().
  3. x is garbage-collected. ART calls x.finalize(), which internally
     calls x.close() again.

 [1]: I4e00c3a2cec93d1dacff20546e481fe757279661
      9b64367193ffb252f869fb9f65a60b51a654119e

Bug: 72974646
Fix: 72974646
Test: Made sure that no error message in question with a test app
      that calls SpellCheckerSession#close() multiple times then
      trigger SpellCheckerSession#finalize().
Change-Id: Ie9690860e6d0406dc6746cd03c28f693e65c1dde

Merge "Disable app_stacks and last_kmsg until they get selinux permissions." into pi-dev

Merge "heif: address api review comments" into pi-dev

Merge changes from topic "heads_up_redesign" into pi-dev

* changes:
  Added new appear and disappear animations for heads up
  Polished the heads up experience
  Removed the heads up scrim and replaced it with more elevation
  Insetting heads up notifications
  Ensured that the heads-up notifications are always rounded

Fix issue with slippery state for quickstep button case.

Bug: 76033653
Test: Swipe up with quickstep fallback activity
Change-Id: I417b9f2fadeb87cbf2daac57a53e7ff9990335a0

Merge "Distinguish between createSurface error codes." into pi-dev

Merge "Move the trusted root CA certs for RecoverableKeyStore to our own folder" into pi-dev

Merge "Fix recovery of stats data from previous input while using ProtoOutputStream" into pi-dev

Merge "API Review: Documentation improvement for standby buckets" into pi-dev

Merge "GPS metrics collection" into pi-dev

Merge "Introduce new constructor for not copying NoCopySpan" into pi-dev

Fix docs for LocationRequest#setLowPowerMode()

Bug: 74218111
Change-Id: Ib77a58c21e2cc3601221c8b5cc13eb416dccfbc9
Fixes: 74218111
Test: N/A

Merge "Fingerprint vibrate doesn't change based on DND" into pi-dev

DO NOT MERGE Update references to backup/restore agent timeouts

With the newly created Global setting for backup/restore agent timeouts
(backup_agent_timeout_parameters introduced in ag/3731401), update
references in backup and restore code to get the timeout values from
the setting instead of referencing constants in BMS.
This makes these timeouts configurable outside of the framework.

The default value of the setting is the same as the constants in BMS so
this should have no change on the timeout values used in the backup and
restore flow.

Bug: 70276070
Test: 1) m -j RunFrameworksServicesRoboTests
Change-Id: I0259bba76d0fed48158b8316f430b315ea98086e

Don't add stopping apps in relaunch based on stopped flag.

The activity was getting added to the list of activities that should be
moved to stopped. This was causing the window to get removed and then
re-added, which is where the flicker was coming from.

The check to add stopping apps when the flag r.stopped is set was a
fix for a lifecycle issue with relaunch. However, this issue is now
fixed without this. The stopped flag is not set appropriately because
activities that are paused can still be considered stopped if they never
went through completeResumeLocked, like in multi-window.

Change-Id: I2e49bfa031caa189cc1e440749d0a22b30b76792
Fixes: 75276931
Test: Tested original issue to make sure it still works from b/30060825
Test: Split screen no longer flickers when moving divider

Merge "Fingerprint navigation settings is a secure setting" into pi-dev

Merge "Ability to hide/unhide suspended app notifications" into pi-dev

Merge "Better javadoc and rename several methods in ApnSetting." into pi-dev

Merge "Check if the given user is unlocked when unlock by token" into pi-dev

Merge "Add alpha Strings to CellIdentityTdscdma" into pi-dev

Merge "Fix pulling interval may become 0 in cts" into pi-dev

Merge "Revert "Dispatch global settings changes to all users."" into pi-dev

Renamed config_dozeAfterScreenOff

New name is config_dozeAfterScreenOffByDefault
Bug: 73178121
Test: make

Change-Id: I6633d995e93c91a36aa5b4b153ff924964cb3595

Revert "Dispatch global settings changes to all users."

This reverts commit 989157e1a6ae6049769a6d02af518cf9cd21bd79.

Reason for revert: Crashing stuff -- http://b/76031092

Bug: 76031092
Change-Id: I190280b4f7bd2c6fda5d88fe25fd609d1cf6d7fb

Add metric ACTION_ANOMALY_TRIGGERED

Bug: 73888115
Test: Build
Change-Id: I519f280536508c18a923c2adc8e2d8f8bdceae34

Added implementation for VTI add/remove address

This change adds implementation details for add/remove addresses onto a
VTI.

Bug: 73675031
Test: New tests added, passing on Walleye
Merged-In: Idde9d943a5285d2c13c5c6b0f7b8a9faf718e6a5
Change-Id: Idde9d943a5285d2c13c5c6b0f7b8a9faf718e6a5
(cherry picked from commit ecc9f7cc08804e3fa15fea04ae94ea1bc74edbfe)

Disable app_stacks and last_kmsg until they get selinux permissions.

Bug: 73487276
Test: atest CtsIncidentHostTestCases:com.android.server.cts.IncidentdIsolatedTest
and atest CtsIncidentHostTestCases:com.android.server.cts.IncidentdTest

Change-Id: Ifc0813d59c243a1b4afa150bbcfa4b072a044ffc