Add FetchUtils::normalizeHeaderValue( )

Add FetchUtils::normalizeHeaderValue( )
These api will be used to normalize the header values in Fetch and XHR module.

Background information:
This is splitted from [1], and will be used by [1] and [2].
[1] https://codereview.chromium.org/1288263003/
[2] https://codereview.chromium.org/1342443004/

BUG=455099
Committed: https://crrev.com/85ad52a02623aca6f70df1bfe156f9002930a518
Cr-Commit-Position: refs/heads/master@{#351053}

[shiva.jm, 2015-09-24 08:50:31]

shiva.jm@samsung.com changed reviewers:
+ hiroshige@chromium.org, japhet@chromium.org, mkwst@chromium.org,
tyoshino@chromium.org, yhirano@chromium.org

[shiva.jm, 2015-09-24 08:51:15]

pls have a look, these CL is merge from
https://codereview.chromium.org/1341273002/.
since after blink merge, we need to create new project now.

[tyoshino (SeeGerritForStatus), 2015-09-24 09:24:02]

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp (right):

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:6: #define
FetchUtilsTest_h
oh, I didn't catch this. remove the include guard

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:48: #endif //
FetchUtilsTest_h
here too

[tyoshino (SeeGerritForStatus), 2015-09-24 09:26:57]

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp (right):

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:11: #include
"wtf/text/AtomicString.h"
is this necessary?

[shiva.jm, 2015-09-24 09:41:39]

pls have a look, updated patch with comments.

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp (right):

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:6: #define
FetchUtilsTest_h
On 2015/09/24 09:24:02, tyoshino wrote:
> oh, I didn't catch this. remove the include guard

actually, its added in few test files, majority test
files are not adding these, will remove it.

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:11: #include
"wtf/text/AtomicString.h"
On 2015/09/24 09:26:57, tyoshino wrote:
> is this necessary?

yes, its needed, otherwise will get compile error.

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:48: #endif //
FetchUtilsTest_h
On 2015/09/24 09:24:02, tyoshino wrote:
> here too

Done.

[shiva.jm, 2015-09-25 14:22:33]

@Mike and Chapin, tests have passed in all bots, pls have look. 
These CL is merge from https://codereview.chromium.org/1341273002/, which had
lgtm already. 
since after blink merge, we need to create new project now.

[shiva.jm, 2015-09-28 03:52:40]

shiva.jm@samsung.com changed reviewers:
+ eae@chromium.org, haraken@chromium.org, morrita@chromium.org,
noel@chromium.org

[tyoshino (SeeGerritForStatus), 2015-09-28 06:02:09]

lgtm

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp (right):

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:11: #include
"wtf/text/AtomicString.h"
On 2015/09/24 09:41:38, shiva.jm wrote:
> On 2015/09/24 09:26:57, tyoshino wrote:
> > is this necessary?
> 
> yes, its needed, otherwise will get compile error.

How about wtf/text/WTFString.h? It's indirectly included via AtomicString.h. I
think this is what is actually needed (comparison between String and char*).

[tyoshino (SeeGerritForStatus), 2015-09-28 06:03:37]

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp (right):

https://codereview.chromium.org/1364993003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/fetch/FetchUtilsTest.cpp:6: #define
FetchUtilsTest_h
On 2015/09/24 09:41:38, shiva.jm wrote:
> On 2015/09/24 09:24:02, tyoshino wrote:
> > oh, I didn't catch this. remove the include guard
> 
> actually, its added in few test files, majority test
> files are not adding these, will remove it.

I've cleaned up the test files. Thank for the info!
https://codereview.chromium.org/1367123002/

[shiva.jm, 2015-09-28 08:22:28]

updated the test file to use header wtf/text/WTFString.h, pls have a look.

[shiva.jm, 2015-09-28 08:22:34]

The CQ bit was checked by shiva.jm@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2015-09-28 08:23:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1364993003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1364993003/40001

[tyoshino (SeeGerritForStatus), 2015-09-28 08:23:08]

lgtm

[commit-bot: I haz the power, 2015-09-28 09:43:15]

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

[commit-bot: I haz the power, 2015-09-28 09:43:16]

Dry run: This issue passed the CQ dry run.

[Mike West, 2015-09-28 10:58:12]

LGTM.

[shiva.jm, 2015-09-28 10:59:01]

The CQ bit was checked by shiva.jm@samsung.com

[commit-bot: I haz the power, 2015-09-28 10:59:14]

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

[commit-bot: I haz the power, 2015-09-28 11:04:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-28 11:04:51]

Patchset 3 (id:??) landed as
https://crrev.com/85ad52a02623aca6f70df1bfe156f9002930a518
Cr-Commit-Position: refs/heads/master@{#351053}