Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2267)

Unified Diff: Tools/GardeningServer/model/ct-builder.html

Issue 555263004: Add hung bots to sheriff-o-matic view (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add a basic test Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Tools/GardeningServer/model/ct-builder.html
diff --git a/Tools/GardeningServer/model/ct-builder.html b/Tools/GardeningServer/model/ct-builder.html
index aa073e447b7fe3d47b3d77a214a5da9c5babfc74..92437d6319f9114de892973aa9971a0bf588b621 100644
--- a/Tools/GardeningServer/model/ct-builder.html
+++ b/Tools/GardeningServer/model/ct-builder.html
@@ -10,7 +10,9 @@ function CTBuilder(masterUrl, builder, firstFailingBuild, failingBuildCount) {
this.builder = builder;
this.firstFailingBuild = firstFailingBuild;
this.failingBuildCount = failingBuildCount;
- this.buildUrl = "{1}/builders/{2}/builds/{3}".assign(
- masterUrl, encodeURIComponent(builder), firstFailingBuild);
+ this.buildUrl = "{1}/builders/{2}".assign(
+ masterUrl, encodeURIComponent(builder));
ojan 2014/09/10 00:40:02 This should be indented 4 spaces.
+ if (firstFailingBuild)
+ this.buildUrl += "/builds/" + firstFailingBuild;
}
</script>

Powered by Google App Engine
This is Rietveld 408576698