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

Side by Side Diff: Tools/GardeningServer/model/ct-failure.html

Issue 526633002: Apply object updates from the network without blowing away object identity or UI attributes. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Initial 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 unified diff | Download patch
OLDNEW
1 <!-- 1 <!--
2 Copyright 2014 The Chromium Authors. All rights reserved. 2 Copyright 2014 The Chromium Authors. All rights reserved.
3 Use of this source code is governed by a BSD-style license that can be 3 Use of this source code is governed by a BSD-style license that can be
4 found in the LICENSE file. 4 found in the LICENSE file.
5 --> 5 -->
6 6
7 <script> 7 <script>
8 function CTFailure(step, reason, resultsByBuilder) { 8 function CTFailure(step, reason, resultsByBuilder) {
9 this.fingerprint = step + '::' + reason;
9 this.step = step; 10 this.step = step;
10 // FIXME: Rename this to reason. 11 // FIXME: Rename this to reason.
11 this.testName = reason; 12 this.testName = reason;
13 // Maps a builder_name (e.g. "Mac ASan Tests (1)")
14 // to the details of the failure on that builder.
12 this.resultNodesByBuilder = resultsByBuilder; 15 this.resultNodesByBuilder = resultsByBuilder;
13 } 16 }
14 17
15 CTFailure.prototype.flakinessDashboardURL = function(tree) { 18 CTFailure.prototype.flakinessDashboardURL = function(tree) {
16 var testType = this.step; 19 var testType = this.step;
17 20
18 // FIXME: Replace the flakiness dashboard's sense of groups with sheriff-o-mat ic's 21 // FIXME: Replace the flakiness dashboard's sense of groups with sheriff-o-mat ic's
19 // sense of trees and get rid of this mapping. 22 // sense of trees and get rid of this mapping.
20 var treeToDashboardGroup = { 23 var treeToDashboardGroup = {
21 blink: '@ToT%20Blink', 24 blink: '@ToT%20Blink',
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 66
64 // FIXME: This should just be provided as part of the alerts feed. 67 // FIXME: This should just be provided as part of the alerts feed.
65 CTFailure.createKey = function(alert) { 68 CTFailure.createKey = function(alert) {
66 function normalize(str) { 69 function normalize(str) {
67 str = str == null ? '' : String(str); 70 str = str == null ? '' : String(str);
68 return str.replace(/:/g, '_'); 71 return str.replace(/:/g, '_');
69 } 72 }
70 return [alert.master_url, alert.builder_name, alert.failing_build, alert.step_ name, alert.reason].map(normalize).join('::'); 73 return [alert.master_url, alert.builder_name, alert.failing_build, alert.step_ name, alert.reason].map(normalize).join('::');
71 } 74 }
72 </script> 75 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698