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

Side by Side Diff: tools/testing/dart/test_progress.dart

Issue 234673003: Reduce test.dart memory usage. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Improve Created 6 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « tools/testing/dart/http_server.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library test_progress; 5 library test_progress;
6 6
7 import "dart:async"; 7 import "dart:async";
8 import "dart:io"; 8 import "dart:io";
9 import "dart:io" as io; 9 import "dart:io" as io;
10 import "dart:convert" show JSON; 10 import "dart:convert" show JSON;
(...skipping 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 print(line); 519 print(line);
520 } 520 }
521 print(''); 521 print('');
522 522
523 print(_buildSummaryEnd(_failedTests)); 523 print(_buildSummaryEnd(_failedTests));
524 } 524 }
525 } 525 }
526 } 526 }
527 } 527 }
528 528
529 class DeleteTestCaseOutput extends EventListener {
530 void done(TestCase test) {
531 for (CommandOutputImpl output in test.commandOutputs.values) {
532 output.stdout = null;
533 output.stderr = null;
534 }
535 }
kustermann 2014/04/11 15:17:37 I think you shouldn't do that. If two test cases (
Bill Hesse 2014/04/23 14:52:41 Removing the code that deletes test output. Leavi
536 }
537
529 class ProgressIndicator extends EventListener { 538 class ProgressIndicator extends EventListener {
530 ProgressIndicator(this._startTime); 539 ProgressIndicator(this._startTime);
531 540
532 541
533 void testAdded() { _foundTests++; } 542 void testAdded() { _foundTests++; }
534 543
535 void done(TestCase test) { 544 void done(TestCase test) {
536 if (test.unexpectedOutput) { 545 if (test.unexpectedOutput) {
537 _failedTests++; 546 _failedTests++;
538 } else { 547 } else {
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
661 return new VerboseProgressIndicator(startTime); 670 return new VerboseProgressIndicator(startTime);
662 case 'status': 671 case 'status':
663 return new ProgressIndicator(startTime); 672 return new ProgressIndicator(startTime);
664 case 'buildbot': 673 case 'buildbot':
665 return new BuildbotProgressIndicator(startTime); 674 return new BuildbotProgressIndicator(startTime);
666 default: 675 default:
667 assert(false); 676 assert(false);
668 break; 677 break;
669 } 678 }
670 } 679 }
OLDNEW
« no previous file with comments | « tools/testing/dart/http_server.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698