Merge "ScreenDecor: Fix cutout overlay regression" into qt-dev

Merge "Stub the oemlock operations when the PersistentDataBlock is not supported" into qt-dev

Merge "AccessibilityNodeIdManager should hold onto weak references of Views." into qt-dev

AccessibilityNodeIdManager should hold onto weak references of Views.

... as opposed to strong references.
In case the calls between register and unregister aren't matched, we don't want to be leaking views.
I haven't seen any real eveidence of there being leaks, just a report. But this is preventative.

Created a "WeakSparseArray", which wraps SparseArray, and passes in a kind of WeakReference that has an id.
The references become unreachable, we use the id to remove the the entry from the SparseArray.

Test: Used the device for a bit with talkback on. CTSAccessibility*
Change-Id: I8d11727428f56fc06007232ae341d1409a11991f
Fix: 134506015

Merge "Try augmented autofill if regular autofill is disabled." into qt-dev

Merge "Fix NPE in removeGeofence" into qt-dev

Merge "Correct sequencing of widget vs boot-complete broadcasts" into qt-dev

Try augmented autofill if regular autofill is disabled.

Fixes: 134914712
Test: atest android.autofillservice.cts.augmented
Change-Id: I7440bc4a9bff0b62b2625cd5bf24d3e276e4802c

Merge "Show notifications when default launcher changes" into qt-dev

Merge "Fixes two data races in USAP pool management." into qt-dev

Merge "Made AccessPoint's scan result list thread safe" into qt-dev

Merge "Make PiP window visible when user launches PiP in landscape mode" into qt-dev

Merge "Guard against null mFunc in PooledLambdaImpl" into qt-dev

Show notifications when default launcher changes

Shows a notification when we have to switch away from Gesture Nav
because the new launcher doesn't support it.

Also shows another notification when previously we had to switch away
from Gesture Nav, but now the new launcher supports it.

Bug: 129532605
Test: Manual test
Change-Id: Ie954dd5dea1b45795f5c0e22ab7cc1cffde4aabe

Merge "Expose PermissionManager as TestApi and fix SplitPermissionInfo equality" into qt-dev

Merge "docs: Fixing broken HTML" into qt-dev

Fix NPE in removeGeofence

Check if geofence is null before reading getRadius()

Test: run cts-dev --module CtsLocationTestCases --test android.location.cts.LocationManagerTest
============================================
================= Results ==================
=============== Consumed Time ==============
    arm64-v8a CtsLocationTestCases: 1m 21s
Total aggregated tests run time: 1m 21s
============== TOP 1 Slow Modules ==============
    arm64-v8a CtsLocationTestCases: 0.29 tests/sec [24 tests / 81579 msec]
============== Modules Preparation Times ==============
    arm64-v8a CtsLocationTestCases => prep = 4604 ms || clean = 750 ms
Total preparation time: 4s  ||  Total tear down time: 750 ms
=======================================================
=============== Summary ===============
Total Run time: 1m 28s
1/1 modules completed
Total Tests       : 24
PASSED            : 24
FAILED            : 0
============== End of Results ==============
============================================

Bug: 134894083
Change-Id: I71f643688855394e14974170bd2380f66cd51308

Merge "Sharesheet - Hide direct share area for low ram devices" into qt-dev

Merge "Inject dependencies instead of fetching them" into qt-dev

Merge "Add timeout for setPermissionGrantState" into qt-dev

Merge "Sharesheet - Adjust timeouts and logic for direct share" into qt-dev

Sharesheet - Hide direct share area for low ram devices

Make sure to force the item count to 0 to make sure the direct share
area stays hidden.

Bug: 134776740
Test: Low ram device, manual
Change-Id: I598752db5747b1c845495af7ae2f591488799352

Sharesheet - Adjust timeouts and logic for direct share

Improve direct share loading by:
1. Do not issue redundant service calls within the watchdog timeout
period
2. Set a min and max watchdog timeout period, and if all services
return early, then informt the user we are done to so that they don't
have to wait until the max timeout period.
3. Be more specific on package changes. Resolver requires any change
to trigger a refresh, but chooser only needs to know about top level
package changes.

Bug: 134416594
Test: Manual
Change-Id: I075fd5b6f64c25637fd7ba7aa6b163dca81371a8

Merge "Ignore brightness change thresholds for manual changes." into qt-dev

Inject dependencies instead of fetching them

Bug: 134712444
Test: atest SystemUITests
Test: atest NotificationStackScrollLayoutTest (wasn't actually able to
      reproduce locally)

Change-Id: Ic1478a3eca164b844ba5721014e5591a6af02ffa

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ida0624dfdb53a688b75bd7e03a7c02e32047ded3

