Change Future.doWhile to use a loop for non-future results instead of recursion.

Change Future.doWhile to use a loop for non-future results instead of recursion.

Avoids stack overflow on long sequences of non-future results.

R=floitsch@google.com

Committed: https://github.com/dart-lang/sdk/commit/8c742bd500747c2f9a3e7d8a1559d79cac99ffb1

[Lasse Reichstein Nielsen, 2017-05-02 07:21:46]

lrn@google.com changed reviewers:
+ floitsch@google.com, sra@google.com

[floitsch, 2017-05-02 10:16:56]

LGTM.

https://codereview.chromium.org/2853203002/diff/1/sdk/lib/async/future.dart
File sdk/lib/async/future.dart (right):

https://codereview.chromium.org/2853203002/diff/1/sdk/lib/async/future.dart#n...
sdk/lib/async/future.dart:487: // context of all the previous iterations'
callbacks.
Mention stacktrace package here.

[Lasse Reichstein Nielsen, 2017-05-02 10:56:59]

Description was changed from

==========
Change Future.doWhile to use a loop for non-future results instead of recursion.

Avoids stack overflow on long sequences of non-future results.
==========

to

==========
Change Future.doWhile to use a loop for non-future results instead of recursion.

Avoids stack overflow on long sequences of non-future results.

R=floitsch@google.com

Committed:
https://github.com/dart-lang/sdk/commit/8c742bd500747c2f9a3e7d8a1559d79cac99ffb1
==========

[Lasse Reichstein Nielsen, 2017-05-02 10:57:00]

Committed patchset #2 (id:20001) manually as
8c742bd500747c2f9a3e7d8a1559d79cac99ffb1 (presubmit successful).