[ServiceWorker] A header list contains lowercased header names

[ServiceWorker] A header list contains lowercased header names

https://github.com/whatwg/fetch/issues/8
https://github.com/whatwg/fetch/commit/8114af96a4e02cd3cfb6156795fe45673217f087

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

[horo, 2014-07-02 04:38:20]

yhirano@
Could you please review?

[yhirano, 2014-07-02 05:49:42]

lgtm

[horo, 2014-07-02 06:28:28]

falken@
Could you please review?

[falken, 2014-07-02 08:12:12]

lgtm

https://codereview.chromium.org/366853003/diff/1/Source/modules/serviceworker...
File Source/modules/serviceworkers/FetchHeaderList.cpp (right):

https://codereview.chromium.org/366853003/diff/1/Source/modules/serviceworker...
Source/modules/serviceworkers/FetchHeaderList.cpp:69: // whose name is |name|,
byte lowercased, from |list|.
enclose this spec quote in quotes

https://codereview.chromium.org/366853003/diff/1/Source/modules/serviceworker...
Source/modules/serviceworkers/FetchHeaderList.cpp:127: return
isValidHTTPHeaderValue(value);
FYI isValidHTTPHeaderValue has a FIXME to follow the HTTP spec, though I don't
know exactly what's different. Anyway I presume this function is used in other
places for HTTP parsing so it's OK to reuse here, but you might want to
investigate.

[horo, 2014-07-02 08:42:46]

https://codereview.chromium.org/366853003/diff/1/Source/modules/serviceworker...
File Source/modules/serviceworkers/FetchHeaderList.cpp (right):

https://codereview.chromium.org/366853003/diff/1/Source/modules/serviceworker...
Source/modules/serviceworkers/FetchHeaderList.cpp:69: // whose name is |name|,
byte lowercased, from |list|.
On 2014/07/02 08:12:12, falken wrote:
> enclose this spec quote in quotes

Done.

[horo, 2014-07-02 08:43:15]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-07-02 08:43:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/366853003/20001

[commit-bot: I haz the power, 2014-07-02 09:57:18]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15019)

[commit-bot: I haz the power, 2014-07-02 10:38:12]

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

[commit-bot: I haz the power, 2014-07-02 10:38:13]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15034)

[horo, 2014-07-02 11:07:13]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-07-02 11:08:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/366853003/20001

[commit-bot: I haz the power, 2014-07-02 13:14:15]

Change committed as 177377