From a3ab33f6d154a794c0940d49619a2b95b39bf006 Mon Sep 17 00:00:00 2001
From: Automerge <automerge@vaadin.com>
Date: Fri, 30 Mar 2012 13:24:26 +0000
Subject: [PATCH] [merge from 6.7] #8580 Added a comment

svn changeset:23371/svn branch:6.8
---
 src/com/vaadin/terminal/gwt/client/ApplicationConnection.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java b/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
index dd998e760c..a452cfff2d 100644
--- 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();
-- 
2.39.5