Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(189)

Unified Diff: LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html

Issue 494343003: Eliminate globalFlag usage from http security layout tests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: consistify Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html
diff --git a/LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html b/LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html
index e7d6756fe0f84f6888c46695c7887e2354692510..5d18a05596b24ae47252ecb29dcbdb1007a92147 100644
--- a/LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html
+++ b/LayoutTests/http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level.html
@@ -15,8 +15,7 @@
+ "<scr" + "ipt>"
+ "window.onload = function()"
+ "{"
- + "if (window.testRunner)"
- + "testRunner.globalFlag = true;"
+ + 'window.top.postMessage(\\\\\\"run test\\\\\\", \\\\\\"*\\\\\\");'
+ "}"
+ "</scr" + "ipt>"
+ "</head>"
@@ -36,25 +35,12 @@
var iframe = document.getElementById("aFrame");
iframe.src = url;
- if (window.testRunner) {
- setTimeout(pollForTest, 1);
- } else {
- log("To run the test, click the button below when the opened window finishes loading.");
- var button = document.createElement("button");
- button.appendChild(document.createTextNode("Run Test"));
- button.onclick = runTest;
- document.body.appendChild(button);
- }
- }
-
- pollForTest = function()
- {
- if (!testRunner.globalFlag) {
- setTimeout(pollForTest, 1);
- return;
- }
- runTest();
- testRunner.notifyDone();
+ window.addEventListener('message', function ()
+ {
+ runTest();
+ if (window.testRunner)
+ testRunner.notifyDone();
+ });
}
runTest = function()

Powered by Google App Engine
This is Rietveld 408576698