Make relayout-nested-positioned-elements-crash.html more reliable
authorbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 23 Oct 2011 00:40:00 +0000 (00:40 +0000)
committerbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 23 Oct 2011 00:40:00 +0000 (00:40 +0000)
https://bugs.webkit.org/show_bug.cgi?id=70672

Reviewed by Kenneth Rohde Christiansen.

The test relayout-nested-positioned-elements-crash relies on implementation
details of the test runner. If the function runTest() is run
during the test, the result will be different than the expected results.

Change the test to use the Javascript test framework.
Ensure the element has the correct style in the final state.

* fast/block/positioning/relayout-nested-positioned-elements-crash-expected.txt:
* fast/block/positioning/relayout-nested-positioned-elements-crash.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/block/positioning/relayout-nested-positioned-elements-crash-expected.txt
LayoutTests/fast/block/positioning/relayout-nested-positioned-elements-crash.html

index ba3bdbfb2f6ba3430f99aa2b4e7ef2803de47970..0819a4df866656cd77929a1ae90897298bbf79f7 100644 (file)
@@ -1,3 +1,20 @@
+2011-10-22  Benjamin Poulain  <benjamin@webkit.org>
+
+        Make relayout-nested-positioned-elements-crash.html more reliable
+        https://bugs.webkit.org/show_bug.cgi?id=70672
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        The test relayout-nested-positioned-elements-crash relies on implementation
+        details of the test runner. If the function runTest() is run
+        during the test, the result will be different than the expected results.
+
+        Change the test to use the Javascript test framework.
+        Ensure the element has the correct style in the final state.
+
+        * fast/block/positioning/relayout-nested-positioned-elements-crash-expected.txt:
+        * fast/block/positioning/relayout-nested-positioned-elements-crash.html:
+
 2011-10-20  Pavel Feldman  <pfeldman@google.com>
 
         Web Inspector: get rid of View::hide, View::set visible, View::attach.
index c9df18943c5281fca2ed876f84e77c713766a9a7..06661b9acd0018ea26b6fa72e3c3f9a634db425f 100644 (file)
@@ -1,3 +1,10 @@
-This tests that we don't cause an assertion failure on relayout of nested positioned elements. This test PASSED if we don't cause an assertion failure.
+This tests that we don't cause an assertion failure on relayout of nested positioned elements. This test pass if we don't cause an assertion failure.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS document.getElementById("hideMe").style.display is "none"
+PASS successfullyParsed is true
+
+TEST COMPLETE
 
-PASS 
index 9bf3a80b06c55f6e4e5ccaa42bbb2c70bc0c7280..db4ca45bcb8e180de184be310ec44a6135216b0d 100644 (file)
@@ -1,27 +1,36 @@
 <html>
 <head>
-<title>Test case for bug 64286</title>
-<script>
-if (window.layoutTestController)
-    layoutTestController.dumpAsText();
-</script>
+<link rel="stylesheet" href="../../js/resources/js-test-style.css">
+<script src="../../../fast/js/resources/js-test-pre.js"></script>
 </head>
 <body>
-<p>This tests that we don't cause an assertion failure on relayout of nested positioned elements. This test PASSED if we don't cause an assertion failure.</p>
+<p id="description"></p>
+<div id="console"></div>
 <div style="position:absolute">
     <span style="position:relative">
-        PASS
         <div style="position:absolute">
             <input id="hideMe"/>
         </div>
     </span>
 </div>
 <script>
+description("This tests that we don't cause an assertion failure on relayout of nested positioned elements. This test pass if we don't cause an assertion failure.");
+window.jsTestIsAsync = true;
+
 window.setTimeout(runTest, 0); // For some reason we need the setTimeout() for this test to work.
 function runTest()
 {
     document.getElementById("hideMe").style.display = "none";
+    window.setTimeout(finalizeTest, 0);
+}
+
+function finalizeTest()
+{
+    successfullyParsed = true;
+    shouldBeEqualToString('document.getElementById("hideMe").style.display', 'none');
+    finishJSTest();
 }
 </script>
+<script src="../../js/resources/js-test-post.js"></script>
 </body>
 </html>