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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/synchronous-frame-load-in-javascript-url-inherits-correct-origin.html

Issue 1685003002: Plumb the correct owner document through DocumentInit::m_owner. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Trim thingie Created 4 years, 10 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: third_party/WebKit/LayoutTests/http/tests/security/synchronous-frame-load-in-javascript-url-inherits-correct-origin.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/xss-DENIED-synchronous-frame-load-in-javascript-url.html b/third_party/WebKit/LayoutTests/http/tests/security/synchronous-frame-load-in-javascript-url-inherits-correct-origin.html
similarity index 79%
rename from third_party/WebKit/LayoutTests/http/tests/security/xss-DENIED-synchronous-frame-load-in-javascript-url.html
rename to third_party/WebKit/LayoutTests/http/tests/security/synchronous-frame-load-in-javascript-url-inherits-correct-origin.html
index 470c620bc64fafcbf462a43c4a24496a7cd1ab10..f1478609f103a2b74ff32adaae6e767e1578964f 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/xss-DENIED-synchronous-frame-load-in-javascript-url.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/synchronous-frame-load-in-javascript-url-inherits-correct-origin.html
@@ -20,12 +20,10 @@ window.onload = function()
location = "javascript:(" + function() {
a = document.createElement("a");
a.href = "about:blank";
- e = document.createEvent("MouseEvent");
- e.initMouseEvent("click");
- a.dispatchEvent(e);
+ a.click();
return "<script>(" + function() {
- opener.location = "javascript:alert(document.body.innerHTML)";
+ opener.location = "javascript:alert(location)";
if (window.testRunner)
setTimeout("testRunner.notifyDone()", 0);
@@ -37,6 +35,6 @@ window.onload = function()
</script>
</head>
<body>
-This test passes if there's no alert dialog.
+This test passes if the alerted location is the same origin as the main frame.
dcheng 2016/02/23 00:11:32 I believe this behavior change is OK: because we i
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698