projects
/
platform
/
framework
/
web
/
crosswalk.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Upstream version 10.39.225.0
[platform/framework/web/crosswalk.git]
/
src
/
third_party
/
WebKit
/
Tools
/
GardeningServer
/
ui
/
ct-results-detail.html
diff --git
a/src/third_party/WebKit/Tools/GardeningServer/ui/ct-results-detail.html
b/src/third_party/WebKit/Tools/GardeningServer/ui/ct-results-detail.html
index
1eef8a1
..
b5230a4
100644
(file)
--- a/
src/third_party/WebKit/Tools/GardeningServer/ui/ct-results-detail.html
+++ b/
src/third_party/WebKit/Tools/GardeningServer/ui/ct-results-detail.html
@@
-46,6
+46,10
@@
found in the LICENSE file.
builder: '_update',
},
builder: '_update',
},
+ _isStdioStep: function(result, step) {
+ return result.actual == 'UNKNOWN' || step == 'compile';
+ },
+
_update: function() {
if (!this.failure || !this.builder)
return;
_update: function() {
if (!this.failure || !this.builder)
return;
@@
-56,7
+60,7
@@
found in the LICENSE file.
var result = this.failure.resultNodesByBuilder[this.builder];
// FIXME: There's probably a less hacky way to check this.
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' ||
this._isStdioStep(result, this.failure.step)
)
this._updateUrls();
else
this._updateWebkitTestUrls();
this._updateUrls();
else
this._updateWebkitTestUrls();
@@
-94,9
+98,15
@@
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!
// 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 testPart = this._isStdioStep(result, this.failure.step) ? 'stdio' : this.failure.testName.split('.')[1];
+ // Fix url for parameterized gtests, e.g.: /2 -> _2.
+ testPart = testPart.replace(/\/(\d+)$/, "_$1");
var url = result.masterUrl +
var url = result.masterUrl +
- '/builders/' + encodeURIComponent(this.builder) +
+ '/builders/' + encodeURIComponent(this.builder);
+
+ // FIXME: Make failure groups aware of their own url
+ if (this.failure.testName)
+ url +=
'/builds/' + result.lastFailingBuild +
'/steps/' + this.failure.step +
'/logs/' + testPart;
'/builds/' + result.lastFailingBuild +
'/steps/' + this.failure.step +
'/logs/' + testPart;