Merge "Import translations. DO NOT MERGE" into qt-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I415dad40c35ad89a25d4df9190742a3515764a27

Merge "Persist metric activation across beta4/5 ota" into qt-dev

Merge "Import translations. DO NOT MERGE" into qt-dev

Merge "Remove obsolete state" into qt-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I8ce265c5d257e5cdbc247dc459a2432713f1c7b6

Remove obsolete state

There isn't a fingerprint animation, we don't need the timeout anymore.
This event was triggering unwanted updates, cancelling animations
during wake-up.

Fixes: 134448130
Test: power to wake-up: observe animation
Test: fingerprint from bouncer: observe animation
Test: fingerprint to wake-up: no animation
Change-Id: I17fcd9eb5eba7e7bc88de09309f5c310fe13433f
Merged-In: I17fcd9eb5eba7e7bc88de09309f5c310fe13433f

Merge "Do not animate status bar icons outside of shade" into qt-dev

Persist metric activation across beta4/5 ota

In beta 4, activeEventActivation proto object does not have a state
field and assumed to be active if it is written to disk. In beta 5, we
add a state field to support persisting metric activation status across
a system server crash. However, if a device updated from beta 4 to 5, we
don't have any knowledge of the state when we read. Therefore, assume
the lack of the state field as the event activation should be active for
the provided ttl.

Bug: 134795027
Test: used flashstation to flash a device to beta 4. Then manually push
and activate a simple config. Then use vendor/google/tools/flashall to
update to qt-dev + this change. Manually ensure metric/config are still
active.
Test: gts-tradefed run gts-dev --module GtsStatsdHostTestCases
Test: bit statsd_test:*

Change-Id: I8594d04efaa2ae1f669113aaffd33e5fa5851e73

Correct sequencing of widget vs boot-complete broadcasts

Past fixes to lock-management issues inadvertently altered the intended
sequencing of the widget-lifecyle and boot-completed broadcasts.  It's
correct now, with the order guaranteed by thread-of-issuance affinity.

Along the way, add the one missing bit of state reporting to the
plaintext broadcast queue dumpsys infrastructure.

Fixes: 131409711
Test: manual - boot and unlock device when widgets are being hosted by
      the home app, and observe the timings & broadcast history.

Change-Id: Iaee75d4f9f85565eab95976633460fc0abd0fa16

Merge "Update visibility in moveTaskToBack if nothing is resumed" into qt-dev

Do not animate status bar icons outside of shade

Test: systrace
Bug: 133182504
Change-Id: Iad0b15f96d935746e16208ae875b69625ba2adfc
Merged-In: I422b3da488fdac6577173d41b26c67f81c7826c5

Merge "Unsaved carrier wifi connection should display carrier in summary" into qt-dev

Merge "Record textAppearance information for TextView." into qt-dev

Merge "Avoid UnsupportedOperationException from getDefaultSmsSubscriptionId()" into qt-dev

Expose PermissionManager as TestApi and fix SplitPermissionInfo equality

Bug: 129200416
Bug: 129200610

Test: atest SplitPermissionsTest

Change-Id: I98795944e6440e5b7ebba134bd7735a6f78ebc4f

Merge "Removing wtf log when crashing alarm manager client" into qt-dev

Merge changes I08c5006a,I6686b77c into qt-dev

* changes:
  Handle null OnSettingsClickListeners in ChannelDialogEditorController
  Close channel dialog when the guts go away

Guard against null mFunc in PooledLambdaImpl

Fixes: 134594642
Test: presubmit
Change-Id: I7b65d18df1625a276986a844569a1275c09df4e4
(cherry picked from commit 1cff0828e32784cbe67f029becedb5ad68cbd932)

Merge "Invalidate talkback accessibility tree after transitioning" into qt-dev

docs: Fixing broken HTML

