Commit Graph

8214 Commits

Author SHA1 Message Date
Jon Miranda ba3ca2f57f Merge changes from topic "presubmit-am-a0fe463985e8453c9c1e5b6ccc543c8b-sc-v2-dev" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Never look for matching view in All Apps when swiping up to go home. 2p: b2068f2eda
  Never look for matching view in All Apps when swiping up to go home.
2022-03-15 18:13:29 +00:00
Presubmit Automerger Backend 8e8638d83f [automerge] Add debug logs for bad launcher state. 2p: f6365d5b85
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17161358

Bug: 223498680
Bug: 221658803
Bug: 223499269
Change-Id: I069cb98329c5e8b37beb5dce9e23bfa306276b52
Merged-In: I138a7ab031269009fa8ca492cd0bd48085aa4e68
2022-03-11 03:35:28 +00:00
Jon Miranda f6365d5b85 Add debug logs for bad launcher state.
Bug: 223498680
Bug: 221658803
Bug: 223499269

Test: manual
Change-Id: I138a7ab031269009fa8ca492cd0bd48085aa4e68
(cherry picked from commit 7ebe209c21)
Merged-In: I138a7ab031269009fa8ca492cd0bd48085aa4e68
2022-03-11 03:35:17 +00:00
Jon Miranda b35524b442 [automerge] Never look for matching view in All Apps when swiping up to go home. 2p: b2068f2eda
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17156660

Bug: 222124240
Change-Id: I13cde344a87a97b938ae067bec088361f45705f9
2022-03-10 20:31:17 +00:00
Jon Miranda b2068f2eda Never look for matching view in All Apps when swiping up to go home.
The swiping up gesture will never return an app in All Apps,
so we can ignore All Apps state in those cases.

This fixes an edge case where user swipes up and launcher state
is still in All Apps. This causes us to animate the icon to
where it would be in All Apps, even though by the time the
animation starts we are actually in Normal state.

Bug: 222124240
Test: open app from  all apps then quickly swipe up to go home
Change-Id: I756a870660a397d6629aec82e4f5ec4914ed0669
(cherry picked from commit b42e124f5b)
2022-03-10 20:31:11 +00:00
Bill Yi bf37866067 Import translations. DO NOT MERGE ANYWHERE
Auto-generated-cl: translation import
Change-Id: Ie305e6a08bf74c0302f843d72786222a519bdf7b
2022-03-02 19:21:36 +00:00
TreeHugger Robot 3bd7bbe47a Merge changes from topic "widget_launch_cookie-sc-v2-dev" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Passing in the launch cookie for widget broadcast trampolines 2p: c519538bca
  Passing in the launch cookie for widget broadcast trampolines
2022-02-28 19:11:10 +00:00
TreeHugger Robot ba297c19c2 Merge "Passing in the launch cookie for widget broadcast trampolines" into sc-v2-dev 2022-02-28 19:11:10 +00:00
Sunny Goyal c6aadf8a74 [automerge] Removing unnecessary AM.getTasks() call on every touch 2p: 8e5a164e05
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17005155

Bug: 220906427
Change-Id: Iedf1d7086d1f9ee1ae167f2a975612de9c84521c
2022-02-24 17:41:01 +00:00
Sunny Goyal 8e5a164e05 Removing unnecessary AM.getTasks() call on every touch
Bug: 220906427
Test: Verified on device
Change-Id: I58ed22c4d6069f10198ec30ec02fa8c2f7c55cbe
(cherry picked from commit d2ec834fac)
2022-02-24 17:40:54 +00:00
Presubmit Automerger Backend a6118187c2 [automerge] Passing in the launch cookie for widget broadcast trampolines 2p: c519538bca
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16968688


Change-Id: I233643ee87042f59a104848d5b6e38a6ac3fa3a2
Merged-In: Ibfe9e5232317837f3111459212a4b016b5828ef4
2022-02-21 23:08:06 +00:00
Selim Cinek c519538bca Passing in the launch cookie for widget broadcast trampolines
The launch cookie can now be transfered via the broadcast

Fixes: 220290671
Test: add Photos widget, return properly to widget
Merged-In: Ibfe9e5232317837f3111459212a4b016b5828ef4
Change-Id: Ibfe9e5232317837f3111459212a4b016b5828ef4
2022-02-21 23:07:57 +00:00
Vinit Nayak a75f2ce2d2 Merge "Add null check for null task in getItemInfo()" into sc-v2-dev 2022-02-16 21:23:47 +00:00
Vinit Nayak 0dffbfcba6 Merge changes from topic "presubmit-am-a8c798abe8b24577a107553295b8cd0f" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Add null check for null task in getItemInfo() 2p: 73c6288038
  Add null check for null task in getItemInfo()
