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

Unified Diff: content/test/data/service_worker/fetch_event.js

Issue 305893003: ServiceWorker: support Request.{url,method,origin,headers} [chromium] (2/3) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add browser test for Request. Created 6 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 side-by-side diff with in-line comments
Download patch
Index: content/test/data/service_worker/fetch_event.js
diff --git a/content/test/data/service_worker/fetch_event.js b/content/test/data/service_worker/fetch_event.js
index 503806b208e88cdc0f311ce6bef50f20f947ad57..123cc13b1653986b9a74b4cbaf22790370c827f7 100644
--- a/content/test/data/service_worker/fetch_event.js
+++ b/content/test/data/service_worker/fetch_event.js
@@ -2,10 +2,29 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+function getSortedHeaderString(headers) {
+ var requestHeaders = [];
+ headers.forEach(function(value, key) {
+ requestHeaders.push(key + ': ' + value + '\n');
+ });
+ requestHeaders.sort();
+ var requestHeaderString = '';
+ requestHeaders.forEach(function(header) {
+ requestHeaderString += header;
+ });
+ return requestHeaderString;
+}
+
this.onfetch = function(event) {
var headers = new HeaderMap;
headers.set('Content-Language', 'fi');
headers.set('Content-Type', 'text/html; charset=UTF-8');
+ headers.set('X-ServiceWorker-Test-RequestURL', event.request.url);
+ headers.set('X-ServiceWorker-Test-RequestMethod', event.request.method);
+ headers.set('X-ServiceWorker-Test-RequestOrigin', event.request.origin);
+ headers.set('X-ServiceWorker-Test-RequestHeaders',
+ getSortedHeaderString(event.request.headers));
falken 2014/05/30 03:39:17 I'm not sure I understand this. Why do we add requ
horo 2014/05/30 04:55:18 It was intended to make it possible to check the v
+
var response = new Response({
status: 301,
statusText: 'Moved Permanently',

Powered by Google App Engine
This is Rietveld 408576698