Closed Bug 1376019 Opened 2 years ago Closed 2 years ago

calc() expressions are rejected, in radial-gradient() radius components


(Core :: CSS Parsing and Computation, defect, P3)






(Reporter: dholbert, Unassigned)


(Blocks 2 open bugs)


(Keywords: dev-doc-complete)


(1 file)

Attached file testcase 1
 1. Load attached testcase.

EXPECTED RESULTS: Every box should be blue with a green circle (not all necessarily the same size).

ACTUAL RESULTS: The second group of boxes (under the "calc() in radii" label) are all empty.

It looks like we're rejecting calc() expressions when parsing the radial-gradient radius (or radii, in the case of ellipse)
Chrome 61, Edge 15, and Safari 10.1 all give EXPECTED RESULTS.

Firefox Nightly 56.0a1 (2017-06-23) gives ACTUAL RESULTS.  So we're on our own on this.
Stylo also gives expected results here. Do we want to do anything before Stylo? If not, we should probably mark it as a dependency of bug 1243581.
(In reply to Emilio Cobos Álvarez [:emilio] from comment #2)
> Stylo also gives expected results here. Do we want to do anything before
> Stylo?

As in bug 1363349 comment 8: I don't think we *need* to do anything before stylo. But it might also be pretty straightforward to fix this before stylo, if we want to.
Priority: -- → P3
Marking as part of documentation plan for intentional stylo differences
Keywords: dev-doc-needed
As expected, this is fixed by stylo (i.e. you get expected results if layout.css.servo.enabled is set to true) -- and stylo seems to be riding the trains with 57.

So, I don't think there's anything to gain by leaving this open, so resolving as FIXED by stylo.

BTW, I tried to mark this as depending on meta bug 1243581 (stylo), but I can't, because that'd create a cycle in bugzilla. This bug here is already marked as blocking bug 1365771 (stylo-behavior-changes), which blocks bug 1374034 (stylo-release), which blocks bug 1243581 (stylo).  So if this bug were to depend on bug 1243581 (stylo), that'd be a cycle.

To the extent we care about tracking the fact that this was fixed-by-stylo: I'll let someone who knows more about our stylo tracking bugs sort out how best to update the dependencies here to reflect this without creating a cycle.
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.