Hi Team, I tried to look for similar bug but couldn't find and it is again based of another Blink commit: Blink Commit - https://chromium.googlesource.com/chromium/blink/+/da85d1a90375fe50a92cf46f8a49571850618294 Webkit GitHub Source - https://github.com/WebKit/WebKit/blob/fa2a14f12031baf24d5bc6d6727c3e25c5052589/Source/WebCore/rendering/style/RenderStyle.h#L1282 Test Case (Can't test) - https://jsfiddle.net/L35709ro/ Because it gives error - "There's no web exposed API usable for this test. The bug doesn't affect layout in a reliable way. Please rerun inside the layout test framework." I am happy to do PR and fix it but would appreciate input of others. Thanks!
I think the problem with this merge is that the test cases use internal APIs which are not present in Webkit. I tried it here: https://github.com/WebKit/WebKit/pull/6368 FAIL internals.updateStyleAndReturnAffectedElementCount() should be 1. Threw exception TypeError: internals.updateStyleAndReturnAffectedElementCount is not a function. (In 'internals.updateStyleAndReturnAffectedElementCount()', 'internals.updateStyleAndReturnAffectedElementCount' is undefined) FAIL internals.needsLayoutCount() should be 0. Threw exception TypeError: internals.needsLayoutCount is not a function. (In 'internals.needsLayoutCount()', 'internals.needsLayoutCount' is undefined) ____ Any solution would be appreciated. Thanks!
Close my PR with following comment: "I have to close this bug because Webkit does not support "-webkit-columns: auto " properly and also don't have related Internals Webkit Testing Framework API to land this as it is. I have added first parsing bug as "See Also" while also explained the internal testing framework thing on the bug for future reference. Closing this now." If we can land this without test case then it is good to go. Thanks!
Committed 256607@main (bc1d0765c50f): <https://commits.webkit.org/256607@main> Reviewed commits have been landed. Closing PR #6368 and removing active labels.
<rdar://problem/102278097>