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

Issue 137143014: Attempt to make stream-timeout test less flaky. (Closed)

Created:
6 years, 10 months ago by Lasse Reichstein Nielsen
Modified:
6 years, 10 months ago
Reviewers:
Anders Johnsen
CC:
reviews_dartlang.org, ricow1
Visibility:
Public.

Description

Attempt to make stream-timeout test less flaky. If the periodic timer gets starved long enough, the timeout might happen. If the 5ms periodic timer hasn't triggered in 250ms, discard the test (i.e., don't fail anyway). R=ajohnsen@google.com Committed: https://code.google.com/p/dart/source/detail?r=32054

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -2 lines) Patch
M tests/lib/async/stream_timeout_test.dart View 2 chunks +6 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Lasse Reichstein Nielsen
6 years, 10 months ago (2014-01-28 08:51:12 UTC) #1
Anders Johnsen
lgtm :(
6 years, 10 months ago (2014-01-28 08:57:27 UTC) #2
Lasse Reichstein Nielsen
6 years, 10 months ago (2014-01-28 08:58:58 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r32054 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698