Make CSSFontFaceRule match new spec and/or change spec to match what's implemented |
|||||||
Issue descriptionhttps://www.w3.org/TR/DOM-Level-2-Style/css.html#CSS-CSSFontFaceRule https://drafts.csswg.org/css-fonts/#om-fontface These two interfaces have no members in common. What Blink has matches the old spec and so does Gecko and WebKit: https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/css/CSSFontFaceRule.idl https://github.com/mozilla/gecko-dev/blob/master/dom/webidl/CSSFontFaceRule.webidl https://github.com/WebKit/webkit/blob/master/Source/WebCore/css/CSSFontFaceRule.idl EdgeHTML also seems to match. The only attribute has a use counter: https://www.chromestatus.com/metrics/feature/timeline/popularity/1082
,
Apr 7 2017
That spec looks incomplete, the attributes don't have any descriptions for what they're supposed to return.
,
Apr 7 2017
,
Apr 7 2017
Hopefully the spec will change instead.
,
Oct 30 2017
,
Oct 31
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 5
,
Nov 5
I think this can be closed, the definition in the draft is the same as in DOM Style level 2 now.
,
Nov 5
Yep, it matches https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/css/css_font_face_rule.idl, just need to update the comment.
,
Nov 5
,
Nov 5
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7cef42dace6a72662e5543bec2d3707e5e22eead commit 7cef42dace6a72662e5543bec2d3707e5e22eead Author: Philip Jägenstedt <foolip@chromium.org> Date: Mon Nov 05 14:20:03 2018 Update CSSFontFaceRule spec link Bug: 709013 Change-Id: I2c68f9b37609e4fc5160d43dde9078fe6a66b33e Reviewed-on: https://chromium-review.googlesource.com/c/1317608 Reviewed-by: Rune Lillesveen <futhark@chromium.org> Commit-Queue: Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#605319} [modify] https://crrev.com/7cef42dace6a72662e5543bec2d3707e5e22eead/third_party/blink/renderer/core/css/css_font_face_rule.idl |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by foolip@chromium.org
, Apr 6 2017