fixes #6315, now proper response and cleanup is done if the infamous last desperate...

svn changeset:17017/svn branch:6.5

diff --git a/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java b/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java
index 068cf2e1fa4830479344cad20ec754b01b0972ff..5dd99f9f0d3afab738c5ac2d1ffde7720bad0e4b 100644 (file)
--- a/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java
+++ b/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java
@@ -1221,7 +1221,13 @@ public abstract class AbstractCommunicationManager implements
 
             }
         }
-        return success;
+        /*
+         * Note that we ignore inconsistencies while handling unload request.
+         * The client can't remove invalid variable changes from the burst, and
+         * we don't have the required logic implemented on the server side. E.g.
+         * a component is removed in a previous burst.
+         */
+        return success || closingWindowName != null;
     }
 
     public boolean handleVariableBurst(Object source, Application app,