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

Unified Diff: tests/standalone/io/process_std_io_script2.dart

Issue 16309014: Add support for binary stdout/stderr data when using Process.run (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Removed debug print Created 7 years, 6 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: tests/standalone/io/process_std_io_script2.dart
diff --git a/tests/standalone/io/process_std_io_script2.dart b/tests/standalone/io/process_std_io_script2.dart
index b6ff0262e4d8c5ce2c19b86095836714ee32cda9..4701be14b80d14ae20ffb597db46746a06e0e32e 100644
--- a/tests/standalone/io/process_std_io_script2.dart
+++ b/tests/standalone/io/process_std_io_script2.dart
@@ -9,11 +9,19 @@ import "dart:io";
writeData(data, encoding, stream) {
if (stream == "stdout") {
- stdout.encoding = encoding;
- stdout.write(data);
+ if (encoding == Encoding.BINARY) {
+ stdout.add(data);
+ } else {
+ stdout.encoding = encoding;
+ stdout.write(data);
+ }
} else if (stream == "stderr") {
- stderr.encoding = encoding;
- stderr.write(data);
+ if (encoding == Encoding.BINARY) {
+ stderr.add(data);
+ } else {
+ stderr.encoding = encoding;
+ stderr.write(data);
+ }
}
}
@@ -21,6 +29,7 @@ main() {
var asciiString = 'abc';
var latin1String = 'æøå';
var utf8String = new String.fromCharCodes([955]);
+ var binary = [0, 1, 2];
var options = new Options();
if (options.arguments.length > 1) {
var stream = options.arguments[1];
@@ -30,6 +39,8 @@ main() {
writeData(latin1String, Encoding.ISO_8859_1, stream);
} else if (options.arguments[0] == "utf8") {
writeData(utf8String, Encoding.UTF_8, stream);
+ } else if (options.arguments[0] == "binary") {
+ writeData(binary, Encoding.BINARY, stream);
}
}
}
« runtime/bin/process_patch.dart ('K') | « tests/standalone/io/process_run_output_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698