Bug 214673 – [Win10] js/weakref-finalizationregistry.html is frequently timing out

• Status: NEW
• Component: JavaScriptCore

[Ryan Haddad]

js/weakref-finalizationregistry.html has been frequently timing out on Win10 bots since it was added with r264617.

--- /home/buildbot/worker/win10-release-tests/build/layout-test-results/js/weakref-finalizationregistry-expected.txt
+++ /home/buildbot/worker/win10-release-tests/build/layout-test-results/js/weakref-finalizationregistry-actual.txt
@@ -1,4 +1,6 @@
+FAIL: Timed out waiting for notifyDone to be called
 PASS successfullyParsed is true
 
 TEST COMPLETE
+Error
 

https://results.webkit.org/?suite=layout-tests&test=js%2Fweakref-finalizationregistry.html

[Radar WebKit Bug Importer]

<rdar://problem/65971901>

[Ryan Haddad]

Marked test as flaky in r264744

[Keith Miller]

Man, gc tests are a real nightmare...

[Fujii Hironori]

WinCairo Debug and Release tester are constantly timing out. GTK Debug is ramdomly timing out.
https://results.webkit.org/?suite=layout-tests&test=js%2Fweakref-finalizationregistry.html

[Fujii Hironori]

It's flaky timeout for WinCairo Debug on my PC.
But, it will pass constantly by applying this patch.

diff --git a/LayoutTests/js/script-tests/weakref-finalizationregistry.js b/LayoutTests/js/script-tests/weakref-finalizationregistry.js
index 130c6c81d4de..be85642f01f0 100644
--- a/LayoutTests/js/script-tests/weakref-finalizationregistry.js
+++ b/LayoutTests/js/script-tests/weakref-finalizationregistry.js
@@ -18,7 +18,7 @@ function turnEventLoop() {
         setTimeout(() => {
             gc();
             resolve();
-        }, 1);
+        }, 30);
     });
 }
 
Does GC thread need a time to collect?

[Fujii Hironori]

The commit message of 248952@main explains why it wait for 1ms.

>    * js/script-tests/weakref-finalizationregistry.js:
>    (turnEventLoop): Use a timeout of 1ms instead of 0ms so that
>    the deferred work task that calls the JS FinalizationRegistry
>    callback gets a chance to run before we continue on to the
>    assertion that it was run.