Revert "Don't suspend requests too early for the first request."

This reverts commit 70ffcba.
lift · Aug 31, 2012 · e686682 · e686682
1 parent 1d9840e
commit e686682
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/web/webkit/src/main/scala/net/liftweb/http/LiftServlet.scala b/web/webkit/src/main/scala/net/liftweb/http/LiftServlet.scala
@@ -626,6 +626,8 @@ class LiftServlet extends Loggable {
                   (renderVersion ->
                     AjaxRequestInfo(handlerVersion, Empty, cont :: Nil, millis))
 
+                suspendRequest()
+
                 Empty // no response available, triggers the actual AJAX computation below
 
               case AjaxRequestInfo(storedVersion, existingResponseBox @ Full(_), _, _) =>
@@ -644,12 +646,14 @@ class LiftServlet extends Loggable {
                 (renderVersion ->
                   AjaxRequestInfo(handlerVersion, Empty, cont :: Nil, millis))
 
+              suspendRequest()
+
               Empty // no response available, triggers the actual AJAX computation below
             }
           }
 
         toReturn or {
-          val result = Full(runAjax(requestState, liftSession, Full((result: LiftResponse) => {
+          Full(runAjax(requestState, liftSession, Full((result: LiftResponse) => {
             // When we get the response, synchronizedly check that the
             // versions are still the same in the map, and, if so, update
             // any waiting actors then clear the actor list and update the
@@ -665,10 +669,6 @@ class LiftServlet extends Loggable {
               }
             }
           })))
-
-          suspendRequest()
-
-          result
         } openOr Empty
 
       case _ =>

diff --git a/web/webkit/src/main/scala/net/liftweb/http/LiftSession.scala b/web/webkit/src/main/scala/net/liftweb/http/LiftSession.scala
@@ -573,7 +573,7 @@ class LiftSession(private[http] val _contextPath: String, val uniqueId: String,
    */
   private var ajaxRequests = scala.collection.mutable.Map[String,AjaxRequestInfo]()
 
-  private[http] def withAjaxRequests[T](fn: (scala.collection.mutable.Map[String, AjaxRequestInfo]) => T): T = {
+  private[http] def withAjaxRequests[T](fn: (scala.collection.mutable.Map[String, AjaxRequestInfo]) => T) = {
     ajaxRequests.synchronized { fn(ajaxRequests) }
   }