Closed
Bug 2048959
Opened 2 months ago
Closed 2 months ago
Perma TEST-UNEXPECTED-FAIL | layout/style/test/test_value_cloning.html | serialization should be nonempty for -webkit-clip-path: ellipse(closest-corner farthest-corner) - didn't expect "", but got it when Gecko 154 merges to Beta on 2026-07-20
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
154 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr115 | --- | unaffected |
| firefox-esr140 | --- | unaffected |
| firefox152 | --- | unaffected |
| firefox153 | --- | fixed |
| firefox154 | + | fixed |
People
(Reporter: abutkovits, Assigned: descalante)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
[task 2026-06-19T10:57:06.839+00:00] 10:57:06 INFO - GECKO(19140) | ### XPCOM_MEM_BLOAT_LOG defined -- logging bloat/leaks to /tmp/tmpxo1wrrrb.mozrunner/runtests_leaks_tab_pid19497.log
[task 2026-06-19T10:57:06.892+00:00] 10:57:06 INFO - GECKO(19140) | -----------------------------------------------------
[task 2026-06-19T10:57:06.893+00:00] 10:57:06 INFO - GECKO(19140) | Suppressions used:
[task 2026-06-19T10:57:06.893+00:00] 10:57:06 INFO - GECKO(19140) | count bytes template
[task 2026-06-19T10:57:06.894+00:00] 10:57:06 INFO - GECKO(19140) | 3 320 libfontconfig.so
[task 2026-06-19T10:57:06.894+00:00] 10:57:06 INFO - GECKO(19140) | 31 16288 style::global_style_data
[task 2026-06-19T10:57:06.895+00:00] 10:57:06 INFO - GECKO(19140) | 1 9496 style::sharing::SHARING_CACHE_KEY
[task 2026-06-19T10:57:06.895+00:00] 10:57:06 INFO - GECKO(19140) | 1 4104 style::bloom::BLOOM_KEY
[task 2026-06-19T10:57:06.896+00:00] 10:57:06 INFO - GECKO(19140) | -----------------------------------------------------
[task 2026-06-19T10:57:10.688+00:00] 10:57:10 INFO - TEST-INFO | started process screentopng
[task 2026-06-19T10:57:10.896+00:00] 10:57:10 INFO - TEST-INFO | screentopng: exit 0
[task 2026-06-19T10:57:10.897+00:00] 10:57:10 INFO - TEST-UNEXPECTED-FAIL | layout/style/test/test_value_cloning.html | serialization should be nonempty for -webkit-clip-path: ellipse(closest-corner farthest-corner) - didn't expect "", but got it
[task 2026-06-19T10:57:10.897+00:00] 10:57:10 INFO - SimpleTest.isnot@SimpleTest/SimpleTest.js:526:14
[task 2026-06-19T10:57:10.897+00:00] 10:57:10 INFO - iframe_loaded@layout/style/test/test_value_cloning.html:124:12
[task 2026-06-19T10:57:11.667+00:00] 10:57:11 INFO - Not taking screenshot here: see the one that was previously logged
[task 2026-06-19T10:57:11.668+00:00] 10:57:11 INFO - TEST-UNEXPECTED-FAIL | layout/style/test/test_value_cloning.html | serialization should be nonempty for shape-outside: ellipse(closest-corner farthest-corner) - didn't expect "", but got it
[task 2026-06-19T10:57:11.668+00:00] 10:57:11 INFO - SimpleTest.isnot@SimpleTest/SimpleTest.js:526:14
[task 2026-06-19T10:57:11.668+00:00] 10:57:11 INFO - iframe_loaded@layout/style/test/test_value_cloning.html:124:12
[task 2026-06-19T10:57:11.830+00:00] 10:57:11 INFO - Not taking screenshot here: see the one that was previously logged
[task 2026-06-19T10:57:11.831+00:00] 10:57:11 INFO - TEST-UNEXPECTED-FAIL | layout/style/test/test_value_cloning.html | serialization should be nonempty for clip-path: ellipse(closest-corner farthest-corner) - didn't expect "", but got it
[task 2026-06-19T10:57:11.831+00:00] 10:57:11 INFO - SimpleTest.isnot@SimpleTest/SimpleTest.js:526:14
[task 2026-06-19T10:57:11.831+00:00] 10:57:11 INFO - iframe_loaded@layout/style/test/test_value_cloning.html:124:12
[task 2026-06-19T10:57:12.497+00:00] 10:57:12 INFO - Not taking screenshot here: see the one that was previously logged
[task 2026-06-19T10:57:12.497+00:00] 10:57:12 INFO - TEST-UNEXPECTED-FAIL | layout/style/test/test_value_cloning.html | serialization should be nonempty for offset-path: ellipse(closest-corner farthest-corner) - didn't expect "", but got it
[task 2026-06-19T10:57:12.497+00:00] 10:57:12 INFO - SimpleTest.isnot@SimpleTest/SimpleTest.js:526:14
[task 2026-06-19T10:57:12.497+00:00] 10:57:12 INFO - iframe_loaded@layout/style/test/test_value_cloning.html:124:12
[task 2026-06-19T10:57:16.349+00:00] 10:57:16 INFO - TEST-PASS | layout/style/test/test_value_cloning.html | serialization should match when cloning link-parameters: param(--foo), param(--bar)
Updated•2 months ago
|
Comment 1•2 months ago
|
||
:descalante, since you are the author of the regressor, bug 2045278, could you take a look? Also, could you set the severity field?
For more information, please visit BugBot documentation.
Flags: needinfo?(descalante)
| Assignee | ||
Comment 2•2 months ago
|
||
Righttt, looks like I forgot to update the test prefs (sorry)
Flags: needinfo?(descalante)
| Assignee | ||
Updated•2 months ago
|
Assignee: nobody → descalante
| Assignee | ||
Comment 3•2 months ago
|
||
Pushed by descalante@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/3853f3f0a8a9
https://hg.mozilla.org/integration/autoland/rev/5ebe17a367f8
Enable ellipse corner parsing and computation for layout mochitests. r=emilio,firefox-style-system-reviewers
Comment 5•2 months ago
|
||
Patch is working in today's central as beta simulation.
Comment 6•2 months ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 154 Branch
| Assignee | ||
Comment 7•1 month ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D307878
Updated•1 month ago
|
Attachment #9599820 -
Flags: approval-mozilla-beta?
Comment 8•1 month ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: The feature landed in 153 bug 2037673 however, concerns about the spec surfaced and we decided to put it behind a nightly-only pref. We also need to uplift this patch as it fixes test metadata to include the pref that turns this on when relevant.
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: no
- Steps to reproduce for manual QE testing:
- Risk associated with taking this patch: low
- Explanation of risk level: Automated tests already cover possible failures
- String changes made/needed?: N/A
- Is Android affected?: no
Updated•1 month ago
|
Attachment #9599820 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•1 month ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•