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

Unified Diff: runtime/bin/socket_patch.dart

Issue 12387014: Correctly close done future with an error, before a normal complete can happen. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 10 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
« no previous file with comments | « no previous file | tests/standalone/io/stdout_bad_argument_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/socket_patch.dart
diff --git a/runtime/bin/socket_patch.dart b/runtime/bin/socket_patch.dart
index 3f7362858ba49241c8432f76e3cbc830ec0b0c32..89f621967716be645f02eb843cc8d19ade37fd09 100644
--- a/runtime/bin/socket_patch.dart
+++ b/runtime/bin/socket_patch.dart
@@ -902,11 +902,11 @@ class _Socket extends Stream<List<int>> implements Socket {
}
void _consumerDone([error]) {
+ _done(error);
if (_raw != null) {
_raw.shutdown(SocketDirection.SEND);
_disableWriteEvent();
}
- _done(error);
}
}
« no previous file with comments | « no previous file | tests/standalone/io/stdout_bad_argument_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698