Restrict font-family in @font-face to <family-name>

Restrict font-family in @font-face to <family-name>

Restrict font-family in @font-face to <family-name> by calling
consumeFamilyName directly. To make sure no generic-name is being
parsed add the test fontfaceset-invalid-family-names.html.

Behavior matches FF.

BUG=499780
Committed: https://crrev.com/3def4ceee56560151683ce688a9b96cc8a9340af
Cr-Commit-Position: refs/heads/master@{#350479}

[rwlbuis, 2015-09-24 02:57:19]

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

[rwlbuis, 2015-09-24 02:58:17]

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

[rwlbuis, 2015-09-24 03:00:02]

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

[rwlbuis, 2015-09-24 03:00:02]

Bringing this patch back post-merge.

[rwlbuis, 2015-09-24 03:00:07]

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

[commit-bot: I haz the power, 2015-09-24 03:01:21]

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

[commit-bot: I haz the power, 2015-09-24 03:01:28]

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

[commit-bot: I haz the power, 2015-09-24 03:01:30]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Timothy Loh, 2015-09-24 03:04:04]

On 2015/09/24 03:01:30, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

lgtm

[rwlbuis, 2015-09-24 03:06:00]

On 2015/09/24 03:04:04, Timothy Loh wrote:
> On 2015/09/24 03:01:30, commit-bot: I haz the power wrote:
> > No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
> > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > _not_ a full super star committer.
> > See http://www.chromium.org/getting-involved/become-a-committer
> > Note that this has nothing to do with OWNERS files.
> 
> lgtm

Thanks! That is quick...

[rwlbuis, 2015-09-24 03:07:49]

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

[commit-bot: I haz the power, 2015-09-24 03:09:08]

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

[commit-bot: I haz the power, 2015-09-24 04:57:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-24 04:59:46]

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

[koppenheffer80, 2015-09-24 07:08:19]

koppenheffer80@gmail.com changed reviewers:
+ koppenheffer80@gmail.com

[koppenheffer80, 2015-09-24 07:08:21]

lgtm