2022-02-16 21:23:47 +00:00
Winson Chung 144b0504ad Merge "Prevent lingering divider animators in AbsSwipeUpHandler" into sc-v2-dev 2022-02-16 19:12:27 +00:00
Presubmit Automerger Backend 21e44c56bb [automerge] Prevent lingering divider animators in AbsSwipeUpHandler 2p: 7faaffa56a
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16693066

Bug: 213403679
Change-Id: I628b14fb01461a103dd0cdd7168835197a28f1c5
Merged-In: I078a7d0f22c2ef2ba847796ec79e740c789ce1ae
2022-02-16 17:13:58 +00:00
Winson Chung 7faaffa56a Prevent lingering divider animators in AbsSwipeUpHandler
- Refactor the util method to create the animator and track the existing
  animation in AbsSwipeUpHandler to be able to cancel it if another call
  to change the visbility comes in.  Note that this doesn't address
  the case where the launch animation overlaps with swipe up (though that
  hopefully shouldn't happen in normal usage)

Bug: 213403679
Test: Tap in the gesture space while split
Change-Id: I078a7d0f22c2ef2ba847796ec79e740c789ce1ae
Merged-In: I078a7d0f22c2ef2ba847796ec79e740c789ce1ae
2022-02-16 17:13:48 +00:00
Vinit Nayak 25e61b2606 [automerge] Add null check for null task in getItemInfo() 2p: 73c6288038
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16828919

Change-Id: I205452840d664917ad1b61d3daec717f211a8ef2
2022-02-10 00:01:19 +00:00
Vinit Nayak 73c6288038 Add null check for null task in getItemInfo()
* Pass back an empty WorkspaceItemInfo with correct
itemType set on it so at least it can be identified
if remaining fields are missing.

Fixes: 218625473
Test: Wasn't able to repro crash
Change-Id: If20d8fa648edf6c210ad5398905bf78e173b23a1
2022-02-09 15:59:13 -08:00
Vinit Nayak 5c6173a8b9 [automerge] Add null check for mRemoteTargetHandles for side task launch 2p: 90e0fe3923
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16828915

Change-Id: I8d41d76a5178c15beca5028291fc0a314a580de3
2022-02-09 23:31:57 +00:00
Vinit Nayak 90e0fe3923 Add null check for mRemoteTargetHandles for side task launch
* Other checks haven't changed, only pulled them out into
separate block

Fixes: 215699962
Test: Unable to repro the crash, I tried ending the
activity and starting via adb shell, but I was never
able to get into the original if-block that triggered the NPE

Change-Id: I83320634f1d059de610176f9031682ca287bd589
2022-02-09 15:30:19 -08:00
Bob Badour 478abd32c3 Fix licenses and license texts.
Entire project is 1p.

Bug: 210912771

Test: m nothing

Change-Id: I1d76cab983a7d1f9078c0e2f1252f5409692f300
Merged-in: Id941d93786882aca78f214ac692433a353b84ed3
Merged-in: I1d76cab983a7d1f9078c0e2f1252f5409692f300
2022-02-01 18:53:38 -08:00
TreeHugger Robot 0702c25340 Merge "Fix taskbar icon dragging issues" into sc-v2-dev am: 9f0d54ebe3
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16738495

Change-Id: If129ddc48e5432bdb5c26280bc7ea20926518fac
2022-02-01 20:08:59 +00:00
Schneider Victor-tulias a8dae26d39 Fix taskbar icon dragging issues
- Fixed icons disappearing when initiating dragging
- Fixed incorrect icon location when initiating dragging after rotating the screen to an orientation other than what launcher started in

Fixes: 215418478
Fixes: 214025075
Fixes: 210460544
Test: dragged icons from the workspace and taskbar
Change-Id: I848138af28802f7d806708c77c25b8de307c70d8
(cherry picked from commit 312b68fde4)
Merged-In: I848138af28802f7d806708c77c25b8de307c70d8
2022-02-01 18:27:59 +00:00
Jon Miranda b2e30bd4e9 Merge "Fix bug where back swipe to icon in All Apps search result." into sc-v2-dev 2022-01-29 00:07:23 +00:00
Jon Miranda ce048f07cc Merge changes from topic "presubmit-am-afa98d56a10a42f4b953a23b9b07db3b" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Fix bug where back swipe to icon in All Apps search result. 2p: db767aa575
  Fix bug where back swipe to icon in All Apps search result.
2022-01-29 00:07:23 +00:00
TreeHugger Robot fea22ae7e9 Merge "Import translations. DO NOT MERGE ANYWHERE" into sc-v2-dev 2022-01-27 12:07:13 +00:00
TreeHugger Robot a9a7d9b5d9 Merge "Update taskbar resume alignment anim if launcher state changes in the middle" into sc-v2-dev 2022-01-25 18:31:32 +00:00
TreeHugger Robot 1d3b256eff Merge changes from topic "presubmit-am-7ba50c92d19c455d8266daf2c96ec31b" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Update taskbar resume alignment anim if launcher state changes in the middle 2p: cd6abc19a8
  Update taskbar resume alignment anim if launcher state changes in the middle
2022-01-25 18:31:32 +00:00
Bill Yi d97da07a54 Import translations. DO NOT MERGE ANYWHERE
Auto-generated-cl: translation import
Change-Id: I9829ed62bfe1ab6f6cdaf011e8b1c75d43b68625
2022-01-25 18:09:35 +00:00
Presubmit Automerger Backend 714c0bf1d6 [automerge] Update taskbar resume alignment anim if launcher state changes in the middle 2p: cd6abc19a8
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16693156

Change-Id: I5fe6e2e1492df3467ae3bc6880ff308e03a401dd
Merged-In: Ie0c6140e14186e41c7e4748dc745f87349b084fe
2022-01-24 23:25:36 +00:00
Presubmit Automerger Backend acbd4852b9 [automerge] Add fallback for missing remote animation callbacks 2p: bb530e9058
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16688543

Bug: 194011186
Change-Id: I975ca18ca54c2ee061b0e8259e4091dc5b999e21
Merged-In: I1b1296fab316b979f441ebb474d1475e3fa68f95
2022-01-24 22:25:30 +00:00
Winson Chung bb530e9058 Add fallback for missing remote animation callbacks
- In some cases WM won't callback the remote animation callbacks (neither
  start nor cancel) and Launcher never finishes executing the pending
  command (preventing the subsequent commands from running).  For the time
  being, just cancel the current state to allow the commands to be
  processed.

