Handling radio group name as case insensitive

Handling radio group name as case insensitive

Added CaseFoldingHash in the NameToGroupMap to handle group name case
insensitive.
(Replicates patch fixed in WebKit for this issue:
https://bugs.webkit.org/attachment.cgi?id=187180)

Note that some test cases in radio-group-name-case.html fails because
CaseFoldingHash doesn't conform to the specification.

R=tkent
BUG=136105
TEST=automated

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

[Habib Virji, 2014-01-29 15:04:28]

Replicates patch from webkit@
https://bug-90617-attachments.webkit.org/attachment.cgi?id=187180.

[tkent, 2014-01-29 23:28:20]

https://codereview.chromium.org/149173007/diff/1/LayoutTests/fast/forms/radio...
File LayoutTests/fast/forms/radio/radio-group-name-case-expected.txt (right):

https://codereview.chromium.org/149173007/diff/1/LayoutTests/fast/forms/radio...
LayoutTests/fast/forms/radio/radio-group-name-case-expected.txt:35: FAIL
form.elements[1].checked should be false. Was true.
There are some FAIL lines.  Are they intentional?

[Habib Virji, 2014-01-30 10:36:18]

On 2014/01/29 23:28:20, tkent wrote:
>
https://codereview.chromium.org/149173007/diff/1/LayoutTests/fast/forms/radio...
> File LayoutTests/fast/forms/radio/radio-group-name-case-expected.txt (right):
> 
>
https://codereview.chromium.org/149173007/diff/1/LayoutTests/fast/forms/radio...
> LayoutTests/fast/forms/radio/radio-group-name-case-expected.txt:35: FAIL
> form.elements[1].checked should be false. Was true.
> There are some FAIL lines.  Are they intentional?

They are failing and found you consented to it:
https://bugs.webkit.org/show_bug.cgi?id=90617#c19. 

The tests failing are above 8 bits and appears StringHash is not that good in
it. Will look into it and update on it.

[Habib Virji, 2014-01-30 12:21:29]

Updated with latest master changes.

[tkent, 2014-01-31 00:12:03]

lgtm 

On 2014/01/30 10:36:18, habib.virji wrote:
> The tests failing are above 8 bits and appears StringHash is not that good in
> it. Will look into it and update on it.

It should be described in a code comment, a test comment, or the CL description.
I'll add it to the CL description.

[commit-bot: I haz the power, 2014-01-31 00:14:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/habib.virji@samsung.com/149173007/20001

[commit-bot: I haz the power, 2014-01-31 03:32:06]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2014-01-31 03:32:10]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 03:32:29]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 03:33:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/habib.virji@samsung.com/149173007/20001

[commit-bot: I haz the power, 2014-01-31 06:21:57]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2014-01-31 06:22:06]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 06:22:20]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 06:33:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/habib.virji@samsung.com/149173007/20001

[commit-bot: I haz the power, 2014-01-31 09:29:04]

Retried try job too often on linux_blink for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2014-01-31 09:29:08]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 09:29:10]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 09:35:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/habib.virji@samsung.com/149173007/20001

[commit-bot: I haz the power, 2014-01-31 12:43:31]

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

[commit-bot: I haz the power, 2014-01-31 12:43:32]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2014-01-31 12:43:32]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 12:43:38]

CQ bit was unchecked on CL. Ignoring.

[Habib Virji, 2014-01-31 17:09:45]

The CQ bit was checked by habib.virji@samsung.com

[Habib Virji, 2014-01-31 17:09:48]

The CQ bit was unchecked by habib.virji@samsung.com

[commit-bot: I haz the power, 2014-01-31 17:09:58]

CQ bit was unchecked on CL. Ignoring.

[Habib Virji, 2014-01-31 17:14:11]

On 2014/01/31 00:12:03, tkent wrote:
> lgtm 
> 
> On 2014/01/30 10:36:18, habib.virji wrote:
> > The tests failing are above 8 bits and appears StringHash is not that good
in
> > it. Will look into it and update on it.
> 
> It should be described in a code comment, a test comment, or the CL
description.
> I'll add it to the CL description.

Thanks, was not aware about it. 

Tests failure does not seem to be related to this CL changes.

[tkent, 2014-01-31 19:25:19]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-01-31 19:25:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/habib.virji@samsung.com/149173007/20001

[commit-bot: I haz the power, 2014-01-31 20:16:49]

Change committed as 166241

[commit-bot: I haz the power, 2014-01-31 20:16:55]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 20:16:55]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 20:17:43]

CQ bit was unchecked on CL. Ignoring.