bug 324501 Fire RequestListener.requestDestroyed in last-to-first order

git-svn-id: svn+ssh://dev.eclipse.org/svnroot/rt/org.eclipse.jetty/jetty/trunk@2249 7e9141cc-0065-0410-87d8-b60c137991c4

VERSION.txt

@@ -26,6 +26,7 @@ jetty-7.2-SNAPSHOT
  + 324369 Implement draft-ietf-hybi-thewebsocketprotocol-01
  + 324377 Allow dispatch of ServletRequest and ServletResponse
  + 324379 Change content type after getWriter
+ + 324501 Fire RequestListener.requestDestroyed in last-to-first order.
  + JETTY-912 added per exchange timeout api
  + JETTY-1245 Do not use direct buffers with NIO SSL
  + JETTY-1249 Apply max idle time to all connectors

jetty-server/src/main/java/org/eclipse/jetty/server/handler/ContextHandler.java

@@ -933,9 +933,8 @@ public class ContextHandler extends ScopedHandler implements Attributes, Server.
             {
                 if (_requestListeners!=null)
                 {
-                    final int s=LazyList.size(_requestListeners);
                     final ServletRequestEvent sre = new ServletRequestEvent(_scontext,request);
-                    for(int i=0;i<s;i++)
+                    for(int i=LazyList.size(_requestListeners);i-->0;)
                         ((ServletRequestListener)LazyList.get(_requestListeners,i)).requestDestroyed(sre);
                 }