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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/fetch/resources/thorough-util.js

Issue 2591723003: [Fetch API] Remove Headers.prototype.getAll() (Closed)
Patch Set: Update get. Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 var SCOPE = BASE_ORIGIN + 1 var SCOPE = BASE_ORIGIN +
2 '/fetch/resources/thorough-iframe.html?' + TEST_OPTIONS; 2 '/fetch/resources/thorough-iframe.html?' + TEST_OPTIONS;
3 var IFRAME_ORIGIN = BASE_ORIGIN; 3 var IFRAME_ORIGIN = BASE_ORIGIN;
4 var BASE_URL = BASE_ORIGIN + 4 var BASE_URL = BASE_ORIGIN +
5 '/serviceworker/resources/fetch-access-control.php?'; 5 '/serviceworker/resources/fetch-access-control.php?';
6 var OTHER_BASE_URL = OTHER_ORIGIN + 6 var OTHER_BASE_URL = OTHER_ORIGIN +
7 '/serviceworker/resources/fetch-access-control.php?'; 7 '/serviceworker/resources/fetch-access-control.php?';
8 var BASE_URL_WITH_USERNAME = BASE_URL.replace('://', '://user@'); 8 var BASE_URL_WITH_USERNAME = BASE_URL.replace('://', '://user@');
9 var OTHER_BASE_URL_WITH_USERNAME = OTHER_BASE_URL.replace('://', '://user@'); 9 var OTHER_BASE_URL_WITH_USERNAME = OTHER_BASE_URL.replace('://', '://user@');
10 var BASE_URL_WITH_PASSWORD = BASE_URL.replace('://', '://user:pass@'); 10 var BASE_URL_WITH_PASSWORD = BASE_URL.replace('://', '://user:pass@');
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 function headersToArray(headers) { 338 function headersToArray(headers) {
339 var ret = []; 339 var ret = [];
340 340
341 // Workaround for Firefox. iterable is not implemented yet. 341 // Workaround for Firefox. iterable is not implemented yet.
342 // This is used only by checkFetchResponseHeader, and 342 // This is used only by checkFetchResponseHeader, and
343 // checkFetchResponseHeader is used only for the header names listed below. 343 // checkFetchResponseHeader is used only for the header names listed below.
344 // FIXME: Replace it with the original code below when Firefox supports 344 // FIXME: Replace it with the original code below when Firefox supports
345 // iterable. 345 // iterable.
346 ['content-length', 'content-type', 'x-serviceworker-serverheader'].forEach( 346 ['content-length', 'content-type', 'x-serviceworker-serverheader'].forEach(
347 function(name) { 347 function(name) {
348 for (var value of headers.getAll(name)) 348 for (var header of headers){
349 ret.push([name, value]); 349 ret.push(header);
350 }
350 }); 351 });
351 352
352 // Original code:
353 // for (var header of headers) {
354 // ret.push(header);
355 // }
356
357 return ret; 353 return ret;
358 } 354 }
359 355
360 function doFetch(request) { 356 function doFetch(request) {
361 var originalURL = request.url; 357 var originalURL = request.url;
362 var params = getQueryParams(originalURL); 358 var params = getQueryParams(originalURL);
363 var init = getRequestInit(params); 359 var init = getRequestInit(params);
364 var url = params['url']; 360 var url = params['url'];
365 try { 361 try {
366 if (url) { 362 if (url) {
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
448 444
449 function executeTests(test_targets) { 445 function executeTests(test_targets) {
450 for (var i = 0; i < test_targets.length; ++i) { 446 for (var i = 0; i < test_targets.length; ++i) {
451 promise_test( 447 promise_test(
452 function(counter, t) { 448 function(counter, t) {
453 return executeTest(test_targets[counter]); 449 return executeTest(test_targets[counter]);
454 }.bind(this, i), 450 }.bind(this, i),
455 "executeTest-" + i); 451 "executeTest-" + i);
456 } 452 }
457 } 453 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698