Remove the forward() from the rebuild AJAX call.

We don't know why the forward exists. It raises a 404 error in the JS console,
so the URL it's trying to forward to apparently doesn't exist/make sense. And
we don't know what work there is to do at the point of the forward() call
anyway, since rebuilding has already been kicked off.

src/main/java/com/etsy/jenkins/MasterProject.java

@@ -58,8 +58,8 @@ public class MasterProject
 extends Project<MasterProject, MasterBuild>
 implements TopLevelItem {
 
-  @BindingAnnotation 
-  @Target({ElementType.FIELD, ElementType.PARAMETER, ElementType.METHOD }) 
+  @BindingAnnotation
+  @Target({ElementType.FIELD, ElementType.PARAMETER, ElementType.METHOD })
   @Retention(RetentionPolicy.RUNTIME)
   public @interface PingTime {}
 
@@ -71,7 +71,7 @@ public class MasterProject
 
   @DataBoundConstructor
   public MasterProject(
-      ItemGroup parent, 
+      ItemGroup parent,
       String name) {
     super(parent, name);
     this.jobNames = Sets.<String>newHashSet();
@@ -97,7 +97,7 @@ public List<Builder> getBuilders() {
   }
 
   public List<Descriptor<Publisher>> getPotentialPublisherDescriptors() {
-    List<Descriptor<Publisher>> publishers = 
+    List<Descriptor<Publisher>> publishers =
         Lists.<Descriptor<Publisher>>newArrayList(MasterMailer.DESCRIPTOR);
     if (hudson.getPlugin("ircbot") != null) {
       publishers.add(IrcPublisher.DESCRIPTOR);
@@ -117,14 +117,13 @@ protected Class<MasterBuild> getBuildClass() {
     return MasterBuild.class;
   }
 
-  public void doRebuild(StaplerRequest req, StaplerResponse res) 
+  public void doRebuild(StaplerRequest req, StaplerResponse res)
       throws IOException, ServletException {
     checkPermission(Permission.READ);
 
     AbstractProject subProject = this.getSubProject(req);
     MasterBuild masterBuild = this.getMasterBuild(req);
     masterBuild.rebuild(subProject);
-    res.forward(masterBuild, masterBuild.getUrl(), req);
   }
 
   private AbstractProject getSubProject(StaplerRequest req)
@@ -174,7 +173,7 @@ public FormValidation doCheck(
         if (item == null) {
           return FormValidation.error(
               Messages.BuildTrigger_NoSuchProject(
-                  projectName, 
+                  projectName,
                   AbstractProject.findNearest(projectName).getName()));
         }
         if (!(item instanceof AbstractProject)) {
@@ -207,9 +206,9 @@ public AutoCompletionCandidates doAutoCompleteSubProjects(
   }
 
   @Override
-  protected void submit(StaplerRequest req, StaplerResponse res) 
+  protected void submit(StaplerRequest req, StaplerResponse res)
       throws IOException, ServletException, Descriptor.FormException {
-  
+
     // Handle the job list
     jobNames.clear();
     for (TopLevelItem item : Hudson.getInstance().getItems()) {
@@ -244,7 +243,7 @@ public Set<String> getSubProjectNames() {
   }
 
   @Extension
-  public static final TopLevelItemDescriptor DESCRIPTOR = 
+  public static final TopLevelItemDescriptor DESCRIPTOR =
       new TopLevelItemDescriptor() {
 
     private Injector injector = Guice.createInjector(new MasterModule());