Changeset 24063 in webkit

Timestamp:

Jul 6, 2007 10:08:50 AM (17 years ago)

Author:

andersca

Message:

Reviewed by Alexey.

<rdar://problem/5313512> fast/frames/frame-set-same-{location,src}.html sometimes hang on Leopard

Make these tests more robust against frames loading in a different order. Also, turn the tests
into text tests and remove them from the Leopard skipped list.

• fast/frames/frame-set-same-location-expected.checksum: Removed.
• fast/frames/frame-set-same-location-expected.png: Removed.
• fast/frames/frame-set-same-location-expected.txt:
• fast/frames/frame-set-same-location.html:
• fast/frames/frame-set-same-src-expected.checksum: Removed.
• fast/frames/frame-set-same-src-expected.png: Removed.
• fast/frames/frame-set-same-src-expected.txt:
• fast/frames/frame-set-same-src.html:
• mac/leopard/Skipped:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/frames/frame-set-same-location-expected.checksum (deleted)
• fast/frames/frame-set-same-location-expected.png (deleted)
• fast/frames/frame-set-same-location-expected.txt (modified) (1 diff)
• fast/frames/frame-set-same-location.html (modified) (1 diff)
• fast/frames/frame-set-same-src-expected.checksum (deleted)
• fast/frames/frame-set-same-src-expected.png (deleted)
• fast/frames/frame-set-same-src-expected.txt (modified) (1 diff)
• fast/frames/frame-set-same-src.html (modified) (1 diff)
• mac/leopard/Skipped (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2007-07-06  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Alexey.
+
+        <rdar://problem/5313512> fast/frames/frame-set-same-{location,src}.html sometimes hang on Leopard
+        
+        Make these tests more robust against frames loading in a different order. Also, turn the tests 
+        into text tests and remove them from the Leopard skipped list.
+        
+        * fast/frames/frame-set-same-location-expected.checksum: Removed.
+        * fast/frames/frame-set-same-location-expected.png: Removed.
+        * fast/frames/frame-set-same-location-expected.txt:
+        * fast/frames/frame-set-same-location.html:
+        * fast/frames/frame-set-same-src-expected.checksum: Removed.
+        * fast/frames/frame-set-same-src-expected.png: Removed.
+        * fast/frames/frame-set-same-src-expected.txt:
+        * fast/frames/frame-set-same-src.html:
+        * mac/leopard/Skipped:
+
 2007-07-06  Mark Rowe  <mrowe@apple.com>
 

trunk/LayoutTests/fast/frames/frame-set-same-location-expected.txt

@@ -1 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderFrameSet {FRAMESET} at (0,0) size 800x600
-      RenderFrame {FRAME} at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderView at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderBlock {HTML} at (0,0) size 397x600
-            RenderBody {BODY} at (8,8) size 381x576
-              RenderBlock {P} at (0,0) size 381x36
-                RenderText {#text} at (0,0) size 53x18
-                  text run at (0,0) width 53: "Test for "
-                RenderInline {A} at (0,0) size 60x18 [color=#0000EE]
-                  RenderText {#text} at (53,0) size 60x18
-                    text run at (53,0) width 60: "bug 3400"
-                RenderText {#text} at (113,0) size 366x36
-                  text run at (113,0) width 253: ": setting the .src of an iframe to the same"
-                  text run at (0,18) width 172: "value does not reload page."
-              RenderBlock {P} at (0,52) size 381x36
-                RenderText {#text} at (0,0) size 375x36
-                  text run at (0,0) width 375: "Testing that setting contentWindow.location.href of a frame"
-                  text run at (0,18) width 152: "to the same value works"
-              RenderBlock {P} at (0,104) size 381x18
-                RenderText {#text} at (0,0) size 71x18
-                  text run at (0,0) width 71: "SUCCESS"
-      RenderFrame {FRAME} at (403,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderView at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderBlock {HTML} at (0,0) size 397x600
-            RenderBody {BODY} at (8,8) size 381x584 [bgcolor=#800080]
+Test for bug 3400: setting the .src of an iframe to the same value does not reload page.
+
+Testing that setting contentWindow.location.href of a frame to the same value works
+
+SUCCESS

trunk/LayoutTests/fast/frames/frame-set-same-location.html

@@ -8 +8 @@
         layoutTestController.waitUntilDone();
 
-    function reportSuccess() {
-                console = document.getElementById("left_frame").contentWindow.document.getElementById("console");
-                if (console.firstChild.nodeValue == "FAILURE") {
+    function rightFrameLoaded() {
+        rightFrameLoadCount++;
+        if (rightFrameLoadCount == 2) {
+            // Console is set by runTest
+            console.firstChild.nodeValue = "SUCCESS";
+            
+            if (window.layoutTestController) {
+                // Put the test result in the main document so we can dump as text
+                var text = console.ownerDocument.documentElement.innerHTML;
+                document.open();
+                document.write(text);
+                document.close();
+                
+                layoutTestController.dumpAsText();
+                layoutTestController.notifyDone();
+            }
+        }
+    }
+    
+    function runTest() {
+        console = document.getElementById("left_frame").contentWindow.document.getElementById("console");
+        if (console.firstChild.nodeValue == "FAILURE") {
             console.firstChild.nodeValue = "PROCESSING...";
             document.getElementById("right_frame").contentWindow.location.href = "resources/purple.html";
         } else {
-            console.firstChild.nodeValue = "SUCCESS";
-            if (window.layoutTestController)
-                layoutTestController.notifyDone();
+            alert('TEST FAILURE: Unexpected onload call')
         }
     }
     
+    rightFrameLoadCount = 0;
 </script>
 </head>
-<frameset cols="50%,*">
+<frameset onload="runTest()" cols="50%,*">
 <frame id="left_frame" src="data:text/html,<body><p>Test for <a href='http://bugzilla.opendarwin.org/show_bug.cgi?id=3400'>bug 3400</a>: setting the .src of an iframe to the same value does not reload page.</p><p>Testing that setting contentWindow.location.href of a frame to the same value works</p><p id=console>FAILURE</p></body>">
-<frame id="right_frame" onload="javascript:reportSuccess()" src="resources/purple.html"></iframe>
+<frame id="right_frame" onload="javascript:rightFrameLoaded()" src="resources/purple.html"></frame>
 </frameset>
 </html>

trunk/LayoutTests/fast/frames/frame-set-same-src-expected.txt

@@ -1 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderFrameSet {FRAMESET} at (0,0) size 800x600
-      RenderFrame {FRAME} at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderView at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderBlock {HTML} at (0,0) size 397x600
-            RenderBody {BODY} at (8,8) size 381x576
-              RenderBlock {P} at (0,0) size 381x36
-                RenderText {#text} at (0,0) size 53x18
-                  text run at (0,0) width 53: "Test for "
-                RenderInline {A} at (0,0) size 60x18 [color=#0000EE]
-                  RenderText {#text} at (53,0) size 60x18
-                    text run at (53,0) width 60: "bug 3400"
-                RenderText {#text} at (113,0) size 366x36
-                  text run at (113,0) width 253: ": setting the .src of an iframe to the same"
-                  text run at (0,18) width 172: "value does not reload page."
-              RenderBlock {P} at (0,52) size 381x36
-                RenderText {#text} at (0,0) size 341x36
-                  text run at (0,0) width 341: "Testing that setting an src of a frame to the same value"
-                  text run at (0,18) width 39: "works"
-              RenderBlock {P} at (0,104) size 381x18
-                RenderText {#text} at (0,0) size 71x18
-                  text run at (0,0) width 71: "SUCCESS"
-      RenderFrame {FRAME} at (403,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderView at (0,0) size 397x600
-        layer at (0,0) size 397x600
-          RenderBlock {HTML} at (0,0) size 397x600
-            RenderBody {BODY} at (8,8) size 381x584 [bgcolor=#800080]
+Test for bug 3400: setting the .src of an iframe to the same value does not reload page.
+
+Testing that setting an src of a frame to the same value works
+
+SUCCESS

trunk/LayoutTests/fast/frames/frame-set-same-src.html

@@ -8 +8 @@
         layoutTestController.waitUntilDone();
 
-    function reportSuccess() {
-                console = document.getElementById("left_frame").contentWindow.document.getElementById("console");
-                if (console.firstChild.nodeValue == "FAILURE") {
+    function rightFrameLoaded() {
+        rightFrameLoadCount++;
+        if (rightFrameLoadCount == 2) {
+            // Console is set by runTest
+            console.firstChild.nodeValue = "SUCCESS";
+
+            if (window.layoutTestController) {
+                // Put the test result in the main document so we can dump as text
+                var text = console.ownerDocument.documentElement.innerHTML;
+                document.open();
+                document.write(text);
+                document.close();
+
+                layoutTestController.dumpAsText();
+                layoutTestController.notifyDone();
+            }
+        }
+    }
+
+    function runTest() {
+        console = document.getElementById("left_frame").contentWindow.document.getElementById("console");
+        if (console.firstChild.nodeValue == "FAILURE") {
             console.firstChild.nodeValue = "PROCESSING...";
             document.getElementById("right_frame").src = "resources/purple.html";
         } else {
-            console.firstChild.nodeValue = "SUCCESS";
-            if (window.layoutTestController)
-                layoutTestController.notifyDone();
+            alert('TEST FAILURE: Unexpected onload call')
         }
     }
     
+    rightFrameLoadCount = 0;
 </script>
 </head>
-<frameset cols="50%,*">
+<frameset onload="runTest()" cols="50%,*">
 <frame id="left_frame" src="data:text/html,<body><p>Test for <a href='http://bugzilla.opendarwin.org/show_bug.cgi?id=3400'>bug 3400</a>: setting the .src of an iframe to the same value does not reload page.</p><p>Testing that setting an src of a frame to the same value works</p><p id=console>FAILURE</p></body>">
-<frame id="right_frame" onload="javascript:reportSuccess()" src="resources/purple.html"></iframe>
+<frame id="right_frame" onload="javascript:rightFrameLoaded()" src="resources/purple.html"></frame>
 </frameset>
 </html>

trunk/LayoutTests/mac/leopard/Skipped

@@ -62 +62 @@
 http/tests/xmlhttprequest/basic-auth.html
 
-# <rdar://problem/5313512> fast/frames/frame-set-same-{location,src}.html sometimes hangs on Leopard
-fast/frames/frame-set-same-location.html
-fast/frames/frame-set-same-src.html
-
 # <rdar://problem/5313531> editing/pasteboard/copy-in-password-field.html sometimes fails on Leopard
 editing/pasteboard/copy-in-password-field.html