158286 2016-06-01 17:19:54 -0700 Vertical writing mode divs don't relayout when their horizontal writing mode containing block changes height 2023-12-26 04:22:33 -0800 1 1 1 Unclassified WebKit Text WebKit Nightly Build Unspecified Unspecified NEW P2 Normal --- 1 mmaxfield mmaxfield ahmad.saleem792 fantasai.bugs mmaxfield zalan oldest_to_newest 1198346 0 280281 mmaxfield 2016-06-01 17:19:54 -0700 Created attachment 280281 Actual result See the attached test. 1198347 1 280282 mmaxfield 2016-06-01 17:20:11 -0700 Created attachment 280282 Expected result 1198375 2 mmaxfield 2016-06-01 18:25:09 -0700 RenderBox::perpendicularContainingBlockLogicalHeight() needs to perform the same treatment to the element which dictates the return value for containingBlock()->availableLogicalHeight() as in https://bugs.webkit.org/show_bug.cgi?id=158245 1942639 3 ahmad.saleem792 2023-03-20 12:27:39 -0700 I am not able to reproduce the bug, both "Actual" and "Expected" are same and matching with Chrome Canary 113 in behavior. Can we get rid of FIXME, if it is working fine now? @Alan & @Myles - any input - https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderBox.cpp#2277 or I am reproducing this bug wrong or missed something? 1942642 4 ahmad.saleem792 2023-03-20 12:34:03 -0700 (In reply to Ahmad Saleem from comment #3) > I am not able to reproduce the bug, both "Actual" and "Expected" are same > and matching with Chrome Canary 113 in behavior. > > Can we get rid of FIXME, if it is working fine now? > > @Alan & @Myles - any input - > https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderBox. > cpp#2277 or I am reproducing this bug wrong or missed something? I came across this FIXME, while trying to look into this Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/fed69992a9de1357f400379e13fa87dbeaae14b4 and noticing that we have not imported this test case and don't have similar code. 280281 2016-06-01 17:19:54 -0700 2016-06-01 17:19:54 -0700 Actual result actual.html text/html 288 mmaxfield PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPC9oZWFkPgo8Ym9keT4KPGRpdiBpZD0iYyIg c3R5bGU9ImhlaWdodDogNzBweDsiPjxkaXY+PGRpdiBzdHlsZT0iLXdlYmtpdC13cml0aW5nLW1v ZGU6IHZlcnRpY2FsLXJsOyI+SGVsbG8gV29ybGQ8L2Rpdj48L2Rpdj48L2Rpdj4KPHNjcmlwdD4K d2luZG93LnNldFRpbWVvdXQoZnVuY3Rpb24oKSB7Cglkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgi YyIpLnN0eWxlLmhlaWdodCA9ICIyMDBweCI7Cn0sIDApOwo8L3NjcmlwdD4KPC9ib2R5Pgo8L2h0 bWw+ 280282 2016-06-01 17:20:11 -0700 2016-06-01 17:20:11 -0700 Expected result expected.html text/html 178 mmaxfield PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPC9oZWFkPgo8Ym9keT4KPGRpdiBpZD0iYyIg c3R5bGU9ImhlaWdodDogMjAwcHg7Ij48ZGl2PjxkaXYgc3R5bGU9Ii13ZWJraXQtd3JpdGluZy1t b2RlOiB2ZXJ0aWNhbC1ybDsiPkhlbGxvIFdvcmxkPC9kaXY+PC9kaXY+PC9kaXY+CjwvYm9keT4K PC9odG1sPg==