Do not accept CSS identifiers for format()

Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc

Committed: https://crrev.com/38d79508a1367a88cb66354fc47d6d939d7b7621
Cr-Commit-Position: refs/heads/master@{#391198}

[rwlbuis, 2016-05-01 23:49:44]

Description was changed from

==========
format WIP

BUG=
==========

to

==========
Do not accept identifiers for format strings

According to [1] format strings must be of type <string>, so do not accept
identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

[rwlbuis, 2016-05-01 23:50:51]

Description was changed from

==========
Do not accept identifiers for format strings

According to [1] format strings must be of type <string>, so do not accept
identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

to

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do not accept
CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

[rwlbuis, 2016-05-01 23:51:00]

Description was changed from

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do not accept
CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

to

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

[rwlbuis, 2016-05-02 02:39:09]

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

[rwlbuis, 2016-05-02 02:39:09]

PTAL.

[Timothy Loh, 2016-05-03 08:06:18]

On 2016/05/02 02:39:09, rwlbuis wrote:
> PTAL.

lgtm

[rwlbuis, 2016-05-03 10:05:06]

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

[commit-bot: I haz the power, 2016-05-03 10:05:22]

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

[commit-bot: I haz the power, 2016-05-03 11:09:09]

Description was changed from

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

to

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

[commit-bot: I haz the power, 2016-05-03 11:09:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-03 11:10:15]

Description was changed from

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc
==========

to

==========
Do not accept CSS identifiers for format()

According to [1] format() must be of type <string>, so do
not accept CSS identifiers any longer.

Behavior matched Firefox.

[1] https://drafts.csswg.org/css-fonts/#src-desc

Committed: https://crrev.com/38d79508a1367a88cb66354fc47d6d939d7b7621
Cr-Commit-Position: refs/heads/master@{#391198}
==========

[commit-bot: I haz the power, 2016-05-03 11:10:15]

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