[ServiceWorker] Set FetchEvent's request's headers' guard to immutable

[ServiceWorker] Set FetchEvent's request's headers' guard to immutable

The spec of ServiceWorker was changed.
https://github.com/slightlyoff/ServiceWorker/issues/559
https://github.com/slightlyoff/ServiceWorker/commit/fe2cd64e9d84c316fc02b7c29648b0c712402746

BUG=N/A
TEST=http/tests/serviceworker/

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185416

[horo, 2014-11-13 08:46:31]

Patchset #1 (id:1) has been deleted

[horo, 2014-11-13 08:46:57]

horo@chromium.org changed reviewers:
+ nhiroki@chromium.org

[horo, 2014-11-13 08:46:58]

nhiroki@
Could you please review?

[nhiroki, 2014-11-17 03:30:42]

LGTM

fyi: The spec was slightly changed (*) after you published this issue, it seems
that the change does not affect this patch though. 

(*)
https://github.com/slightlyoff/ServiceWorker/commit/11fd4b5b0c73c835e0f7546b2...

[horo, 2014-11-17 03:45:59]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-11-17 03:46:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/723963003/20001

[commit-bot: I haz the power, 2014-11-17 03:46:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-17 03:46:47]

Failed to apply patch for
LayoutTests/http/tests/serviceworker/request-end-to-end.html:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file LayoutTests/http/tests/serviceworker/request-end-to-end.html
  Hunk #1 FAILED at 41.
  1 out of 1 hunk FAILED -- saving rejects to file
LayoutTests/http/tests/serviceworker/request-end-to-end.html.rej

Patch:       LayoutTests/http/tests/serviceworker/request-end-to-end.html
Index: LayoutTests/http/tests/serviceworker/request-end-to-end.html
diff --git a/LayoutTests/http/tests/serviceworker/request-end-to-end.html
b/LayoutTests/http/tests/serviceworker/request-end-to-end.html
index
0c05233edb208d9371290e4a8ceef98fd3fce67b..103e59acf322e51ecd3d1a3885dd06cf442e230f
100644
--- a/LayoutTests/http/tests/serviceworker/request-end-to-end.html
+++ b/LayoutTests/http/tests/serviceworker/request-end-to-end.html
@@ -41,12 +41,16 @@ t.step(function() {
     function onMessage(event) {
         assert_equals(
             event.data.url,
-            location.href.substring(0, location.href.lastIndexOf('/') + 1) +
scope,
+            location.href.substring(0, location.href.lastIndexOf('/') + 1) +
+            scope,
             'request.url should be passed to onfetch event.');
         assert_equals(event.data.method, 'GET',
                       'request.method should be passed to onfetch event.');
         assert_equals(event.data.headers['user-agent'], navigator.userAgent,
                       'User-Agent header should be passed to onfetch event.')
+        assert_equals(event.data.errorNameWhileAppendingHeader, 'TypeError',
+                      'Appending a new header to the request must throw a ' +
+                      'TypeError.')
         service_worker_unregister_and_done(t, scope);
     }
 });

[horo, 2014-11-17 03:55:42]

horo@chromium.org changed reviewers:
+ haraken@chromium.org

[horo, 2014-11-17 03:55:42]

haraken@
Could you please review Source/web/ServiceWorkerGlobalScopeProxy.cpp?

Thank you.

[haraken, 2014-11-17 03:56:12]

LGTM for web/.

[horo, 2014-11-17 04:01:35]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-11-17 04:02:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/723963003/40001

[commit-bot: I haz the power, 2014-11-17 04:57:39]

Committed patchset #2 (id:40001) as 185416