Fix deferred shortcut container removal.

- Previously, the drag listener listener was removed from the
  container as soon as it started animating closed, which got
  rid of its opportunity to complete the deferred removal.
- The deferred removal now calls close(), which does further
  cleanup including updating the app icon's text visibility.

Bug: 31248681
Change-Id: I18e545e7e19e50b7d47e23a5beb8f254ba1a4ccf
This commit is contained in:
Tony Wickham
2016-09-08 13:58:35 -07:00
parent 285c76dded
commit ec383dfc63

View File

@@ -578,8 +578,7 @@ public class DeepShortcutsContainer extends LinearLayout implements View.OnLongC
} else {
// Close animation is not running.
if (mDeferContainerRemoval) {
mDeferContainerRemoval = false;
mLauncher.getDragLayer().removeView(this);
close();
}
}
}
@@ -600,7 +599,6 @@ public class DeepShortcutsContainer extends LinearLayout implements View.OnLongC
mOpenCloseAnimator.cancel();
}
mIsOpen = false;
mLauncher.getDragController().removeDragListener(this);
final AnimatorSet shortcutAnims = LauncherAnimUtils.createAnimatorSet();
final int shortcutCount = getShortcutCount();