diff options
author | Roman Birg <roman@cyngn.com> | 2016-04-14 20:06:03 -0700 |
---|---|---|
committer | Roman Birg <roman@cyngn.com> | 2016-04-18 00:13:42 -0700 |
commit | a88d902d7bc88f8b885d06468a8fa5e1498f3b23 (patch) | |
tree | 39e1c5a419d3dd0c84ceaf3ef3d1f90540663497 /packages/SystemUI/src/com/android/systemui/qs/QSBooleanSettingRow.java | |
parent | 0053be36623525909784d83979a4f9aa42ba17e2 (diff) | |
download | frameworks_base-a88d902d7bc88f8b885d06468a8fa5e1498f3b23.zip frameworks_base-a88d902d7bc88f8b885d06468a8fa5e1498f3b23.tar.gz frameworks_base-a88d902d7bc88f8b885d06468a8fa5e1498f3b23.tar.bz2 |
SystemUI: fix some qs tile race conditions
Whenever setTiles() was called, we assumed all the tile animations had
finished, but if the eager beaver had grabbed a tile and dropped it on the
remove target before waiting for all the animations to complete, a lot of
assumptions are broken and crashes happen. Mainly because tiles aren't removed
from pages and we try to add them to a different page (like if they
needed to be placed a page back).
Implemented cancel for ongoing animations, which we call whenever
setTiles comes in. So all tiles should be in their proper state before
we try to cache/re-add them. Also cleaned up and documented setTiles().
Also, don't try to place the edit tile in the proper place, just add it
to the list if it's not present.
Ticket: CYNGNOS-2472
Change-Id: I5c066abbc16f1fe7173525ea6a8a8b39460461ae
Signed-off-by: Roman Birg <roman@cyngn.com>
Diffstat (limited to 'packages/SystemUI/src/com/android/systemui/qs/QSBooleanSettingRow.java')
0 files changed, 0 insertions, 0 deletions