Bug 253547 - The 'length' argument of U16_FWD_1 in TextUtil::breakWord is wrong
Summary: The 'length' argument of U16_FWD_1 in TextUtil::breakWord is wrong
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-03-07 17:06 PST by Fujii Hironori
Modified: 2023-03-08 22:34 PST (History)
4 users (show)

See Also:

Attachments
WIP patch (750 bytes, patch)
2023-03-07 17:08 PST, Fujii Hironori 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2023-03-07 17:06:38 PST 
The 'length' argument of U16_FWD_1 in TextUtil::breakWord is wrong

If I applied bug#215318 patch, fast/text/midword-break-before-surrogate-pair.html is failing as assertion failure for WinCairo debug, and it is failing as infinite loop for WinCairo release.

> ASSERTION FAILED: middle >= left && middle < right
> C:\home\webkit\gc\Source\WebCore\layout/formattingContexts/inline/text/TextUtil.cpp(276) : WebCore::Layout::TextUtil::breakWord::<lambda_1>::operator ()
Comment 1 Fujii Hironori 2023-03-07 17:08:06 PST 
Created attachment 465349 [details]
WIP patch
Comment 2 Fujii Hironori 2023-03-08 19:30:42 PST 
Pull request: https://github.com/WebKit/WebKit/pull/11271
Comment 3 EWS 2023-03-08 22:33:29 PST 
Committed 261406@main (b625b7e2bbd3): <https://commits.webkit.org/261406@main>

Reviewed commits have been landed. Closing PR #11271 and removing active labels.
Comment 4 Radar WebKit Bug Importer 2023-03-08 22:34:16 PST 
<rdar://problem/106484626>