Milo: Don't float step times if build has finished

milo/appengine/buildbot/builder.go

 // Copyright 2016 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 package buildbot
 
 import (
         "errors"
         "fmt"
         "sort"
@@ skipping 46 matching lines @@
         if len(result) > 1 {
                 switch result[0] {
                 case "failed steps", "exception steps":
                         result = result[1:]
                 }
         }
         return result
 }
 
 func getBuildSummary(b *buildbotBuild) *resp.BuildSummary {
-        started, finished, duration := parseTimes(b.Times)
+        started, finished, duration := parseTimes(nil, b.Times)
         return &resp.BuildSummary{
                 Link: &resp.Link{
                         URL:   fmt.Sprintf("%d", b.Number),
                         Label: fmt.Sprintf("#%d", b.Number),
                 },
                 Status: b.toStatus(),
                 ExecutionTime: resp.Interval{
                         Started:  started,
                         Finished: finished,
                         Duration: duration,
@@ skipping 103 matching lines @@
         }
         result.CurrentBuilds = currentBuilds
 
         for _, fb := range finishedBuilds {
                 if fb != nil {
                         result.FinishedBuilds = append(result.FinishedBuilds, fb)
                 }
         }
         return result, nil
 }