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

Issue 2993683002: Fix flake in code_collection_test. (Closed)

Created:
3 years, 4 months ago by rmacnak
Modified:
3 years, 4 months ago
Reviewers:
zra
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix flake in code_collection_test. foo's code may be detached twice if the machine is slow/loaded and GC happens during shutdown. R=zra@google.com Committed: https://github.com/dart-lang/sdk/commit/544713bceda0895f195b6da54cbf84a802b4efda

Patch Set 1 #

Patch Set 2 : . #

Total comments: 3

Patch Set 3 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -8 lines) Patch
M tests/standalone/io/code_collection_test.dart View 1 2 2 chunks +16 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
rmacnak
https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart File tests/standalone/io/code_collection_test.dart (right): https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart#newcode39 tests/standalone/io/code_collection_test.dart:39: var ret = foo(2); GC after returning here may ...
3 years, 4 months ago (2017-08-04 00:44:21 UTC) #2
zra
lgtm https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart File tests/standalone/io/code_collection_test.dart (right): https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart#newcode39 tests/standalone/io/code_collection_test.dart:39: var ret = foo(2); On 2017/08/04 00:44:21, rmacnak ...
3 years, 4 months ago (2017-08-04 05:19:43 UTC) #3
rmacnak
https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart File tests/standalone/io/code_collection_test.dart (right): https://codereview.chromium.org/2993683002/diff/20001/tests/standalone/io/code_collection_test.dart#newcode39 tests/standalone/io/code_collection_test.dart:39: var ret = foo(2); On 2017/08/04 05:19:43, zra wrote: ...
3 years, 4 months ago (2017-08-04 17:33:44 UTC) #4
rmacnak
3 years, 4 months ago (2017-08-04 17:52:10 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
544713bceda0895f195b6da54cbf84a802b4efda (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698