Support mutation of CSSFontFaceRule DOM objects from JavaScript

NEW
Assigned to

Status

()

defect
P3
normal
11 years ago
3 months ago

People

(Reporter: zwol, Assigned: emilio)

Tracking

(Blocks 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

11 years ago
The initial implementation of CSSFontFaceRule DOM objects as coded in bug 441469 does not include support for modifying those objects from JavaScript.  Specifically, there are stub methods for three interfaces, that always return NS_ERROR_NOT_IMPLEMENTED:

 nsIDOMCSSFontFaceRule::SetCssText
 nsIDOMCSSStyleDeclaration::SetCssText
 nsIDOMCSSStyleDeclaration::SetProperty

(note that there is a custom implementation of nsIDOMCSSStyleDeclaration for CSSFontFaceRule objects, currently called CSSFontDescriptorBlock and hidden away in nsCSSRules.cpp).

These are not technically difficult to implement but are not necessary for people to proceed with bug 441473, and so I am splitting their implementation from bug 441469.
Flags: wanted1.9.1?
Version: unspecified → Trunk
Flags: wanted1.9.1? → wanted1.9.1+
Reporter

Comment 1

11 years ago
These are actually much harder to implement than I thought, because we don't have a good way to make consumers of style data aware that font-face descriptors have changed.  And we're running out of time for feature work, so I'm taking this off the 1.9.1 radar.
Flags: wanted1.9.1+
Reporter

Comment 2

9 years ago
I no longer work for Mozilla, I am deassigning myself from bugs I have no intention of working on as a volunteer.
Assignee: zackw → nobody
Status: ASSIGNED → NEW
Sigh. This is severely blocking BlueGriffon's CSS Pro Editor...
Assignee

Comment 4

4 months ago

This makes testability of some @font-face bits much harder, and shouldn't be hard now, I'd think...

See https://github.com/w3c/csswg-drafts/issues/3588.

Component: CSS Parsing and Computation → DOM: CSS Object Model
Flags: needinfo?(emilio)
Priority: -- → P3
Blocks: 1529054
Assignee

Comment 5

3 months ago

And fix change handling for RemoveProperty, which was supported but didn't
invalidate anything.

Assignee

Updated

3 months ago
Assignee: nobody → emilio
Flags: needinfo?(emilio)
You need to log in before you can comment on or make changes to this bug.