LayoutTests: Adjust webaudio test for unified heap garbage collection

Use asyncGC instead of window.gc(). Bug: 843903 Change-Id: Id65e65d4b3cd68a1510b3d7aff759232fc6c972b Reviewed-on: https://chromium-review.googlesource.com/c/1324616 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#606358}

LayoutTests: Adjust webaudio test for unified heap garbage collection
Use asyncGC instead of window.gc(). Bug: 843903 Change-Id: Id65e65d4b3cd68a1510b3d7aff759232fc6c972b Reviewed-on: https://chromium-review.googlesource.com/c/1324616 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#606358}
7f2dee6e · Michael Lippautz · Commit Bot · cd1f909b · 7f2dee6e
Commit 7f2dee6e authored Nov 08, 2018 by Michael Lippautz Committed by Commit Bot Nov 08, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 20 additions and 14 deletions

third_party/WebKit/LayoutTests/webaudio/internals/cycle-connection-gc.html ...t/LayoutTests/webaudio/internals/cycle-connection-gc.html +20 -14

No files found.
--- a/third_party/WebKit/LayoutTests/webaudio/internals/cycle-connection-gc.html
+++ b/third_party/WebKit/LayoutTests/webaudio/internals/cycle-connection-gc.html
@@ -4,6 +4,7 @@
    <title>
      cycle-connection-gc.html
    </title>
+    <script src="../../resources/gc.js"></script>
    <script src="../../resources/js-test.js"></script>
  </head>
  <body>
@@ -11,21 +12,26 @@
      description('Cycles of AudioNode connections should be collected.');
      window.jsTestIsAsync = true;

-      gc();
-      let context = new OfflineAudioContext(2, 1024, 44100);
-      let initialCount = internals.audioHandlerCount();
-      createCycle();
-      debug('A cycle was created:');
-      shouldBeTrue('internals.audioHandlerCount() > initialCount');
+      let context;
+      let initialCount;

-      // Need to render to cleanup the cycle on an offline context
-      context.startRendering()
-        .then(() => {
-          gc();
-          debug('GC happened:');
-          shouldBe('internals.audioHandlerCount()', 'initialCount');
-          finishJSTest();
-      });
+      (async () => {
+        await asyncGC();
+
+        context = new OfflineAudioContext(2, 1024, 44100);
+        initialCount = internals.audioHandlerCount();
+        createCycle();
+        debug('A cycle was created:');
+        shouldBeTrue('internals.audioHandlerCount() > initialCount');
+
+        // Need to render to cleanup the cycle on an offline context
+        await context.startRendering();
+
+        await asyncGC();
+        debug('GC happened:');
+        shouldBe('internals.audioHandlerCount()', 'initialCount');
+        finishJSTest();
+      })();

      function createCycle() {
        let source = context.createBufferSource();