Title: [233701] trunk/LayoutTests
Revision
233701
Author
sbar...@apple.com
Date
2018-07-10 15:00:43 -0700 (Tue, 10 Jul 2018)

Log Message

Layout Test editing/selection/navigation-clears-editor-state.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=187389
<rdar://problem/41898822>

Rubber-stamped by Ryosuke Niwa.

Make this test less susceptible to timing out on debug builds by:
- lowering the number of iframes we add by a factor of two
- removing the tests own internal 18 second timeout

* editing/selection/navigation-clears-editor-state.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (233700 => 233701)


--- trunk/LayoutTests/ChangeLog	2018-07-10 21:59:37 UTC (rev 233700)
+++ trunk/LayoutTests/ChangeLog	2018-07-10 22:00:43 UTC (rev 233701)
@@ -1,3 +1,17 @@
+2018-07-10  Saam Barati  <sbar...@apple.com>
+
+        Layout Test editing/selection/navigation-clears-editor-state.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=187389
+        <rdar://problem/41898822>
+
+        Rubber-stamped by Ryosuke Niwa.
+
+        Make this test less susceptible to timing out on debug builds by:
+        - lowering the number of iframes we add by a factor of two
+        - removing the tests own internal 18 second timeout
+
+        * editing/selection/navigation-clears-editor-state.html:
+
 2018-07-10  Zalan Bujtas  <za...@apple.com>
 
         FragmentInterval, FragmentIntervalTree and FragmentSearchAdapter should hold not hold raw pointers to renderers.

Modified: trunk/LayoutTests/editing/selection/navigation-clears-editor-state.html (233700 => 233701)


--- trunk/LayoutTests/editing/selection/navigation-clears-editor-state.html	2018-07-10 21:59:37 UTC (rev 233700)
+++ trunk/LayoutTests/editing/selection/navigation-clears-editor-state.html	2018-07-10 22:00:43 UTC (rev 233701)
@@ -36,7 +36,7 @@
     initialDocumentCount = internals.numberOfLiveDocuments();
 
     let frames = [];
-    const count = 200;
+    const count = 100;
     for (let i = 0; i < count; ++i)
         frames.push(appendIframe());
 
@@ -71,13 +71,10 @@
 if (!window.GCController || !window.internals) {
     debug('This test requires GCController and internals');
 } else {
-    if (window.testRunner)
-        setTimeout(() => testRunner.notifyDone(), 18000);
     // Clear out any lingering documents from the previous tests.
     GCController.collect();
     GCController.collect();
     runTest();
-
 }
 
 </script>
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to