Skip to content

Commit 7d954e5

Browse files
jensjohaCommit Queue
authored andcommitted
[CFE] Weekly leak tester should accept first compile too
This should fix the leak found by the weekly bot. https://dart-review.googlesource.com/c/sdk/+/411580 should fix the leak too, but we really should always have run accept here. Change-Id: I223e03e3ddf90b0c8f08e77da17000ecec60f173 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/411601 Reviewed-by: Johnni Winther <[email protected]> Commit-Queue: Jens Johansen <[email protected]>
1 parent 4e76761 commit 7d954e5

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

pkg/front_end/test/flutter_gallery_leak_tester.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,7 @@ Future<void> main(List<String> args) async {
222222
Stopwatch stopwatch = new Stopwatch()..start();
223223
await sendAndWait(heapHelper.process, ['compile package:gallery/main.dart']);
224224
print("First compile took ${stopwatch.elapsedMilliseconds} ms");
225+
await accept(heapHelper);
225226
await pauseAndWait(heapHelper);
226227

227228
await recompileAndWait(heapHelper.process, "package:gallery/main.dart", []);

0 commit comments

Comments
 (0)