Skip to content

Conversation

@chirags27
Copy link
Contributor

@chirags27 chirags27 commented Jun 29, 2023

db8fe90

REGRESSION: (264863@main) Fix regression in CSSCalcValue
https://bugs.webkit.org/show_bug.cgi?id=258684
rdar://111512503

Reviewed by Antti Koivisto.

This change fixes a regression introduced in rdar://109503971 where we
now create a step operation with no children. This makes sense for
CalcOperator::Round, but not for CalcOperator::Mod and CalcOperator::Rem

* LayoutTests/fast/css/css-calc-rem-no-children-crash-expected.txt: Added.
* LayoutTests/fast/css/css-calc-rem-no-children-crash.html: Added.
* Source/WebCore/css/calc/CSSCalcValue.cpp:
(WebCore::createCSS):

Canonical link: https://commits.webkit.org/265667@main

56fc962

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🛠 gtk
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🧪 gtk-wk2
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 tv-sim ✅ 🧪 mac-wk2-stress
✅ 🛠 🧪 merge ✅ 🛠 watch
✅ 🛠 watch-sim

@chirags27 chirags27 self-assigned this Jun 29, 2023
@chirags27 chirags27 added the CSS Cascading Style Sheets implementation label Jun 29, 2023
@chirags27 chirags27 requested review from anttijk and darinadler June 29, 2023 18:17
@nt1m nt1m requested review from nmoucht and smfr June 29, 2023 18:20
@chirags27
Copy link
Contributor Author

This fixes a regression from #14208

Copy link
Contributor

@nmoucht nmoucht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smfr
Copy link
Contributor

smfr commented Jun 30, 2023

Fix regression in CSSCalcValue

The title says it's a regression, so should give the revision at which it regressed (like REGRESSION 123@main:).

@chirags27 chirags27 changed the title Fix regression in CSSCalcValue REGRESSION: (264863@main) Fix regression in CSSCalcValue Jun 30, 2023
@chirags27 chirags27 requested review from nmoucht and smfr June 30, 2023 17:04
@chirags27 chirags27 added the merge-queue Applied to send a pull request to merge-queue label Jun 30, 2023
https://bugs.webkit.org/show_bug.cgi?id=258684
rdar://111512503

Reviewed by Antti Koivisto.

This change fixes a regression introduced in rdar://109503971 where we
now create a step operation with no children. This makes sense for
CalcOperator::Round, but not for CalcOperator::Mod and CalcOperator::Rem

* LayoutTests/fast/css/css-calc-rem-no-children-crash-expected.txt: Added.
* LayoutTests/fast/css/css-calc-rem-no-children-crash.html: Added.
* Source/WebCore/css/calc/CSSCalcValue.cpp:
(WebCore::createCSS):

Canonical link: https://commits.webkit.org/265667@main
@webkit-commit-queue
Copy link
Collaborator

Committed 265667@main (db8fe90): https://commits.webkit.org/265667@main

Reviewed commits have been landed. Closing PR #15414 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit db8fe90 into WebKit:main Jun 30, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CSS Cascading Style Sheets implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants