Merge "Revert "Fix exit transition and dismiss callback for popup exit transition"" into nyc-dev

am: 9cae665410

* commit '9cae665410e339b19633645a67c353fe4b10a559':
  Revert "Fix exit transition and dismiss callback for popup exit transition"
This commit is contained in:
Alan Viverette
2016-02-25 22:23:20 +00:00
committed by android-build-merger

View File

@@ -1632,7 +1632,6 @@ public class PopupWindow {
final PopupDecorView decorView = mDecorView;
final View contentView = mContentView;
final OnDismissListener dismissListener = mOnDismissListener;
final ViewGroup contentHolder;
final ViewParent contentParent = contentView.getParent();
@@ -1676,16 +1675,19 @@ public class PopupWindow {
new TransitionListenerAdapter() {
@Override
public void onTransitionEnd(Transition transition) {
dismissImmediate(decorView, contentHolder,
contentView, dismissListener);
dismissImmediate(decorView, contentHolder, contentView);
}
});
} else {
dismissImmediate(decorView, contentHolder, contentView, dismissListener);
dismissImmediate(decorView, contentHolder, contentView);
}
// Clears the anchor view.
unregisterForViewTreeChanges();
if (mOnDismissListener != null) {
mOnDismissListener.onDismiss();
}
}
/**
@@ -1727,8 +1729,7 @@ public class PopupWindow {
* Removes the popup from the window manager and tears down the supporting
* view hierarchy, if necessary.
*/
private void dismissImmediate(View decorView, ViewGroup contentHolder,
View contentView, OnDismissListener listener) {
private void dismissImmediate(View decorView, ViewGroup contentHolder, View contentView) {
// If this method gets called and the decor view doesn't have a parent,
// then it was either never added or was already removed. That should
// never happen, but it's worth checking to avoid potential crashes.
@@ -1745,10 +1746,6 @@ public class PopupWindow {
mDecorView = null;
mBackgroundView = null;
mIsTransitioningToDismiss = false;
if (mOnDismissListener != null) {
mOnDismissListener.onDismiss();
}
}
/**