Reviewed by Adele.

        Fix for http://bugs.webkit.org/show_bug.cgi?id=13933 REGRESSION:
        fast/history/clicked-link-is-visited is failing

        * fast/frames/resources/iframe-focus.html: Cause the page to reload
        with layoutTestController.queueReload instead of using JavaScript
        to reload. Using JS seems to get DRT's history all messed up (off
        by one?!).

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

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 7a31aba73dd28535d197cf6e13d26d373793c711..52fbe2c5f0cab88fbafee8e8145e68c5d4b24dc1 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,15 @@
+2007-06-10  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Adele.
+
+        Fix for http://bugs.webkit.org/show_bug.cgi?id=13933 REGRESSION: 
+        fast/history/clicked-link-is-visited is failing
+
+        * fast/frames/resources/iframe-focus.html: Cause the page to reload 
+        with layoutTestController.queueReload instead of using JavaScript 
+        to reload. Using JS seems to get DRT's history all messed up (off 
+        by one?!).
+
 2007-06-08  Sam Weinig  <sam@webkit.org>
 
         Reviewed by Oliver.
@@ -7,7 +19,7 @@
 
         * tables/mozilla/bugs/bug53690-1-expected.txt: Now throws Undefined Error instead of Null Error
 
-2007-06-08  dethbakin  <bdakin@apple.com>
+2007-06-08  Beth Dakin  <bdakin@apple.com>
 
         Reviewed by Brady.
 

diff --git a/LayoutTests/fast/frames/resources/iframe-focus.html b/LayoutTests/fast/frames/resources/iframe-focus.html
index 2c88f94959dc00da8748c941d7de9c2819131057..e63b6b88fa0424b337c8cb5608428921f9537250 100644 (file)
--- a/LayoutTests/fast/frames/resources/iframe-focus.html
+++ b/LayoutTests/fast/frames/resources/iframe-focus.html
@@ -4,9 +4,11 @@
             function test()
             {
                 window.focus();
-                if (parent.location.href.indexOf('?') == -1) {
-                    parent.location.href= parent.location.href + "?";
-                }
+                if (!layoutTestController.boo)
+                    return;
+
+                layoutTestController.boo = false;
+                layoutTestController.queueReload;
             }
         </script>
     </head>