+2015-06-23 Ryosuke Niwa <rniwa@webkit.org>
+
+ Build fix. A/B testing is broken when continuous builders report revisions out of order.
+
+ * public/v2/app.js:
+ (App.AnalysisTaskController.Ember.Controller.extend.):
+
2015-06-22 Ryosuke Niwa <rniwa@webkit.org>
A/B testing results should be shown even if they were submitted to different platforms
var revisions = point.measurement.formattedRevisions();
for (var repositoryId in revisions) {
if (!repositoryToRevisions[repositoryId])
- repositoryToRevisions[repositoryId] = {commits: null, revisions: []};
+ repositoryToRevisions[repositoryId] = new Array();
var revision = revisions[repositoryId];
- repositoryToRevisions[repositoryId].revisions[pointIndex] = {
- label: point.label + ': ' + revision.label,
- value: revision.currentRevision,
- };
+ repositoryToRevisions[repositoryId].push({time: point.measurement.latestCommitTime(), value: revision.currentRevision});
}
});
var commitsPromises = [];
var repositoryToIndex = {};
for (var repositoryId in repositoryToRevisions) {
- var revisions = repositoryToRevisions[repositoryId].revisions;
+ var revisions = repositoryToRevisions[repositoryId].sort(function (a, b) { return a.time - b.time; });
repositoryToIndex[repositoryId] = commitsPromises.length;
commitsPromises.push(CommitLogs.fetchCommits(repositoryId, revisions[0].value, revisions[revisions.length - 1].value));
}