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

Unified Diff: sdk/lib/_internal/pub/test/serve/utils.dart

Issue 391363004: Include file path in JSON error when possible. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 5 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: sdk/lib/_internal/pub/test/serve/utils.dart
diff --git a/sdk/lib/_internal/pub/test/serve/utils.dart b/sdk/lib/_internal/pub/test/serve/utils.dart
index 0e5d021483fd6b67cc5e7d81f399f8c67eacf53a..d83530f13ea32ce2bdd405f1487083f4bfe8e27f 100644
--- a/sdk/lib/_internal/pub/test/serve/utils.dart
+++ b/sdk/lib/_internal/pub/test/serve/utils.dart
@@ -344,7 +344,7 @@ Future _ensureWebSocket() {
/// Schedules closing the web socket connection to the currently-running pub
/// serve.
Future closeWebSocket() {
- schedule(() {
+ return schedule(() {
nweiz 2014/07/16 18:01:42 Since this return value is clearly never used, I'd
Bob Nystrom 2014/07/16 18:34:45 Done.
return _ensureWebSocket().then((_) => _webSocket.close())
.then((_) => _webSocket = null);
}, "closing web socket");

Powered by Google App Engine
This is Rietveld 408576698