Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2011 Google Inc. All rights reserved. | 2 * Copyright (C) 2011 Google Inc. All rights reserved. |
| 3 * | 3 * |
| 4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
| 5 * modification, are permitted provided that the following conditions | 5 * modification, are permitted provided that the following conditions |
| 6 * are met: | 6 * are met: |
| 7 * 1. Redistributions of source code must retain the above copyright | 7 * 1. Redistributions of source code must retain the above copyright |
| 8 * notice, this list of conditions and the following disclaimer. | 8 * notice, this list of conditions and the following disclaimer. |
| 9 * 2. Redistributions in binary form must reproduce the above copyright | 9 * 2. Redistributions in binary form must reproduce the above copyright |
| 10 * notice, this list of conditions and the following disclaimer in the | 10 * notice, this list of conditions and the following disclaimer in the |
| (...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 305 }, | 305 }, |
| 306 onRollout: function(revision, testNameList) | 306 onRollout: function(revision, testNameList) |
| 307 { | 307 { |
| 308 checkout.rollout(revision, ui.rolloutReasonForTestNameList(testNameList) ).then($.noop, function() { | 308 checkout.rollout(revision, ui.rolloutReasonForTestNameList(testNameList) ).then($.noop, function() { |
| 309 // FIXME: We should have a better error UI. | 309 // FIXME: We should have a better error UI. |
| 310 alert(kCheckoutUnavailableMessage); | 310 alert(kCheckoutUnavailableMessage); |
| 311 }); | 311 }); |
| 312 } | 312 } |
| 313 }); | 313 }); |
| 314 | 314 |
| 315 controllers.Failures = base.extends(FailureStreamController, { | |
|
ojan
2014/06/14 18:42:11
Dead code.
| |
| 316 _resultsFilter: results.expectedFailuresByTest, | |
| 317 | |
| 318 _keyFor: function(failureAnalysis) | |
| 319 { | |
| 320 return base.dirName(failureAnalysis.testName); | |
| 321 }, | |
| 322 _createFailureView: function(failureAnalysis) | |
| 323 { | |
| 324 return new ui.notifications.FailingTests(); | |
| 325 }, | |
| 326 }); | |
| 327 | |
| 328 controllers.FailingBuilders = base.extends(Object, { | 315 controllers.FailingBuilders = base.extends(Object, { |
| 329 init: function(view, message) | 316 init: function(view, message) |
| 330 { | 317 { |
| 331 this._view = view; | 318 this._view = view; |
| 332 this._message = message; | 319 this._message = message; |
| 333 this._notification = null; | 320 this._notification = null; |
| 334 }, | 321 }, |
| 335 hasFailures: function() | 322 hasFailures: function() |
| 336 { | 323 { |
| 337 return !!this._notification; | 324 return !!this._notification; |
| (...skipping 11 matching lines...) Expand all Loading... | |
| 349 this._notification = new ui.notifications.BuildersFailing(this._mess age); | 336 this._notification = new ui.notifications.BuildersFailing(this._mess age); |
| 350 this._view.add(this._notification); | 337 this._view.add(this._notification); |
| 351 } | 338 } |
| 352 // FIXME: We should provide regression ranges for the failing builders. | 339 // FIXME: We should provide regression ranges for the failing builders. |
| 353 // This doesn't seem to happen often enough to worry too much about that , however. | 340 // This doesn't seem to happen often enough to worry too much about that , however. |
| 354 this._notification.setFailingBuilders(failuresList); | 341 this._notification.setFailingBuilders(failuresList); |
| 355 } | 342 } |
| 356 }); | 343 }); |
| 357 | 344 |
| 358 })(); | 345 })(); |
| OLD | NEW |