Implement CSS.escape

Implement CSS.escape

Implement CSS.escape method as specified in [1]. In order to support this
use serializeIdentifier. Note that serializeIdentifier is modified to return a
boolean since if it fails we need to throw an exception.

Add a layout test based on [2]. serializeIdentifier was changed to fix these
tests.

BUG=331218
[1] https://drafts.csswg.org/cssom/#the-css.escape()-method
[2] https://github.com/mathiasbynens/CSS.escape

Committed: https://crrev.com/81430c372a9a97e91a9a95580611b9b3f1cc4ece
git-svn-id: svn://svn.chromium.org/blink/trunk@199580 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[rwlbuis, 2015-07-27 23:25:05]

rob.buis@samsung.com changed reviewers:
+ timloh@chromium.org

[rwlbuis, 2015-07-27 23:25:05]

PTAL

[Timothy Loh, 2015-07-28 04:47:12]

On 2015/07/27 23:25:05, rwlbuis wrote:
> PTAL

lgtm (FF has the same output for the added test, right?)

[rwlbuis, 2015-07-28 13:40:44]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2015-07-28 13:40:50]

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

[rwlbuis, 2015-07-28 13:42:02]

On 2015/07/28 04:47:12, Timothy Loh wrote:
> On 2015/07/27 23:25:05, rwlbuis wrote:
> > PTAL
> 
> lgtm (FF has the same output for the added test, right?)

FF fails subtests, but that looks like a bug to me, reading the identifier
serialization rules. It seems to insert an unneeded space character in the >=
0x80 case.

[commit-bot: I haz the power, 2015-07-28 14:52:24]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199580

[commit-bot: I haz the power, 2015-09-23 11:50:48]

Patchset 3 (id:??) landed as
https://crrev.com/81430c372a9a97e91a9a95580611b9b3f1cc4ece