A couple of broken </code> tags were messing up the formatting
for the entire page (see b/134696600 , http://screen/aThdcpoDvJo.png )

Staged the fixed doc to:

http://go/dac-stage/reference/android/app/Activity

(The live version of this doc has already been fixed -- I fixed
the HTML by hand in cl/251937759 -- but we need to fix the Java
comment, or the file will revert when we publish beta 5.)

Bug: 134696600
Test: make ds-docs (& staged HTML output)
Exempt-From-Owner-Approval: Docs-only fix
Change-Id: I7e6d11ca358d747cdc6d04741d31c7acd85aa080

Merge "Whitelist START_ACTIVITIES_FROM_BACKGROUND permission for permissionController" into qt-dev

Made AccessPoint's scan result list thread safe

If wifi verbose logging is enabled, the main thread may iterate through
the scan result list while the worker thread updates it, resulting in
an ArrayIndexOutOfBounds exception. This change adds a lock to every
iteration through mScanResults and mExtraScanResults.

Bug: 134098126
Test: atest WifiTrackerTest && atest AccessPointTest
Change-Id: If42a9ad8031ff66b1732a79772ba441185f7a18f

Fixes two data races in USAP pool management.

The USAP pool management code contained two data races.  One could cause
a double decrement if a runtime thread executed the SIGCHLD handler
while the server was responding to a pool exit message from a USAP.  The
other data race could occur when the SIGCHLD handler executed in the
middle of a USAP pool flush.

The solution to the first race is to change the return value from a
helper function to ensure that the decrement only occurs when the entry
is invalidated through that specific invocation of the helper.

The second data race was fixed by using SIGTERM instead of SIGKILL when
flushing the USAP pool.  This allows the Zygote to clear the table
entries outside of the SIGCHLD handler, and the handler to avoid
duplicate bookkeeping cleanup when this occurs.  SIGTERM is used so that
it can be differentiated from other process termination events and so
that it can be blocked while the USAP is specializing, but hasn't yet
informed the Zygote of it's removal from the pool.  This issue and this
fix will no longer be necessary once the Zygote signal handler has been
replaced with a signalfd.

Bug: 132794985
Test: atest SignedConfigHostTest
Change-Id: Ie01637a10b356b80b5aa62291a97f2c167242827
Merged-In: Ie01637a10b356b80b5aa62291a97f2c167242827
(cherry picked from commit fb329ba7c858bd73f5a57b2b700c5d042ab8675d)

Handle null OnSettingsClickListeners in ChannelDialogEditorController

The property was already nullable, but prepareDialogForApp() didn't
accept a null click listener.

Test: atest ChannelDialogEditorControllerTest
Fixes: 134697988
Change-Id: I08c5006ac5519f2baa0df1a5ad29d6802c63fb3c

Add timeout for setPermissionGrantState

Fixes: 134380096
Test: presubmit
Change-Id: Ifff243068eea2637cf8f6391b52a4c53bc544615

Unsaved carrier wifi connection should display carrier in summary

Autoconnected carrier wifi should display "Connected via <carrier>"
in the wifi picker even if the network has not been saved.

Bug: 134525097
Test: atest AccessPointTest
Change-Id: Id28daec3f80124f5329386ef5ff2b32cda505355

Invalidate talkback accessibility tree after transitioning

Using AutoTransition seems to allow the talkback accessibility nodes to
stay out of sync if the user starts iterating the view during the
transition. Calling `notifySubtreeAccessibilityStateChangedIfNeeded()`
on the end of the transition gets us back into the correct state.

Test: manual; start iterating view via talkback swiping while the
channel dialog editor transitions between app notifications on/off
Fixes: 133822221

Change-Id: I7218bc0734cf05372b8fbecfb6eeef69cb3f1d40

Merge "Import translations. DO NOT MERGE" into qt-dev

Merge "Restore WATCH_APPOPS permission for systemui" into qt-dev

Whitelist START_ACTIVITIES_FROM_BACKGROUND permission for
permissionController

Bug:134429397
Test: Locally flash on sailfish
Change-Id: I650c08a7f0a823085e0eaa0e1cae8b5b44cabd8f

Merge "Resetting widget cache when some configurations change" into qt-dev

Close channel dialog when the guts go away

Also don't open the NotificationInfo panel from the keyguard without
authenticating first.

Test: long press on notification from lockscreen
Test: lock the device while the channel editor dialog is open
Bug: 133182818
Change-Id: I6686b77cf54ed5c1b82596217263a53d9664fc64

Make PiP window visible when user launches PiP in landscape mode

Two PiP animations are triggered when launching PiP in landscape mode
(one when first swiping up (FADE_IN), the other moves the PiP window to
portrait position (BOUNDS)). When screen rotation happens after the first
animation is started, it cancels it while leaving the window alpha to
close to 0. In order to make sure the window after rotation has the correct
alpha, we should set the alpha of the window when mCancelCurrentBoundsAnimation is true to 1.

Fixes: 133711307
Test: Play a YouTube video in landscape mode, and then swipe up. Make
sure that PiP window shows up after screen rotation forced by launcher
Test: atest PinnedStackTests
Change-Id: I3ec186b0e1e3f50dc91cfb03fc3847ddd7b5faef

Avoid UnsupportedOperationException from getDefaultSmsSubscriptionId()

This is basically reverting to earlier behavior.

Test: basic sanity
Bug: 134375393
Change-Id: I44dc66f9f3d5d4dfcab95220d7a6a6be0ed5a825

Restore WATCH_APPOPS permission for systemui

This permission is necessary for Q as it allows systemui to tag
notifications of apps that are using certain AppOps. This is a P
feature that has regressed when the permission was made privileged.

This is used by AppOpsController in order to obtain updates on any apps
(onOpActiveChanged, onOpNoted). There is no other way to obtain the
updates from apps that are not systemui, as specified in AppOpsManager.

Talked to moltmann@ to verify that this is the correct API to use for this feature.
I'll remove this permission once SystemUI does not need it anymore.
In particular, if AppOpsController does not require it for being notified of AppOps changes.

Fixes: 134747188
Test: build
Test: dumpsys SystemUI. Check that AppOpsController is getting updates.
Change-Id: I08cca4361a7fbfa8b2eb419f1459b0b8bce93a5f

Merge "Dramatically simplify NotificationRankingUpdate." into qt-dev

Merge "You get a memory tile, and you get a memory tile." into qt-dev

Add OWNERS for DeviceConfig

Test: N/A

Bug: 134753749

Change-Id: Ia1b10287a0d056fd9d242936e90d8f91d847fe59

Ignore brightness change thresholds for manual changes.

Bug: 131395565
Test: Manual - set screen_brightness to 32, then 26. Verify it changes.
Verify that threshold still hold for small sensor changes.

Change-Id: I9d9521d9dcfabdeb57eb77b4d968aa0e0e53b946

Merge "Add a rolling log of external vibrations" into qt-dev

Merge "Use nav bar color for invocation lights" into qt-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Iac4022ac6ff25ea77bc986924127d970e25f5444

Stub the oemlock operations when the PersistentDataBlock is not supported

The PersistentDataBlockService is not a compulsory component of framework thus we need to
handle cases when the PersistentDataBlockService is unavailable.

Bug: 132943408
Bug: 123777418
Test: toggle the oemlock in setting menu.

Change-Id: I59eec0a4905b588bf261b8447ad4b4e26ba5278d

Merge "Remove deleted strings from symbols.xml." into qt-dev

Remove deleted strings from symbols.xml.

The background activity start toast strings were removed from
strings.xml in commit 9e2457622abb8611876185f09cb79f5c3c2f4711, but
not from symbols.xml. (Because I didn't know it existed.)

Bug: 131747138
Fixes: 134686670
Test: Still builds.
Change-Id: I934e96155dcc4404f50a309f07ba4ba6c86f151f

Revert "Take all VPN underlying networks into account when migrating traffic for"

This reverts commit 917327b1933c6edc8314fe728e5f14f6e29dd5f4.

Reason for revert: This change has been implicated in 4-way deadlocks as seen in b/134244752.

Bug: 134244752
Change-Id: Ibdaad3a4cbf0d8ef1ed53cfab1e454b9b878bae9

Revert "NetworkStatsService: Fix getDetailedUidStats to take VPNs into account."

This reverts commit ed11b84131eae1e104af9be2c17fc9b07183eee3.

Reason for revert: This change has been implicated in 4-way deadlocks as seen in b/134244752.

Bug: 134244752
Change-Id: I0c00e8f0e30cee987b71b561079a97bf09d4dae4

Revert "Addressing comments for ag/7700679."

This reverts commit a229333986f95e48347fa8b653d7e1776019f84a.

Reason for revert: This change has been implicated in 4-way deadlocks as seen in b/134244752.

Bug: 134244752
Change-Id: I5fbb3443a39a21fc9d96442726cd10d20e8d61cd

Revert "Add one more test for VPN usage stats."

This reverts commit 1419b0b20e4b079bfa66662d042b4dc35fb22d78.

Reason for revert: This change has been implicated in 4-way deadlocks as seen in b/134244752.

Bug: 134244752
Change-Id: I64b48d575f2e6ec4cb7d2d100a859a30af0501dc

You get a memory tile, and you get a memory tile.

Turn on the SysUI memory QS tile for all debuggable builds.
This has the side effect of enabling ActivityManager heap
tracking for com.android.systemui, so don't be surprised
when you start seeing notifications saying "here's a heap
dump!"

Bug: 133671238
Test: runtest systemui
Change-Id: Ib40850bdcc0d67bfbcaf797886d53f2ae706bd72

Merge "Blacklist truncated atoms instead of whitelistnig" into qt-dev

Merge "Don't abort multi-package install early" into qt-dev

Merge "Fixed issue where negative angles were not parsed properly" into qt-dev

Resetting widget cache when some configurations change

Sometimes widget providers hardcode colors in their remoteViews
by loading them from resources. On UI mode change, this can lead to
inconsistent UI if some widgets use a different configuration
than others.

Test: Verified with calendar widget on device
Bug: 133064045
Change-Id: If47a6b1973f55b7590f5d4116813d6a332951697

Merge "Reset bottom area opacity when making it visible" into qt-dev

Merge "Decouple Game Driver from preloading graphics driver" into qt-dev

Removing wtf log when crashing alarm manager client

Removing before final release. Issues with accounting seem to be fixed.
App crashes should serve as a signal now onwards.

Test: Logcat emits no wtf logs when running
CtsAlarmManagerTestCases:UidCapTests#alarmsCannotExceedLimit

Fixes: 128866264
Change-Id: Iad5914ee46baa2b8bd8a596ecd71a5e8bda7b9c8

Don't abort multi-package install early

The multi-package install flow depends upon all of the packages being
processed. If we ever skip one, any installation status will never be
returned to the installation observer.

Fixes: 134632533
Test: adb install-multi-package app1/base.apk app2/base.apk app3/base.apk app4/base.apk
Test: adb did not hang and returned proper error "Failure [INSTALL_FAILED_VERSION_DOWNGRADE"
Change-Id: I3c0d1df059e1411c2d77f96b8fb656c9d1050ce1

Use nav bar color for invocation lights

Registers a listener so that the assistant invocation lights are
set to the same color as the nav bar (and therefore show up on
white backgrounds). Changes the stroke caps to BUTT rather than
ROUND/SQUARE so that they do not overlap (since the nav bar color
can be non-opaque).

If the lights are explicitly set to some other color, the listener
is not used.

Bug: 134523274
Test: manual
Change-Id: I4fc2dd4c609dcada6acdf34e269a0e62547747d0

Merge "Improves the REMINDER_EXP mode for Assist Handle behavior" into qt-dev

Merge "Hide keyguard bottom area when SHADE_LOCKED" into qt-dev

Merge "Fix charging animation clipping" into qt-dev

Merge "OMS: add TEST_MAPPING" into qt-dev

Merge "Inset floating rotation button." into qt-dev

Improves the REMINDER_EXP mode for Assist Handle behavior

- User learning state is reset when leaving REMINDER_EXP behavior.
- Default parameters updated for easier testing and better experience
- Show and go only performed once per day when gesture has been learned
- Slightly delay handle showing when entering an apps with a longer
  delay when switching between apps rapidly.
- Adds state dump to improve debugability.
- Adds ability to suppress handle showing in specific states using
  flags.

Test: atest AssistHandleBehaviorControllerTest
Test: Tested locally
BUG:132983599
Change-Id: I7829ad55d7667f5db3633e702add709968dcd0a9

Dramatically simplify NotificationRankingUpdate.

It is now simply a container for a parceled RankingMap,
which can be unparceled entirely into objects in the
listener's heap and discarded. Notification key strings are
interned (by reference into the Ranking data structure).

This should save a considerable amount of RAM:

 - parceled Bundles hanging around in the listener process,
   waiting to be unparceled, never freed
 - a dozen copies of the notification key string

Bug: 133763354
Test: atest NotificationManagerServiceTest NotificationListenerServiceTest NotificationDataTest
Test: coretests, cts, etc.
Change-Id: I5d8ab5c8f27c70f4af7051b24608ba0d1f2dd5cd

Fix charging animation clipping

Test: visual
Fixes: 134653127
Change-Id: I43eb2523a481237da27f159121bdc5b44b966d75

Hide keyguard bottom area when SHADE_LOCKED

Fixes: 133308946
Test: manual
Change-Id: I1e855f204cf43672cd0bb95c137b48be333c821c

Merge "Default silent notifs on lockscreen to on" into qt-dev

[DO NOT MERGE] Handle config override via settings correctly

Initialize persistence only after we can read the settings
provider as we need to take into account the current config
potentially set as an override in settings vs using the
hard coded defaults.

Make the last persist time more robust to ensure it always
termines to guard against other unforeseen cases wehere the
persistet files don't match what is expected under the current
config.

Test: atest CtsAppOpsTestCases
Test: atest android.appsecurity.cts.AppOpsTest

Bug: 134093967

Change-Id: I4bd272d3908eb8fc2afcdb7c1bd96b05a182d8c4

Merge "loadUnbadgedItemIcon doesn't crash from getUserIcon" into qt-dev

Merge "Refine zoom in and zoom out transition of iamge wallpaper." into qt-dev

Merge "Clarify problems with Linkify.MAP_ADDRESSES." into qt-dev