Change isCSSWideKeyword() to take a StringView instead of a CSSValueID

Change isCSSWideKeyword() to take a StringView instead of a CSSValueID

This patch changes the isCSSWideKeyword function to take a StringView
instead of a CSSValueID. Currently all its (two) callers just have a
CSSParserToken so it seems more efficient to just do string comparisons
instead of hashing the string into a CSSValueID first. A subsequent
patch will use this function in CSSSyntaxDescriptor.cpp where we will
only have a string and making a CSSValueID for that seems wrong.

Committed: https://crrev.com/692c9aef14c0699ef68a6796b68c93df69b4e731
Cr-Commit-Position: refs/heads/master@{#422004}

[Timothy Loh, 2016-09-29 09:35:31]

The CQ bit was checked by timloh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-29 09:35:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Timothy Loh, 2016-09-29 09:36:24]

timloh@chromium.org changed reviewers:
+ nainar@chromium.org

[commit-bot: I haz the power, 2016-09-29 10:54:58]

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

[commit-bot: I haz the power, 2016-09-29 10:54:59]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[nainar, 2016-09-29 23:19:01]

lgtm

[Timothy Loh, 2016-09-30 00:35:40]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2016-09-30 00:35:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 01:14:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-30 01:16:11]

Description was changed from

==========
Change isCSSWideKeyword() to take a StringView instead of a CSSValueID

This patch changes the isCSSWideKeyword function to take a StringView
instead of a CSSValueID. Currently all its (two) callers just have a
CSSParserToken so it seems more efficient to just do string comparisons
instead of hashing the string into a CSSValueID first. A subsequent
patch will use this function in CSSSyntaxDescriptor.cpp where we will
only have a string and making a CSSValueID for that seems wrong.
==========

to

==========
Change isCSSWideKeyword() to take a StringView instead of a CSSValueID

This patch changes the isCSSWideKeyword function to take a StringView
instead of a CSSValueID. Currently all its (two) callers just have a
CSSParserToken so it seems more efficient to just do string comparisons
instead of hashing the string into a CSSValueID first. A subsequent
patch will use this function in CSSSyntaxDescriptor.cpp where we will
only have a string and making a CSSValueID for that seems wrong.

Committed: https://crrev.com/692c9aef14c0699ef68a6796b68c93df69b4e731
Cr-Commit-Position: refs/heads/master@{#422004}
==========

[commit-bot: I haz the power, 2016-09-30 01:16:12]

Patchset 1 (id:??) landed as
https://crrev.com/692c9aef14c0699ef68a6796b68c93df69b4e731
Cr-Commit-Position: refs/heads/master@{#422004}