Don't scan classpath for any development servlet (#14604)

Change-Id: If3db4a35609260beff1079cbaafa8827643d9271

WebContent/WEB-INF/web.xml

@@ -24,10 +24,6 @@
             <param-name>application</param-name>
             <param-value>com.vaadin.tests.components.button.Buttons</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <async-supported>true</async-supported>
     </servlet>
     <servlet>
@@ -37,10 +33,6 @@
             <param-name>UI</param-name>
             <param-value>com.vaadin.tests.components.label.MarginsInLabels</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <async-supported>true</async-supported>
     </servlet>
     <servlet>
@@ -59,10 +51,6 @@
             <param-name>UI</param-name>
             <param-value>com.vaadin.tests.VerifyAssertionsEnabled</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <async-supported>true</async-supported>
     </servlet>
 
@@ -79,10 +67,6 @@
             <param-name>legacyPropertyToString</param-name>
             <param-value>false</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <init-param>
             <param-name>heartbeatInterval</param-name>
             <param-value>301</param-value>
@@ -108,10 +92,6 @@
             because it modifies the VaadinService timeout parameters -->
         <servlet-name>VaadinApplicationRunnerWithTimeoutRedirect</servlet-name>
         <servlet-class>com.vaadin.launcher.ApplicationRunnerServlet</servlet-class>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
     </servlet>
 
     <servlet>
@@ -121,10 +101,6 @@
             <param-name>pushmode</param-name>
             <param-value>automatic</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <async-supported>true</async-supported>
     </servlet>
 
@@ -136,10 +112,6 @@
             <param-name>UI</param-name>
             <param-value>com.vaadin.tests.integration.ServletIntegrationUI</param-value>
         </init-param>
-        <init-param>
-            <param-name>org.atmosphere.cpr.scanClassPath</param-name>
-            <param-value>false</param-value>
-        </init-param>
         <async-supported>true</async-supported>
     </servlet>
 

uitest/src/com/vaadin/launcher/DevelopmentServerLauncher.java

@@ -48,6 +48,7 @@ import org.eclipse.jetty.server.Connector;
 import org.eclipse.jetty.server.Server;
 import org.eclipse.jetty.server.nio.SelectChannelConnector;
 import org.eclipse.jetty.server.ssl.SslSocketConnector;
+import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.util.Scanner;
 import org.eclipse.jetty.util.ssl.SslContextFactory;
 import org.eclipse.jetty.webapp.WebAppContext;
@@ -273,6 +274,19 @@ public class DevelopmentServerLauncher {
             }
         }
 
+        // Read web.xml to find all configured servlets
+        webappcontext.start();
+
+        // Reconfigure all servlets to avoid startup delay
+        for (ServletHolder servletHolder : webappcontext.getServletHandler()
+                .getServlets()) {
+            if (servletHolder
+                    .getInitParameter("org.atmosphere.cpr.scanClassPath") == null) {
+                servletHolder.setInitParameter(
+                        "org.atmosphere.cpr.scanClassPath", "false");
+            }
+        }
+
         try {
             server.start();