Title: [210380] branches/safari-603-branch/LayoutTests
Revision
210380
Author
matthew_han...@apple.com
Date
2017-01-05 15:49:03 -0800 (Thu, 05 Jan 2017)

Log Message

Merge r210022. rdar://problem/29316054

Modified Paths

Diff

Modified: branches/safari-603-branch/LayoutTests/ChangeLog (210379 => 210380)


--- branches/safari-603-branch/LayoutTests/ChangeLog	2017-01-05 23:42:05 UTC (rev 210379)
+++ branches/safari-603-branch/LayoutTests/ChangeLog	2017-01-05 23:49:03 UTC (rev 210380)
@@ -1,5 +1,22 @@
 2017-01-05  Matthew Hanson  <matthew_han...@apple.com>
 
+        Merge r210022. rdar://problem/29316054
+
+    2016-12-20  Megan Gardner  <megan_gard...@apple.com>
+
+            Fix memory leak in malformed test
+            https://bugs.webkit.org/show_bug.cgi?id=165917
+
+            Reviewed by Alexey Proskuryakov.
+
+            Navigation caused 'onload' to be called twice, causing test harness to have extraneous
+            HID callback around after script controller was dismantled. Loading in body will eliminate
+            double load, and also prevent other race conditions where the body doesn't load fast enough.
+
+            * http/tests/quicklook/hide-referer-on-navigation.html:
+
+2017-01-05  Matthew Hanson  <matthew_han...@apple.com>
+
         Merge r210266. rdar://problem/29447824
 
     2017-01-03  Tim Horton  <timothy_hor...@apple.com>

Modified: branches/safari-603-branch/LayoutTests/http/tests/quicklook/hide-referer-on-navigation.html (210379 => 210380)


--- branches/safari-603-branch/LayoutTests/http/tests/quicklook/hide-referer-on-navigation.html	2017-01-05 23:42:05 UTC (rev 210379)
+++ branches/safari-603-branch/LayoutTests/http/tests/quicklook/hide-referer-on-navigation.html	2017-01-05 23:49:03 UTC (rev 210380)
@@ -3,6 +3,10 @@
 <head>
 <script src=""
 <script>
+function doTest(){
+    tapAtPointAndNotifyDone(document.getElementsByTagName('iframe')[0].offsetLeft + 5, document.getElementsByTagName('iframe')[0].offsetTop + 5)
+}
+
 if (window.testRunner) {
     testRunner.dumpAsText();
     testRunner.dumpChildFramesAsText();
@@ -10,8 +14,9 @@
 }
 </script>
 </head>
-<body>
+<body _onload_="doTest()">
 <p>This test verifies that the HTTP referrer is hidden when navigating from a Microsoft Word QuickLook preview. This test PASSED only if nothing is printed below.</p>
-<iframe src="" _onload_="tapAtPoint(this.offsetLeft + 5, this.offsetTop + 5, function () { /* Do nothing */ })"></iframe>
+
+<iframe src=""
 </body>
 </html>
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to