[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.18-1-697-g2f78b87
eric at webkit.org
eric at webkit.org
Wed Jan 20 22:18:54 UTC 2010
The following commit has been merged in the debian/unstable branch:
commit 9079afed1d3fc5a2798d9ce83e44f5ca4ad43986
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Mon Jan 11 00:38:16 2010 +0000
2010-01-10 Adam Barth <abarth at webkit.org>
Reviewed by Nikolas Zimmermann.
http/tests/security/xss-DENIED-window-open-javascript-url.html timed out on Windows Debug Bot
https://bugs.webkit.org/show_bug.cgi?id=33349
Previously we were using JavaScript URLs to return to the main event
loop, but that is potentially unreliable because there can be only one
scheduled redirect in flight per frame. Let's try using setTimeout
instead. Notice that there isn't a race condition here because the
event loop will always dispatch the actions in the proper order.
* http/tests/security/xss-DENIED-window-open-javascript-url.html:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@53055 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index b68637b..d9e3042 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,18 @@
+2010-01-10 Adam Barth <abarth at webkit.org>
+
+ Reviewed by Nikolas Zimmermann.
+
+ http/tests/security/xss-DENIED-window-open-javascript-url.html timed out on Windows Debug Bot
+ https://bugs.webkit.org/show_bug.cgi?id=33349
+
+ Previously we were using JavaScript URLs to return to the main event
+ loop, but that is potentially unreliable because there can be only one
+ scheduled redirect in flight per frame. Let's try using setTimeout
+ instead. Notice that there isn't a race condition here because the
+ event loop will always dispatch the actions in the proper order.
+
+ * http/tests/security/xss-DENIED-window-open-javascript-url.html:
+
2010-01-09 Daniel Bates <dbates at webkit.org>
No review, rolling out r53044.
diff --git a/LayoutTests/http/tests/security/xss-DENIED-window-open-javascript-url.html b/LayoutTests/http/tests/security/xss-DENIED-window-open-javascript-url.html
index bb59b75..0c5a1bb 100644
--- a/LayoutTests/http/tests/security/xss-DENIED-window-open-javascript-url.html
+++ b/LayoutTests/http/tests/security/xss-DENIED-window-open-javascript-url.html
@@ -7,8 +7,10 @@ if (window.layoutTestController) {
layoutTestController.waitUntilDone();
}
function runTest() {
+ // Schedule a return to the main event loop so that the javascript: URL
+ // has a chance to run.
+ setTimeout(done, 0);
window.open("javascript:alert('FAIL')", "tg");
- window.open("javascript:done()", "_self");
}
function done() {
if (window.layoutTestController)
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list