tree 525b2d33270b365d53d59ae50114601653911b7c
parent 6aafc063c8b66f71b256202a15e316acc7c2c3e0
author Andrew Xu <andrewxu@chromium.org> 1647467810 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1647467810 +0000

[Launcher Sort] Fix potential data races on closures

Now in `AppListBubbleAppsPage` and `AppsContainerView`,
`update_position_closure_` is injected before triggering the fade
out animation. Meanwhile, triggering the fade out animation should
abort the old running animation. If the animation abortion uses the
closures, which is not the case right now, it is the new closure rather
than the old ones to be used.

This CL fixes such an issue by: abort the old reorder process if any
before injecting the new closure.

Bug: 1305378
Change-Id: I71d66aa3f543c9cf25143e9431035031dc5c12ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3530691
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Commit-Queue: Andrew Xu <andrewxu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#981876}