[merge from 6.7] #8580 Added a comment

svn changeset:23371/svn branch:6.8

diff --git a/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java b/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
index dd998e760c65c29a55e4f7991962bfc74cd6033e..a452cfff2d87ebb947e1724362f75aac2599defb 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
+++ b/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
@@ -768,6 +768,10 @@ public class ApplicationConnection {
         if (!hasActiveRequest) {
             throw new IllegalStateException("No active request");
         }
+        // After checkForPendingVariableBursts() there may be a new active
+        // request, so we must set hasActiveRequest to false before, not after,
+        // the call. Active requests used to be tracked with an integer counter,
+        // so setting it after used to work but not with the #8505 changes.
         hasActiveRequest = false;
         if (applicationRunning) {
             checkForPendingVariableBursts();