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

Issue 14237002: Add StreamSubscription.asFuture. (Closed)

Created:
7 years, 8 months ago by floitsch
Modified:
7 years, 8 months ago
CC:
reviews_dartlang.org, blois
Visibility:
Public.

Description

Add StreamSubscription.asFuture. Committed: https://code.google.com/p/dart/source/detail?r=21441

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+120 lines, -0 lines) Patch
M runtime/embedders/openglui/common/gl.dart View 1 chunk +6 lines, -0 lines 0 comments Download
M sdk/lib/async/stream.dart View 1 chunk +14 lines, -0 lines 2 comments Download
M sdk/lib/async/stream_impl.dart View 1 chunk +13 lines, -0 lines 2 comments Download
M sdk/lib/async/stream_pipe.dart View 1 chunk +13 lines, -0 lines 0 comments Download
A tests/lib/async/stream_subscription_as_future_test.dart View 1 chunk +68 lines, -0 lines 0 comments Download
M tools/dom/src/EventStreamProvider.dart View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
floitsch
7 years, 8 months ago (2013-04-12 23:23:03 UTC) #1
blois
DOM changes lgtm.
7 years, 8 months ago (2013-04-13 00:38:27 UTC) #2
Anders Johnsen
lgtm https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream_impl.dart File sdk/lib/async/stream_impl.dart (right): https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream_impl.dart#newcode965 sdk/lib/async/stream_impl.dart:965: cancel(); Set unsubscribeOnError = true?
7 years, 8 months ago (2013-04-15 09:15:47 UTC) #3
Lasse Reichstein Nielsen
lgtm https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream.dart File sdk/lib/async/stream.dart (right): https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream.dart#newcode843 sdk/lib/async/stream.dart:843: * when it was listening with `unsubscribeOnError` set ...
7 years, 8 months ago (2013-04-15 10:19:15 UTC) #4
floitsch
https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream.dart File sdk/lib/async/stream.dart (right): https://codereview.chromium.org/14237002/diff/1/sdk/lib/async/stream.dart#newcode843 sdk/lib/async/stream.dart:843: * when it was listening with `unsubscribeOnError` set to ...
7 years, 8 months ago (2013-04-15 12:16:37 UTC) #5
floitsch
7 years, 8 months ago (2013-04-15 12:17:22 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r21441 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698