Bug 250280 - Simplify computation of text selection top/bottom in Legacy Root Inline
Summary: Simplify computation of text selection top/bottom in Legacy Root Inline
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-01-07 14:39 PST by Ahmad Saleem
Modified: 2023-01-14 14:40 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-01-07 14:39:58 PST
Hi Team,

This is another Blink commit and it is with own merits and issue in terms of text selection but just wanted to raise it for input purposes before doing PR:

Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/e1ac96c0207dd18057fc1f385362b1bee52bb26f

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/rendering/LegacyRootInlineBox.cpp#513 & https://searchfox.org/wubkat/source/Source/WebCore/rendering/LegacyRootInlineBox.cpp#580

One of Test Case - https://jsfiddle.net/p7hged1y/show (from Bug Report) & https://jsfiddle.net/265wb09s/1/show (latter show that the text selection covers part of second line compared to Chrome Canary 111 and Firefox Nightly 110) in Safari 16.2.

Appreciate if someone can share their input whether it is good idea to merge this or not because as per Chrome bug history, it is lesser devil's approach where neither solutions are perfect.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-01-14 14:40:16 PST
<rdar://problem/104266007>