Ensures that screen is on for testGlobalActions_doesntStealImeControl
Fixes: 157174684
Test: atest com.android.systemui.globalactions.GlobalActionsImeTest
Change-Id: I067ccca29cbbba3831d46ba79d819fda0eb22969
The background is unnecessary and UX wise doesn't look
good when scrolling. We therefore remove it from
the media header.
Because the screen animation still depends on having the
bounds calculated correctly, we now make all of the
background views ExpandableViews instead of
ActivatableNotificationRows.
Bug: 154137987
Test: add media notification, observe no background, normal screen on / off animation
Change-Id: Ie18af9d61d8d128c7a9397660910403e87c6fd51
Java subtype and HAL subtype don't match, which lead to a bug in
configure();
Other minor fixes.
Bug: 157278203
Test: make; cts;
Change-Id: I791d3a06c68499cf359fd718d2ca24a20ac35796
When expanding the PiP window, hide first the close/resize button/handle
if applicable to eliminate possible jankiness.
Vide: http://go/recall/-/aaaaaabFQoRHlzixHdtY/fNFmYOXaILVu0OpHtBNDey
Bug: 157303621
Test: manually expand PiP window
Change-Id: Ic15173f96fc02e38728957e64bb8d90cada8e974
Because qs collapses when we start dozing, we triggered an animation
that looked janky. We now instead don't reattach them at all
and block the animation.
Test: atest SystemUItests
Bug: 154137987
Change-Id: I75a386d7f1dfb65c040d42b8d7aeaf34314b5d10
Motionlayout had a few limitations because we didn't have
any control over the measure cache and the resizing
behavior wasn't very performant for our usecase.
We therefore implement a new Layout called DyanmicLayout
that dynamically allows you to move between different
ViewStates in a performance optimized way.
A new controller is introduced that is responsible
for caching the individual viewstates and react to
possible changes in the view hosts. This will make
sure that our layout is more stable than ever while
beeing fully optimized for multiple players with
various different viewstates, allowing us to implement
usecases much easier where players are conditionally
hidden or replaced when long-pressing.
MediaLocationStates are now fully owned by the
MediaStatesManager and the hiearchyManager only
operates on the bounds of the views, making it more
generic and resuable for future usecases.
Fixes: 156465387
Test: atest SystemUITests
Change-Id: Ica8c2b1404df52d3cc3bf3d0ee5dabbc886557cf
Instead of hardcoding values for IDLE_JOB_PERIOD_MS and
SESSION_EXPIRY_TIMEOUT_MS, make them configurable.
Bug: 157503601
Test: atest --test-mapping apex/blobstore
Change-Id: I4846a98144224873444d4d73b2bedde258992ac4
Fixes an issue where a barrier is set on a layer that will not
submit a buffer and transactions will get queued up causing
the system to eventually watchdog.
Fixes: 150578848
Test: Run manual repro steps in bug
Test: go/wm-smoke
Change-Id: Icba60e60fa9e62cca2fb3ff06d73f1975485a91d
Clicking on the notification will only redirect to the
bubble if the notification has FLAG_BUBBLE. This changes
the logic so that it will happen if the notification is
allowed to bubble.
This also includes a fix for an issue where we would
remove the bubble (and notification) incorrectly on updates:
Previously we'd always remove a bubble if the notif was
updated & no longer had FLAG_BUBBLE. There are some
situations where a notif might not have the flag but also
shouldn't be removed (e.g. bubble dismissed). This CL alters
it so that the bubble is removed when canBubble is false
instead of FLAG_BUBBLE being removed.
Test: atest SystemUITests
Bug: 149033291
Bug: 152883583
Change-Id: If9c61295d09dfa9f36360b02bcd66967bcb7d667
If the notif is unable to present as a bubble:
- set the record#canBubble value to false
- null out bubble metadata if present (we show bubble
button if there is metadata, so shouldn't show it
if it can't present for some reason)
Because bubble metadata is now null'd out we don't have
the shortcutId to check / update the tracked shortcut
list, so instead if it's null check for that entry in
the list and remove it. Also fixes one spot where we
weren't unregistering the listener before and adds a
test for it.
Test: atest ShortcutHelperTest BubbleExtractorTest NotificationManagerServiceTest
Bug: 152883583
Change-Id: I29e993c01a8fb9fcdf2386b18a94adc712fe101f
* We don't allow ongoing bubbles so remove logic
* We were still grouping bubbles by app and this is
something we're no longer supposed to do as per ux
so removed this code
Bug: 155515999
Test: atest BubbleDataTest
Change-Id: I5de5dec6d8be70f638c0cc6121e859a51e44c563
