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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/embeddedEnforcement/iframe-csp-attribute.html

Issue 2370783002: Adding CSP attribute on HTMLIFrameElement (Closed)
Patch Set: Adding formatting to the test file Created 4 years, 3 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/contentSecurityPolicy/embeddedEnforcement/iframe-csp-attribute.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/embeddedEnforcement/iframe-csp-attribute.html b/third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/embeddedEnforcement/iframe-csp-attribute.html
new file mode 100644
index 0000000000000000000000000000000000000000..d1402059e4ffb1c2d936885fd3c78a6fa3e33ba8
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/embeddedEnforcement/iframe-csp-attribute.html
@@ -0,0 +1,39 @@
+<!DOCTYPE html>
+<html>
+<head>
+ <script src="/resources/testharness.js"></script>
+ <script src="/resources/testharnessreport.js"></script>
+</head>
+<body>
+ <script>
+ test(t => {
+ var i = document.createElement('iframe');
+ assert_equals('', i.csp);
+ assert_equals('string', typeof (i.csp));
Mike West 2016/09/26 14:55:02 Nit: Can you add `assert_true('csp' in i)`?
+ }, "<iframe> has a 'csp' attibute which is an empty string if undefined.");
+
+ test(t => {
+ var i = document.createElement('iframe');
+ i.setAttribute('csp', 123456);
+ assert_equals('123456', i.csp);
+ }, "<iframe>'s csp attribute is always a string.");
+
+ test(t => {
+ var i = document.createElement('iframe');
+ i.csp = 'value';
+ assert_equals('value', i.getAttribute('csp'));
+ }, "<iframe> has a 'csp' attibute.");
Mike West 2016/09/26 14:55:02 Nit: Perhaps "<iframe>'s 'csp' content attribute r
+
+ test(t => {
+ var first_iframe = document.createElement('iframe');
+ var second_iframe = document.createElement('iframe');
+
+ first_iframe.setAttribute('csp', 'value 1');
+ assert_equals('value 1', first_iframe.csp);
+
+ second_iframe.setAttribute('csp', 'value 2');
+ assert_equals('value 2', second_iframe.csp);
+ }, "'csp' attribute of <iframe> has proper value.");
Mike West 2016/09/26 14:55:02 Nit: Perhaps "<iframe>'s IDL attribute reflects th
+</script>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698