Closed Bug 1864442 Opened 7 months ago Closed 7 months ago

css/css-values/calc-complex-sign-function-crash.html crashes

Categories

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

defect

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(2 files)

This test was imported in bug 1857211, but somehow it didn't cause a bug to be filed on it?

James do you know what might have happened?

See comment 0 :)

Flags: needinfo?(james)
Severity: -- → S2
Priority: -- → P2

Ni? To add a more exhaustive test for these.

Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/252cf80bd0c6
Implement a missing case in CalcLengthPercentageLeaf::try_op. r=tlouw,firefox-style-system-reviewers,layout-reviewers
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Flags: needinfo?(emilio)
Blocks: 1866246
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/78ad9af76606
Add a more exhaustive test for the kind of expression tested in calc-complex-sign-function-crash.html. r=tlouw
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/43321 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

I eventually got round to investigating this and it turned out to be a more serious problem than I'd anticipated :/ https://github.com/mozilla/wpt-sync/pull/2134 should fix the underlying issue.

Flags: needinfo?(james)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: