Use ASCII case-insensitive matching for ident-likes in the CSS parser

Use ASCII case-insensitive matching for ident-likes in the CSS parser

This CL replaces CSSParserToken::valueEqualsIgnoringCase and
CSSParserString::equalIgnoringCase with valueEqualsIgnoringASCIICase
and equalIgnoringASCIICase respectively - calling the similarly named
function in WTF.
Some cases where CSSParserToken::value() was coerced to a String is
changed to avoid the coercion - using the CSSParserToken method
directly.
Similarly some cases that use the CSSParserString overload for
equalIgnoringCase is changed to use the method on CSSParserToken.
The dead equalIgnoringCase(const CSSParserString&, ...) function in
LegacyCSSPropertyParser.cpp is removed.

BUG=581001
Committed: https://crrev.com/b03ed6724b0805b5c1092167b2cfdf581e700545
Cr-Commit-Position: refs/heads/master@{#371500}

[fs, 2016-01-25 16:34:06]

fs@opera.com changed reviewers:
+ rune@opera.com, timloh@chromium.org

[rune, 2016-01-25 18:28:41]

lgtm

https://codereview.chromium.org/1636453002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/css-parser/ascii-case-insensitive-matching.html
(right):

https://codereview.chromium.org/1636453002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/css-parser/ascii-case-insensitive-matching.html:20:
assert_equals(getComputedStyle(a).backgroundColor, "rgb(0, 128, 0)");
micro-nit: inconsistent indentation. 4 spaces in js, 2 in css.

[fs, 2016-01-26 09:25:08]

https://codereview.chromium.org/1636453002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/css-parser/ascii-case-insensitive-matching.html
(right):

https://codereview.chromium.org/1636453002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/css-parser/ascii-case-insensitive-matching.html:20:
assert_equals(getComputedStyle(a).backgroundColor, "rgb(0, 128, 0)");
On 2016/01/25 at 18:28:41, rune wrote:
> micro-nit: inconsistent indentation. 4 spaces in js, 2 in css.

Fixed.

[fs, 2016-01-26 11:21:59]

The CQ bit was checked by fs@opera.com

[fs, 2016-01-26 11:21:59]

The patchset sent to the CQ was uploaded after l-g-t-m from rune@opera.com
Link to the patchset: https://codereview.chromium.org/1636453002/#ps20001
(title: "Consistent indentation in test.")

[commit-bot: I haz the power, 2016-01-26 11:22:28]

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

[commit-bot: I haz the power, 2016-01-26 13:34:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-26 13:36:23]

Description was changed from

==========
Use ASCII case-insensitive matching for ident-likes in the CSS parser

This CL replaces CSSParserToken::valueEqualsIgnoringCase and
CSSParserString::equalIgnoringCase with valueEqualsIgnoringASCIICase
and equalIgnoringASCIICase respectively - calling the similarly named
function in WTF.
Some cases where CSSParserToken::value() was coerced to a String is
changed to avoid the coercion - using the CSSParserToken method
directly.
Similarly some cases that use the CSSParserString overload for
equalIgnoringCase is changed to use the method on CSSParserToken.
The dead equalIgnoringCase(const CSSParserString&, ...) function in
LegacyCSSPropertyParser.cpp is removed.

BUG=581001
==========

to

==========
Use ASCII case-insensitive matching for ident-likes in the CSS parser

This CL replaces CSSParserToken::valueEqualsIgnoringCase and
CSSParserString::equalIgnoringCase with valueEqualsIgnoringASCIICase
and equalIgnoringASCIICase respectively - calling the similarly named
function in WTF.
Some cases where CSSParserToken::value() was coerced to a String is
changed to avoid the coercion - using the CSSParserToken method
directly.
Similarly some cases that use the CSSParserString overload for
equalIgnoringCase is changed to use the method on CSSParserToken.
The dead equalIgnoringCase(const CSSParserString&, ...) function in
LegacyCSSPropertyParser.cpp is removed.

BUG=581001

Committed: https://crrev.com/b03ed6724b0805b5c1092167b2cfdf581e700545
Cr-Commit-Position: refs/heads/master@{#371500}
==========

[commit-bot: I haz the power, 2016-01-26 13:36:24]

Patchset 2 (id:??) landed as
https://crrev.com/b03ed6724b0805b5c1092167b2cfdf581e700545
Cr-Commit-Position: refs/heads/master@{#371500}