fixed merge

jetty-continuation/src/main/java/org/eclipse/jetty/continuation/Servlet3Continuation.java

@@ -34,7 +34,7 @@ import javax.servlet.ServletResponseWrapper;
 /* ------------------------------------------------------------ */
 /**
  * This implementation of Continuation is used by {@link ContinuationSupport}
- * when it detects that the application has been deployed in a non-jetty Servlet 3
+ * when it detects that the application has been deployed in a Servlet 3
  * server.
  */
 public class Servlet3Continuation implements Continuation

jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannel.java

@@ -355,7 +355,7 @@ public class HttpChannel<T> implements HttpParser.RequestHandler<T>, Runnable
             _transport.completed();
         }
 
-        LOG.debug("{} handle exit, result {}", this, next);
+        LOG.debug("{} handle exit, result {}", this, action);
 
         return action!=Action.WAIT;
     }

jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannelState.java

@@ -163,9 +163,7 @@ public class HttpChannelState
     {
         synchronized (this)
         {
-            return _state+
-            (_initial?",initial ":" ")+
-            _async;
+            return String.format("s=%s i=%b a=%s",_state,_initial,_async);
         }
     }
 
@@ -343,7 +341,7 @@ public class HttpChannelState
         synchronized (this)
         {
             if (_async!=Async.STARTED && _async!=Async.EXPIRING)
-                throw new IllegalStateException(this.getStatusString());
+                throw new IllegalStateException("AsyncContext#dispath "+this.getStatusString());
             _async=Async.DISPATCH;
             _event.setDispatchTarget(context,path);
            
@@ -552,7 +550,8 @@ public class HttpChannelState
     {
         synchronized (this)
         {            
-            return _async==Async.STARTED;
+            // Either we are started, or we are still dispatched and async has been completed or dispatched
+            return _async==Async.STARTED || _async!=null && _state==State.DISPATCHED;
         }
     }
 

tests/test-continuation/src/test/java/org/eclipse/jetty/continuation/ContinuationBase.java

@@ -77,7 +77,6 @@ public abstract class ContinuationBase
     protected void doSuspendResume() throws Exception
     {
         String response=process("suspend=200&resume=0",null);
-        System.err.println(response);
         assertContains("RESUMED",response);
         assertNotContains("history: onTimeout",response);
         assertContains("history: onComplete",response);
@@ -503,7 +502,6 @@ public abstract class ContinuationBase
         public void onTimeout(Continuation continuation)
         {
             ((HttpServletResponse)continuation.getServletResponse()).addHeader("history","onTimeout");
-            continuation.resume();
         }
 
     };

tests/test-sessions/test-sessions-common/src/main/java/org/eclipse/jetty/server/session/AbstractProxySerializationTest.java

@@ -119,10 +119,6 @@ public abstract class AbstractProxySerializationTest
                 //stop the context to be sure the sesssion will be passivated
                 context.stop();
                 
-                //after a stop some of the volatile info is lost, so reinstate it
-                context.setClassLoader(loader);
-                context.addServlet("TestServlet", servletMapping);
-                
                 //restart the context
                 context.start();