Summary: | Mismatched whitespace widths cause a wrong width of inline box | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | bfulgham, karlcow, simon.fraser, webkit-bug-importer, zalan |
Priority: | P2 | Keywords: | InRadar |
Version: | Safari Technology Preview | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Ahmad Saleem
2022-11-10 01:33:14 PST
Ahmad, I am not sure there is anything to do here. At least with regards to the test available in the chromium project. The test as-is passes on Safari Technical Preview 157, Firefox Nightly 108, Chromium 109 (In reply to Karl Dubost from comment #1) > Ahmad, > > I am not sure there is anything to do here. > At least with regards to the test available in the chromium project. > The test as-is passes on Safari Technical Preview 157, Firefox Nightly 108, > Chromium 109 This change led to failing following: Test Case - https://github.com/WebKit/WebKit/blob/main/LayoutTests/fast/text/simple-line-layout-fallback-space-glyph.html @Alan - we have following in expected file of above: <!-- webkit-test-runner [ InlineFormattingContextIntegrationEnabled=false ] --> Does this fix bug in legacy pipeline? As discussed with Alan on PR, it seems that it has some pre-requisite, which I will look into later but for time being I have closed my PR because the longer, it will be the more issue it might have and can have merge conflicts etc. So I am going to still investigate about what is the pre-requisite of this and then will try to land both in one go. Thanks! (In reply to Ahmad Saleem from comment #0) > Hi Team, > > This is another blink commit, which we can merge to fix few more edge cases > of whitespace in inline box. > > Blink Commit - > https://src.chromium.org/viewvc/blink?view=revision&revision=195307 > > Webkit Source - > https://github.com/WebKit/WebKit/blob/ > 7a292520f6b12e8d4d9001d1480474b5c83cb0f8/Source/WebCore/rendering/ > RenderBlockFlow.cpp#L4099 > > Just wanted to create a bug report before trying. > > Thanks! This was regression fix from this change: https://src.chromium.org/viewvc/blink?revision=192404&view=revision ^ I am not sure whether we have an issue about float conversion issue or not. Link - https://github.com/WebKit/WebKit/blob/703f285930dc2b9c229d5630872d7bd52858171d/Source/WebCore/rendering/RenderText.cpp#L809 Yeah, I don't think this applies to us (also IFC does not use this codepath for computing preferred widths anymore.) |