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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php

Issue 2370843003: Suborigin LayoutTest formatting and refactoring (Closed)
Patch Set: 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
« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-change-document-domain.html ('k') | third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cssRules.php » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php b/third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php
index 61d425f0b91789193be8f759bc3cc41bbfca628c..7812e1b2797e3dd025f2a573ade6d0a28421deaf 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php
+++ b/third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cookies.php
@@ -5,6 +5,7 @@ header("Suborigin: foobar");
<!DOCTYPE html>
<html>
<head>
+<meta charset="utf-8">
<title>Validate behavior of cookies in Suborigins</title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
@@ -12,99 +13,108 @@ header("Suborigin: foobar");
<body>
<script>
test(t => {
- assert_equals(document.cookie, "");
- document.cookie = "foo=bar";
- assert_equals(document.cookie, "");
+ assert_equals(document.cookie, '');
+ document.cookie = 'foo=bar';
+ assert_equals(document.cookie, '');
t.done();
-}, "Document is cookie-averse");
+ }, 'Document is cookie-averse');
test(t => {
Object.defineProperties(document, {
- "cookie": {
- get: function() { return this.x; },
- set: function(x) { this.x = x; }
+ 'cookie': {
+ get: function() { return this.x; },
+ set: function(x) { this.x = x; }
}
- });
+ });
- document.cookie = "foo";
- assert_equals(document.cookie, "foo");
+ document.cookie = 'foo';
+ assert_equals(document.cookie, 'foo');
delete document.cookie;
t.done();
-}, "Document getters and setters still work");
+ }, 'Document getters and setters still work');
async_test(t => {
window.addEventListener('message', function(event) {
- if (event.data.test_name != "iframetest")
+ if (event.data.test_name != 'iframetest')
return;
var cookie_val = event.data.cookie_val;
- assert_equals(cookie_val, "test_cookie=a_value");
+ assert_equals(cookie_val, 'test_cookie=a_value');
t.done();
});
var iframe = document.createElement('iframe');
- iframe.src = "resources/post-document-cookie.php?testname=iframetest";
+ iframe.src = 'resources/post-document-cookie.php?testname=iframetest';
document.body.appendChild(iframe);
-}, "Cookies set in a frame with a regular, same-origin src do not modify the suborigin's document.cookie");
-
-function makeIframeString(test_name) {
- var postMessageContent = "{cookie_val: document.cookie, test_name: '" + test_name + "'}";
- return "document.cookie = 'foo=bar'; window.parent.postMessage(" + postMessageContent + ", '*')";
+ },
+ 'Cookies set in a frame with a regular, same-origin src do not modify ' +
+ 'the suborigin\'s document.cookie');
+
+function make_iframe_string(test_name) {
+ var post_message_content = '{cookie_val: document.cookie, ' +
+ 'test_name: \'' + test_name + '\'}';
+ return 'document.cookie = \'foo=bar\'; window.parent.postMessage(' +
+ post_message_content + ', \'*\')';
}
async_test(t => {
window.addEventListener('message', function(event) {
- if (event.data.test_name != "about:blanktest")
- return;
+ if (event.data.test_name != 'about:blanktest')
+ return;
- assert_equals(event.data.cookie_val, "");
+ assert_equals(event.data.cookie_val, '');
t.done();
});
var iframe = document.createElement('iframe');
- iframe.src = "about:blank";
+ iframe.src = 'about:blank';
iframe.onload = function() {
- var script = iframe.contentWindow.document.createElement("script");
- script.innerHTML = makeIframeString("about:blanktest");
+ var script = iframe.contentWindow.document.createElement('script');
+ script.innerHTML = make_iframe_string('about:blanktest');
iframe.contentWindow.document.body.appendChild(script);
};
document.body.appendChild(iframe);
-}, "Cookies set in an about:blank frame do not modify the suborigin's document.cookie and also have an empty document.cookie");
+ },
+ 'Cookies set in an about:blank frame do not modify the suborigin\'s ' +
+ 'document.cookie and also have an empty document.cookie');
// TODO(jww): Re-enabled this test after https://crbug.com/649893 is addressed.
/*
async_test(t => {
window.addEventListener('message', function(event) {
- if (event.data.test_name != "blob:test")
- return;
+ if (event.data.test_name != 'blob:test')
+ return;
- assert_equals(event.data.cookie_val, "");
+ assert_equals(event.data.cookie_val, '');
t.done();
- });
+ });
var iframe = document.createElement('iframe');
- var script = "<" + "script>" + makeIframeString("blob:test") + "<" + "/script>";
+ var script = '<' + 'script>' + make_iframe_string('blob:test') + '<' + '/script>';
var blob = new Blob([script], {type: 'text/html'});
iframe.src = URL.createObjectURL(blob);
document.body.appendChild(iframe);
-}, "Cookies set in a blob: frame do not modify the suborigin's document.cookie and also have an empty document.cookie");
- */
+ },
+ 'Cookies set in a blob: frame do not modify the suborigin\'s ' +
+ 'document.cookie and also have an empty document.cookie');
+*/
async_test(t => {
window.addEventListener('message', function(event) {
- if (event.data.test_name != "srcdoc:test")
- return;
+ if (event.data.test_name != 'srcdoc:test')
+ return;
- assert_equals(event.data.cookie_val, "");
+ assert_equals(event.data.cookie_val, '');
t.done();
- });
+ });
var iframe = document.createElement('iframe');
- var script = "<" + "script>" + makeIframeString("srcdoc:test") + "<" + "/script>";
- iframe.srcdoc = "srcdoc:" + script;
+ var script = '<' + 'script>' + make_iframe_string('srcdoc:test') + '<' + '/script>';
+ iframe.srcdoc = 'srcdoc:' + script;
document.body.appendChild(iframe);
-}, "Cookies set in a srcdoc frame do not modify the suborigin's document.cookie and also have an empty document.cookie");
-
+ },
+ 'Cookies set in a srcdoc frame do not modify the suborigin\'s ' +
+ 'document.cookie and also have an empty document.cookie');
</script>
</body>
</html>
« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-change-document-domain.html ('k') | third_party/WebKit/LayoutTests/http/tests/security/suborigins/suborigin-cssRules.php » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698