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

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

Issue 557173002: Some perf improvements to sheriff-o-matics updates. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: 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-failure.html
diff --git a/Tools/GardeningServer/model/ct-failure.html b/Tools/GardeningServer/model/ct-failure.html
index 30576156a7eed48aaab8c5236db8951d2f7f14a2..90a9bbc3c5254c4edf44daaa33fb6480770f3359 100644
--- a/Tools/GardeningServer/model/ct-failure.html
+++ b/Tools/GardeningServer/model/ct-failure.html
@@ -68,7 +68,7 @@ CTFailure.prototype.annotations = function() {
CTFailure.createKey = function(alert) {
function normalize(str) {
str = str == null ? '' : String(str);
- return str.replace(/:/g, '_');
+ return str.replace(':', '_');
esprehn 2014/09/10 02:40:23 This doesn't do the same thing, it used to replace
}
return [alert.master_url, alert.builder_name, alert.failing_build, alert.step_name, alert.reason].map(normalize).join('::');
}
« no previous file with comments | « no previous file | Tools/GardeningServer/model/ct-failure-group.html » ('j') | Tools/GardeningServer/model/ct-failure-group.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698