Index: Tools/GardeningServer/ui/ct-results-detail.html |
diff --git a/Tools/GardeningServer/ui/ct-results-detail.html b/Tools/GardeningServer/ui/ct-results-detail.html |
index 1eef8a14105f66eef37959de6cf394bc4356c3b5..e172b429b1ce0010b2eb318b4f76f67a80cc8b7d 100644 |
--- a/Tools/GardeningServer/ui/ct-results-detail.html |
+++ b/Tools/GardeningServer/ui/ct-results-detail.html |
@@ -56,7 +56,8 @@ found in the LICENSE file. |
var result = this.failure.resultNodesByBuilder[this.builder]; |
// FIXME: There's probably a less hacky way to check this. |
- if (result.actual == 'FAIL' || result.actual == 'UNKNOWN') |
+ if (result.actual == 'FAIL' || result.actual == 'UNKNOWN' || |
+ this.failure.step == 'compile') |
this._updateUrls(); |
else |
this._updateWebkitTestUrls(); |
@@ -94,7 +95,8 @@ found in the LICENSE file. |
// FIXME: We only store build logs by the test name, not the testsuite.testname, |
// which means that two failing tests from different test suites conflict! |
- var testPart = result.actual == 'UNKNOWN' ? 'stdio' : this.failure.testName.split('.')[1]; |
+ var use_stdio = result.actual == 'UNKNOWN' || this.failure.step == 'compile'; |
ojan
2014/09/11 01:58:41
Lets make a helper function to avoid code duplicat
|
+ var testPart = use_stdio ? 'stdio' : this.failure.testName.split('.')[1]; |
var url = result.masterUrl + |
'/builders/' + encodeURIComponent(this.builder) + |
'/builds/' + result.lastFailingBuild + |