font-feature-settings should be serialized with double-quotes |
||
Issue descriptionfont-feature-settings is serialized with single-quotes, contradicting the behavior of Firefox and Edge, and contrary to the CSSOM spec for serializing strings. This should be changed to serialize with double-quotes.
,
Jan 29 2018
,
Feb 6 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/08ddbcb08896ee7e472115a985456eeb7e8304f8 commit 08ddbcb08896ee7e472115a985456eeb7e8304f8 Author: Chris Nardi <cnardi@chromium.org> Date: Tue Feb 06 21:48:15 2018 Serialize font-variation-settings with double-quotes According to the CSSOM spec [1], all strings should be serialized with double-quotes. The axis name in font-variation-settings was previously serialized with single-quotes; change this to double-quotes to match the spec and non-WebKit browsers. [1]: https://drafts.csswg.org/cssom/#common-serializing-idioms Bug: 806412 Change-Id: Ib0da551c2799a9dc4e273ac150972a7bd5c7a34f Reviewed-on: https://chromium-review.googlesource.com/900843 Reviewed-by: Rune Lillesveen <futhark@chromium.org> Commit-Queue: Chris Nardi <cnardi@chromium.org> Cr-Commit-Position: refs/heads/master@{#534803} [modify] https://crrev.com/08ddbcb08896ee7e472115a985456eeb7e8304f8/third_party/WebKit/LayoutTests/fast/css/resources/font-variation-settings-css-support.js [modify] https://crrev.com/08ddbcb08896ee7e472115a985456eeb7e8304f8/third_party/WebKit/Source/core/css/CSSFontVariationValue.cpp |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jan 29 2018