show a list of average test times in the treemap
Created attachment 101523 [details] Patch
Comment on attachment 101523 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=101523&action=review > Tools/TestResultServer/static-dashboards/treemap.html:206 > + if (avgA > avgB) > + return -1; > + else if (avgA == avgB) > + return 0; > + else > + return 1; It's simpler to say: return avgB - avgA;
(In reply to comment #2) > (From update of attachment 101523 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=101523&action=review > > > Tools/TestResultServer/static-dashboards/treemap.html:206 > > + if (avgA > avgB) > > + return -1; > > + else if (avgA == avgB) > > + return 0; > > + else > > + return 1; > > It's simpler to say: > > return avgB - avgA; Whoops. I had it in my head that this was an issue in some cases, but that's only when strings are involved.
Committed r91415: <http://trac.webkit.org/changeset/91415>