[FIXED JENKINS-36732] Use build start times instead of build scheduled times...

[FIXED JENKINS-36732] Use build start times instead of build scheduled times in `BuildTimelineWidget`. (#2460)

core/src/main/java/hudson/model/BuildTimelineWidget.java

@@ -62,8 +62,8 @@ public class BuildTimelineWidget {
         TimelineEventList result = new TimelineEventList();
         for (Run r : builds.byTimestamp(min,max)) {
             Event e = new Event();
-            e.start = r.getTime();
-            e.end   = new Date(r.timestamp+r.getDuration());
+            e.start = new Date(r.getStartTimeInMillis());
+            e.end   = new Date(r.getStartTimeInMillis()+r.getDuration());
             e.title = r.getFullDisplayName();
             // what to put in the description?
             // e.description = "Longish description of event "+r.getFullDisplayName();

core/src/main/java/jenkins/widgets/BuildListTable.java

@@ -25,9 +25,13 @@
 package jenkins.widgets;
 
 import hudson.Functions;
+import hudson.Util;
 import hudson.model.BallColor;
 import hudson.model.Run;
 import net.sf.json.JSONObject;
+
+import java.util.Date;
+
 import org.kohsuke.accmod.Restricted;
 import org.kohsuke.accmod.restrictions.DoNotUse;
 
@@ -45,6 +49,7 @@ public class BuildListTable extends RunListProgressiveRendering {
         element.put("displayName", build.getDisplayName());
         element.put("timestampString", build.getTimestampString());
         element.put("timestampString2", build.getTimestampString2());
+        element.put("timestampString3", Util.XS_DATETIME_FORMATTER.format(new Date(build.getStartTimeInMillis())));
         Run.Summary buildStatusSummary = build.getBuildStatusSummary();
         element.put("buildStatusSummaryWorse", buildStatusSummary.isWorse);
         element.put("buildStatusSummaryMessage", buildStatusSummary.message);

core/src/main/resources/lib/hudson/buildListTable.jelly

@@ -47,7 +47,7 @@ THE SOFTWARE.
                   insert('\u00A0').
                   insert(new Element('a', {href: '${rootURL}/' + e.url, 'class': 'model-link inside'}).
                       update(e.displayName.escapeHTML())));
-              tr.insert(new Element('td', {data: e.timestampString2, tooltip: '${%Click to center timeline on event}', onclick: 'javascript:tl.getBand(0).scrollToCenter(Timeline.DateTime.parseGregorianDateTime("' + e.timestampString2 + '"))'}).
+              tr.insert(new Element('td', {data: e.timestampString2, tooltip: '${%Click to center timeline on event}', onclick: 'javascript:tl.getBand(0).scrollToCenter(Timeline.DateTime.parseGregorianDateTime("' + e.timestampString3 + '"))'}).
                   update(e.timestampString.escapeHTML()));
               tr.insert(new Element('td', {style: e.buildStatusSummaryWorse ? 'color: red' : ''}).
                   update(e.buildStatusSummaryMessage.escapeHTML()));