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

Side by Side Diff: utils/testrunner/layout_test_controller.dart

Issue 22872012: Remove Encoding-enum from dart:io and add interface in dart:convert. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix typo. Created 7 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 | Annotate | Revision Log
« no previous file with comments | « tools/ddbg.dart ('k') | utils/testrunner/pipeline_utils.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) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 // The following set of variables should be set by the caller that 5 // The following set of variables should be set by the caller that
6 // #sources this file. 6 // #sources this file.
7 /** Whether to include elapsed time. */ 7 /** Whether to include elapsed time. */
8 8
9 part of test_controller; 9 part of test_controller;
10 10
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 return exitCode; 161 return exitCode;
162 }) 162 })
163 .catchError((e) { 163 .catchError((e) {
164 stderr.add("#Error starting process $command: ${e.error}"); 164 stderr.add("#Error starting process $command: ${e.error}");
165 }); 165 });
166 } 166 }
167 167
168 void _pipeStream(Stream stream, List<String> destination, 168 void _pipeStream(Stream stream, List<String> destination,
169 Function outputMonitor) { 169 Function outputMonitor) {
170 stream 170 stream
171 .transform(new StringDecoder()) 171 .transform(UTF8.decoder)
172 .transform(new LineTransformer()) 172 .transform(new LineTransformer())
173 .listen((String line) { 173 .listen((String line) {
174 if (outputMonitor != null) { 174 if (outputMonitor != null) {
175 outputMonitor(line); 175 outputMonitor(line);
176 } 176 }
177 destination.add(line); 177 destination.add(line);
178 }); 178 });
179 } 179 }
180 180
181 /** 181 /**
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
359 359
360 void runPixelLayoutTests() { 360 void runPixelLayoutTests() {
361 init(); 361 init();
362 runPixelLayoutTest(0); 362 runPixelLayoutTest(0);
363 } 363 }
364 364
365 void runTextLayoutTests() { 365 void runTextLayoutTests() {
366 init(); 366 init();
367 runTextLayoutTest(0); 367 runTextLayoutTest(0);
368 } 368 }
OLDNEW
« no previous file with comments | « tools/ddbg.dart ('k') | utils/testrunner/pipeline_utils.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698