Implement parseShorthand API for shorthand properties, "overflow", "font" and "font-variant"

third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h

Index: third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h
diff --git a/third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h b/third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h
index fe5e17adbb5dfece85455150ad6138364dc24b25..924c226877696baa8969ff3f68e46ce65fcc9768 100644
--- a/third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h
+++ b/third_party/WebKit/Source/core/css/parser/CSSPropertyParserHelpers.h
@@ -111,11 +111,13 @@ bool IsCSSWideKeyword(StringView);
 
 CSSIdentifierValue* ConsumeShapeBox(CSSParserTokenRange&);
 
+enum ImplicitProperty { kNotImplicit, kImplicit };

[Bugs Nash, 2017/06/19 23:30:12]

nit: enum class is safer as it's type safe. especially if the important argument
is changed to an enum later, you don't want people being able to accidentally
pass the arguments in the wrong order. with an enum class you need to include
the prefix though, so I would shorten the enum name to Implicit.

[Jia, 2017/06/20 01:05:51]

I've changed it to enum class. However, I think ImplicitProperty is clearer in
meaning (we know what is implicit).

+
 void AddProperty(CSSPropertyID resolved_property,
                  CSSPropertyID current_shorthand,
                  const CSSValue&,
                  bool important,
-                 bool implicit,
+                 ImplicitProperty,
                  HeapVector<CSSProperty, 256>& properties);
 
 // Template implementations are at the bottom of the file for readability.