Bug: 194011186
Test: Mash on overview and home buttons with a 3p launcher

Signed-off-by: Winson Chung <winsonc@google.com>
Change-Id: I1b1296fab316b979f441ebb474d1475e3fa68f95
Merged-In: I1b1296fab316b979f441ebb474d1475e3fa68f95
2022-01-24 20:11:04 +00:00
Tony Wickham cd6abc19a8 Update taskbar resume alignment anim if launcher state changes in the middle
One way to reproduce this issue is to run `adb shell input keyevent KEYCODE_HOME`, which happens to pause and immediately resume launcher. For example, let's say we run this while in All Apps. Because the isResumed=true comes before the state transition to Normal, we behave as if we are still going to All Apps, specifically goingToUnstashedState = false (since we stash in All Apps). To fix this, we now listen to state changes while the resume alignment animation is playing, and update it if necessary.

Also did the same correction for the gesture alignment animation, though I don't have a specific repo for that.

Finally, because there are now more triggers for alignment animations to play, we add a check to only play them if it's not animating to the same value it's already animating towards. One notable experience this improves is swiping down from All Apps to home; if you do it quick enough, the state animation ends before the taskbar unstash animation, and thus the unstash animation would cancel and start again with the full duration, making it look laggy/disjointed (this behavior existed before this change as well).

Test: TaplTestsQuickstep
Test: Go to All Apps, run `adb shell input keyevent KEYCODE_HOME`, open an app and ensure taskbar icons are visible
Test: Quick switch from home when taskbar is present in apps, but instead go to overview; ensure no jump when taskbar stashes
Test: Swipe down quickly from All Apps, ensure taskbar unstashing doesn't slow down when reaching the end of the state transition
Fixes: 214562370
Change-Id: Ie0c6140e14186e41c7e4748dc745f87349b084fe
Merged-In: Ie0c6140e14186e41c7e4748dc745f87349b084fe
(cherry picked from commit 5fa2ed27bf)
2022-01-24 16:56:12 +00:00
Winson Chung a84f86f254 Merge "Dispatch hover events through the input consumer proxy" into sc-v2-dev 2022-01-21 22:29:43 +00:00
Winson Chung b7225856ad Merge changes from topic "presubmit-am-cf03cdfdc7444d7faac91b866c05eae8" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Dispatch hover events through the input consumer proxy 2p: eff9a120c6
  Dispatch hover events through the input consumer proxy
