Dismiss immediately to maintain consistent state.

Fix bug introduced by deferring nulling of mParent.

In dismissDialog the removal was being put on a queue while the
state of the Dialog was being updated immediately. This meant that
if a show() was called before the remove was executed it would try
and add the DecorView a second time. Boom!

Fixes bug 9370301.

Change-Id: I576d1e207c786bc2e21dfd40cb94f2b63a020fe2

1 files changed, 3 insertions, 5 deletions

diff --git a/core/java/android/view/WindowManagerGlobal.java b/core/java/android/view/WindowManagerGlobal.java
index 6bcf863..f7b85cc 100644
--- a/core/java/android/view/WindowManagerGlobal.java
+++ b/core/java/android/view/WindowManagerGlobal.java
@@ -335,15 +335,13 @@ public final class WindowManagerGlobal {
             }
         }
         root.die(immediate);
+        if (view != null) {
+            view.assignParent(null);
+        }
     }
 
     void doRemoveView(ViewRootImpl root) {
         synchronized (mLock) {
-            final View view = root.getView();
-            if (view != null) {
-                view.assignParent(null);
-            }
-
             final int index = mRoots.indexOf(root);
             if (index >= 0) {
                 mRoots.remove(index);