Chromium Code Reviews| 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 + |