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

Unified Diff: LayoutTests/http/tests/fetch/script-tests/headers.js

Issue 1288263003: Normalize and update the header value checks to RFC 7230 for Fetch Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 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: LayoutTests/http/tests/fetch/script-tests/headers.js
diff --git a/LayoutTests/http/tests/fetch/script-tests/headers.js b/LayoutTests/http/tests/fetch/script-tests/headers.js
index 4dd6037237a186f84604b764124bd4815f2e63bb..e8d450f302330d2f1a66722ce370e6a34a9eb778 100644
--- a/LayoutTests/http/tests/fetch/script-tests/headers.js
+++ b/LayoutTests/http/tests/fetch/script-tests/headers.js
@@ -176,6 +176,31 @@ test(function() {
assert_equals(headers.get('a'), 'b');
assert_equals(headers.get('c'), 'd');
+ // new valid Headers values
+ headers = new Headers();
+ headers.set('a', '\xd0\xa1');
+ assert_equals(headers.get('a'), '\xd0\xa1');
+ headers.append('b', 't t');
+ assert_equals(headers.get('b'), 't t');
+ headers.append('c', 't\tt');
+ assert_equals(headers.get('c'), 't\tt');
+
+ // test for removing leading and trailing white spaces
hiroshige 2015/09/09 07:01:48 nit: How about "Tests for normalizing header value
shiva.jm 2015/09/10 10:10:26 Done.
+ headers.append('d', ' t');
+ assert_equals(headers.get('d'), 't');
+ headers.set('e', 't ');
+ assert_equals(headers.get('d'), 't');
+ headers.append('f', ' t ');
+ assert_equals(headers.get('f'), 't');
+ headers.append('g', 'test\r');
+ assert_equals(headers.get('g'), 'test');
+ headers.set('h', 'test\n');
+ assert_equals(headers.get('h'), 'test');
+ headers.append('i', 'test\r\n');
+ assert_equals(headers.get('i'), 'test');
+ headers.append('j', 'test\t');
+ assert_equals(headers.get('j'), 'test');
+
// Throw errors
INVALID_HEADER_NAMES.forEach(function(name) {
assert_throws({name: 'TypeError'},

Powered by Google App Engine
This is Rietveld 408576698