JENKINS-15174 Timeout option not retrieved correctly

jenkinsci · Sep 15, 2012 · 253a5bd · 253a5bd
1 parent 40e566d
commit 253a5bd
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper.java b/src/main/java/org/jenkinsci/plugins/xvfb/XvfbBuildWrapper.java
@@ -53,6 +53,8 @@
 
 public class XvfbBuildWrapper extends BuildWrapper {
 
+    private static final int MILLIS_IN_SECOND = 1000;
+
     @Extension
     public static class XvfbBuildWrapperDescriptor extends BuildWrapperDescriptor {
 
@@ -136,7 +138,7 @@ public XvfbBuildWrapper(final String installationName, final Integer displayName
         }
 
         this.debug = Boolean.TRUE.equals(debug);
-        this.timeout = timeout * 1000;
+        this.timeout = timeout;
         this.additionalOptions = additionalOptions;
     }
 
@@ -196,7 +198,7 @@ public Launcher decorateLauncher(@SuppressWarnings("rawtypes") final AbstractBui
 
         process = procStarter.start();
 
-        Thread.sleep(timeout);
+        Thread.sleep(timeout * MILLIS_IN_SECOND);
 
         return launcher;
     }