Gardening: Layout Test js/error-should-not-strong-reference-global-object.html is...
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Jun 2018 21:21:21 +0000 (21:21 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Jun 2018 21:21:21 +0000 (21:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186372

Not reviewed.

Speculative fix to green the bots plus improved the error message logging to get
more info if this fails again.

* js/error-should-not-strong-reference-global-object.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232810 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/js/error-should-not-strong-reference-global-object.html

index c04c60a..4d65862 100644 (file)
@@ -1,3 +1,15 @@
+2018-06-13  Mark Lam  <mark.lam@apple.com>
+
+        Gardening: Layout Test js/error-should-not-strong-reference-global-object.html is a flaky failure
+        https://bugs.webkit.org/show_bug.cgi?id=186372
+
+        Not reviewed.
+
+        Speculative fix to green the bots plus improved the error message logging to get
+        more info if this fails again. 
+
+        * js/error-should-not-strong-reference-global-object.html:
+
 2018-06-13  Timothy Hatcher  <timothy@apple.com>
 
         Input form controls have a white background in dark mode, when they should not.
index 0365cc6..75b789f 100644 (file)
@@ -40,10 +40,13 @@ function run() {
 for (let i = 0; i < 50; i++)
     run();
 
-$vm.gc();
-// We shouldn't have more than 10% of the global objects we allocated.
-if ($vm.globalObjectCount() >= 51)
-    throw new Error("There are more global objects than there should be");
+setTimeout(() => {
+    $vm.gc();
+    // We shouldn't have more than 10% of the global objects we allocated.
+    var globalObjectCount = $vm.globalObjectCount();
+    if (globalObjectCount >= 51)
+        throw new Error("There are more global objects than there should be: actual count = " + globalObjectCount);
+}, 1);
 
 </script>
 <script src="../resources/js-test-post.js"></script>