font shorthand should accept font-stretch values
See also https://github.com/w3c/csswg-drafts/issues/515
Simply removing consumeFontWeightCSS21() causes these tests to fail: +fast/parser/style-script-head-test.html expected actual diff pretty diff images diff (0.62%) text image+text pass history +fast/text/font-weight-parse.html expected actual diff pretty diff text pass history +fast/text/simple-line-layout-innerText-with-newline.html reference images diff (0.03%) image pass history +http/tests/misc/acid3.html expected actual diff pretty diff images diff (2.67%) text image+text pass history
Created attachment 304955 [details] WIP
Created attachment 304961 [details] Patch
Comment on attachment 304961 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=304961&action=review > Source/WebCore/ChangeLog:12 > + accepts percentages, which are also accepted by font-size, which means those can't "those" here is ambiguous. > Source/WebCore/css/parser/CSSPropertyParserHelpers.cpp:202 > + // Values less than or equal to 0 or greater than or equal to 1000 are parse errors. Is this covered in the testing?
Committed r214324: <http://trac.webkit.org/changeset/214324>