Implemented the CSSPropertyAPI for the font-family property.

Implemented the CSSPropertyAPI for the font-family property.

Part of Project Ribbon, separating the parsing logic for CSS
properties from the parser into an API.

This patch
- added CSSPropertyAPIFontFamily.cpp, which implements CSSPropertyAPI.h
- added parseSingleValue method to the CSSPropertyAPIFontFamily API
- removed CSSPropertyFontFamily from the switch
  statement in parseSingleValue, and calls the API instead.
- added api implementation details to CSSProperties.json5, indicating
  that CSSPropertyAPIFontFamily.h is generated.

A function pointer to the parseSingleValue function from the API for
the font-family property is stored in a CSSPropertyDescriptor, and is
called from CSSPropertyParser.

Originally authored in patch 2641123002 by sashab@chromium.org

BUG=668012
Review-Url: https://codereview.chromium.org/2731953004
Cr-Commit-Position: refs/heads/master@{#455988}
Committed: https://chromium.googlesource.com/chromium/src/+/b1575e90e984d8aea66570d07b7126034e9a7b8c

[Bugs Nash, 2017-03-08 05:07:26]

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

[commit-bot: I haz the power, 2017-03-08 05:07:44]

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

[Bugs Nash, 2017-03-08 05:07:52]

bugsnash@chromium.org changed reviewers:
+ rvera@chromium.org

[commit-bot: I haz the power, 2017-03-08 06:16:16]

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

[commit-bot: I haz the power, 2017-03-08 06:16:17]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Raul Vera, 2017-03-08 21:55:32]

lgtm

[Bugs Nash, 2017-03-08 22:28:47]

bugsnash@chromium.org changed reviewers:
+ meade@chromium.org

[Bugs Nash, 2017-03-08 22:28:54]

bugsnash@chromium.org changed reviewers:
- meade@chromium.org

[Bugs Nash, 2017-03-08 22:28:54]

meade@ for owners

[Bugs Nash, 2017-03-08 22:30:05]

bugsnash@chromium.org changed reviewers:
+ meade@chromium.org

[Bugs Nash, 2017-03-08 22:30:19]

meade@ for owners

[meade_UTC10, 2017-03-09 05:48:40]

lgtm

https://codereview.chromium.org/2731953004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/BUILD.gn (right):

https://codereview.chromium.org/2731953004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/BUILD.gn:538:
"$blink_core_output_dir/css/properties/CSSPropertyAPIFontFamily.h",
Nit: should these be in alphabetical order?

[Bugs Nash, 2017-03-09 20:52:50]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-03-09 20:53:18]

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, 2017-03-09 22:51:27]

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

[commit-bot: I haz the power, 2017-03-09 22:51:31]

Failed to apply patch for third_party/WebKit/Source/core/BUILD.gn:
While running git apply --index -p1;
  error: patch failed: third_party/WebKit/Source/core/BUILD.gn:535
  error: third_party/WebKit/Source/core/BUILD.gn: patch does not apply

Patch:       third_party/WebKit/Source/core/BUILD.gn
Index: third_party/WebKit/Source/core/BUILD.gn
diff --git a/third_party/WebKit/Source/core/BUILD.gn
b/third_party/WebKit/Source/core/BUILD.gn
index
59ff677779732698f374fc444b004e38e08db40a..3e9b7d8db580807969f52dfc6a2026e0c2db7942
100644
--- a/third_party/WebKit/Source/core/BUILD.gn
+++ b/third_party/WebKit/Source/core/BUILD.gn
@@ -535,6 +535,7 @@ css_properties("make_core_generated_css_property_apis") {
     "$blink_core_output_dir/css/properties/CSSPropertyAPIWebkitBoxFlex.h",
     "$blink_core_output_dir/css/properties/CSSPropertyAPIGridAutoLine.h",
     "$blink_core_output_dir/css/properties/CSSPropertyAPIOffsetAnchor.h",
+    "$blink_core_output_dir/css/properties/CSSPropertyAPIFontFamily.h",
   ]
 }

[Bugs Nash, 2017-03-10 01:09:21]

The CQ bit was checked by bugsnash@chromium.org

[Bugs Nash, 2017-03-10 01:09:21]

The patchset sent to the CQ was uploaded after l-g-t-m from meade@chromium.org,
rvera@chromium.org
Link to the patchset: https://codereview.chromium.org/2731953004/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-03-10 01:10:09]

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

[commit-bot: I haz the power, 2017-03-10 01:10:11]

This CL has an open dependency (Issue 2735093005 Patch 60001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[Bugs Nash, 2017-03-10 02:25:48]

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

[commit-bot: I haz the power, 2017-03-10 02:26:13]

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

[Bugs Nash, 2017-03-10 03:56:13]

The CQ bit was unchecked by bugsnash@chromium.org

[Bugs Nash, 2017-03-10 03:56:16]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-03-10 03:56:38]

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, 2017-03-10 04:13:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489118176462390,
"parent_rev": "c0438224434b32dff5d3e15fb1b4b7a03d7e343a", "commit_rev":
"b1575e90e984d8aea66570d07b7126034e9a7b8c"}

[commit-bot: I haz the power, 2017-03-10 04:14:10]

Description was changed from

==========
Implemented the CSSPropertyAPI for the font-family property.

Part of Project Ribbon, separating the parsing logic for CSS
properties from the parser into an API.

This patch
- added CSSPropertyAPIFontFamily.cpp, which implements CSSPropertyAPI.h
- added parseSingleValue method to the CSSPropertyAPIFontFamily API
- removed CSSPropertyFontFamily from the switch
  statement in parseSingleValue, and calls the API instead.
- added api implementation details to CSSProperties.json5, indicating
  that CSSPropertyAPIFontFamily.h is generated.

A function pointer to the parseSingleValue function from the API for
the font-family property is stored in a CSSPropertyDescriptor, and is
called from CSSPropertyParser.

Originally authored in patch 2641123002 by sashab@chromium.org

BUG=668012
==========

to

==========
Implemented the CSSPropertyAPI for the font-family property.

Part of Project Ribbon, separating the parsing logic for CSS
properties from the parser into an API.

This patch
- added CSSPropertyAPIFontFamily.cpp, which implements CSSPropertyAPI.h
- added parseSingleValue method to the CSSPropertyAPIFontFamily API
- removed CSSPropertyFontFamily from the switch
  statement in parseSingleValue, and calls the API instead.
- added api implementation details to CSSProperties.json5, indicating
  that CSSPropertyAPIFontFamily.h is generated.

A function pointer to the parseSingleValue function from the API for
the font-family property is stored in a CSSPropertyDescriptor, and is
called from CSSPropertyParser.

Originally authored in patch 2641123002 by sashab@chromium.org

BUG=668012

Review-Url: https://codereview.chromium.org/2731953004
Cr-Commit-Position: refs/heads/master@{#455988}
Committed:
https://chromium.googlesource.com/chromium/src/+/b1575e90e984d8aea66570d07b71...
==========

[commit-bot: I haz the power, 2017-03-10 04:14:11]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/b1575e90e984d8aea66570d07b71...