CSS Tokenizer: Consume a single whitespace after escapes if present

CSS Tokenizer: Consume a single whitespace after escapes if present

The css-syntax spec says to consume a single whitespace after a
hex-digit escape if present.

http://dev.w3.org/csswg/css-syntax/#consume-escaped-code-point

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

[Timothy Loh, 2014-10-20 03:48:00]

timloh@chromium.org changed reviewers:
+ alancutter@chromium.org, eseidel@chromium.org, yoav@yoav.ws

[Timothy Loh, 2014-10-20 03:48:00]

Depends on https://codereview.chromium.org/662343002/ landing first (only so far
as they add tests in the same place)

[Yoav Weiss, 2014-10-20 07:42:07]

I'd like to see some extra tests for other newline/white-space chars. Otherwise,
LGTM

https://codereview.chromium.org/649493003/diff/1/Source/core/css/parser/CSSTo...
File Source/core/css/parser/CSSTokenizerTest.cpp (right):

https://codereview.chromium.org/649493003/diff/1/Source/core/css/parser/CSSTo...
Source/core/css/parser/CSSTokenizerTest.cpp:68: { "wor\\6c\n\rd", "worl d" },
Can you add tests for '\f' and '\t' as well?

[Timothy Loh, 2014-10-20 13:05:03]

The CQ bit was checked by timloh@chromium.org

[Timothy Loh, 2014-10-20 13:05:15]

https://codereview.chromium.org/649493003/diff/1/Source/core/css/parser/CSSTo...
File Source/core/css/parser/CSSTokenizerTest.cpp (right):

https://codereview.chromium.org/649493003/diff/1/Source/core/css/parser/CSSTo...
Source/core/css/parser/CSSTokenizerTest.cpp:68: { "wor\\6c\n\rd", "worl d" },
On 2014/10/20 07:42:07, Yoav Weiss wrote:
> Can you add tests for '\f' and '\t' as well?

Done.

[commit-bot: I haz the power, 2014-10-20 13:07:03]

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

[commit-bot: I haz the power, 2014-10-20 14:51:09]

Committed patchset #2 (id:20001) as 183989