2022-01-21 22:29:43 +00:00
Winson Chung cda2b35971 Merge "Skip animation if there are no targets" into sc-v2-dev 2022-01-21 20:04:04 +00:00
Winson Chung 4f9711c182 Merge changes from topic "presubmit-am-eb42a70960e740e988dfbddc552edcae" into sc-v2-dev-plus-aosp
* changes:
  [automerge] Skip animation if there are no targets 2p: 6751202328
  Skip animation if there are no targets
2022-01-21 20:04:04 +00:00
Presubmit Automerger Backend 2ec4b4bdf4 [automerge] Dispatch hover events through the input consumer proxy 2p: eff9a120c6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16685428

Bug: 197043796
Change-Id: Ic29524b5310cfa89ef9a45e8201fa95e2f873648
2022-01-21 19:40:41 +00:00
Winson Chung eff9a120c6 Dispatch hover events through the input consumer proxy
- Touch explore uses hover events to focus views for accessibility, but
  we were dropping these events when handling them through the input
  consumer proxy.  The reason this changed is that in sc-v2 we moved the
  recents input consumer to the top of the task display area to ensure
  that it was always above any of the tasks in splitscreen, but by doing
  so, it was always above launcher even after settling in overview.  The
  existing path for handling motion events is heavily tied to touch
  handling (action down/move/up) so we just add a separate path for
  dispatching hover events through the normal mechanism to launcher via
  the consumer.

Bug: 197043796
Change-Id: I5f8cfd357ff13971fe172ce1d0179535479cd26c
2022-01-21 19:40:01 +00:00
Presubmit Automerger Backend 0d388f277a [automerge] Fix bug where back swipe to icon in All Apps search result. 2p: db767aa575
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16684091

Bug: 213306709
Change-Id: Id202b84a430c1e812537afecbf72233a806e7099
2022-01-21 02:08:54 +00:00
Jon Miranda db767aa575 Fix bug where back swipe to icon in All Apps search result.
Issue is that All Apps is scaling during the animation, so when
FloatingIconView looks for it in the view hierarchy,
it's not in its final position.

This would be the cleanest approach for a scv2 fix

Bug: 213306709
Test: manual
Change-Id: Iaec77d15c9533edccd9c82164143af8fa522158f
Merged-In: Iaec77d15c9533edccd9c82164143af8fa522158f
2022-01-21 02:08:18 +00:00
Tracy Zhou b5e6dbc91e End live tile when notification panel is pulled down for the first time in Overview. am: 0077caa15d
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16591950

Change-Id: Ibb334ad989b707213dc9038123312b15b31bfdf0
2022-01-21 02:01:05 +00:00
Winson Chung c923ee1a5d [automerge] Skip animation if there are no targets 2p: 6751202328
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16683595

Bug: 210118444
Change-Id: I0d551c933b0aaf6c12fcd96cd50fb0650467ca3e
2022-01-20 21:34:34 +00:00
Winson Chung 6751202328 Skip animation if there are no targets
Bug: 210118444
Test: Manual
Change-Id: I362ca30d0e00b647d84b5df38b35b924d5ccfb33
2022-01-20 21:32:45 +00:00
Tracy Zhou 0077caa15d End live tile when notification panel is pulled down for the first time in Overview.
Fixes: 211556489
Test: Go to overview with live tile. Turn on dark theme. Pull the panel back up. Make sure everything looks fine (live tile is ended).

Change-Id: I51cb81718a489ad7568c5e05ace0b3dbc6ca5443
2022-01-19 18:30:29 +00:00
Bill Yi 88c671ee71 Import translations. DO NOT MERGE ANYWHERE
Auto-generated-cl: translation import
Change-Id: Iec90538bf8dd18fede035d7f548a71b757d2356c
2022-01-14 04:17:46 +00:00
Lucas Dupin 7b71c2ee59 Merge "Directly assign mDepth" into sc-v2-dev am: c518932f74
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16622831

Change-Id: I834c566de468c086c6f27aa8e71cbcdfd996f27d
2022-01-13 23:37:21 +00:00
Lucas Dupin c518932f74 Merge "Directly assign mDepth" into sc-v2-dev 2022-01-13 23:19:20 +00:00
TreeHugger Robot e81cab2fa4 Merge "TalkBack doesn't announce when a task has being closed." into sc-v2-dev am: c0a6d3fc8f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16346766

Change-Id: I96ae73987e33c706f408d443559836a8d5024e37
2022-01-13 20:49:42 +00:00