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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/popup-allowed-by-sandbox-when-allowed.html

Issue 2741513002: [Merge M-57] Add a warning for the deprecation of content-initiated data URL navigations (Closed)
Patch Set: Created 3 years, 9 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/popup-allowed-by-sandbox-when-allowed.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/popup-allowed-by-sandbox-when-allowed.html b/third_party/WebKit/LayoutTests/http/tests/security/popup-allowed-by-sandbox-when-allowed.html
index 26e581024bb74474144180ea9cd9b7b1807e4792..9e0e34cd8f5dbc230b12a2135068f1fdc87c6be2 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/popup-allowed-by-sandbox-when-allowed.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/popup-allowed-by-sandbox-when-allowed.html
@@ -7,10 +7,17 @@ if (window.testRunner) {
}
</script>
<p>To run this test outside of DumpRenderTree, please disable your popup blocker!</p>
-<iframe sandbox="allow-scripts allow-popups"
+<iframe sandbox="allow-scripts allow-popups allow-same-origin"
src="data:text/html,
<script>
- var win = window.open('data:text/html,<script>if (window.testRunner) testRunner.notifyDone();<\/script>', '_blank');
- console.log(win ? 'PASS' : 'FAIL');
+ var win = window.open('about:blank', '_blank');
+ if (win) {
+ console.log('PASS');
+ win.document.write(
+ '<script>if (window.testRunner) testRunner.notifyDone();<\/script>');
+ } else {
+ console.log('FAIL');
+ if (window.testRunner) testRunner.notifyDone();
+ }
</script>"
></iframe>

Powered by Google App Engine
This is Rietveld 408576698