RESOLVED FIXED 240129
Remove abandoned WKPreference for SelectionPaintingWithoutSelectionGaps 
https://bugs.webkit.org/show_bug.cgi?id=240129
Summary Remove abandoned WKPreference for SelectionPaintingWithoutSelectionGaps
Brent Fulgham
Reported 2022-05-05 10:57:41 PDT
This setting was added in Bug 156900 for the EFL port, which has since left the WebKit project. As the port is no longer buildable, and this code is never used outside of EFL, we should remove this dead code.
Attachments
Patch (6.44 KB, patch)
2022-05-05 11:01 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2022-05-05 11:01:40 PDT
Created attachment 458894 [details] Patch
Chris Dumez
Comment 2 2022-05-05 15:55:21 PDT
Comment on attachment 458894 [details] Patch What about this check? Source/WebCore/rendering/RenderBlock.cpp: if (settings().selectionPaintingWithoutSelectionGapsEnabled())
Brent Fulgham
Comment 3 2022-05-05 16:14:41 PDT
(In reply to Chris Dumez from comment #2) > Comment on attachment 458894 [details] > Patch > > What about this check? > Source/WebCore/rendering/RenderBlock.cpp: if > (settings().selectionPaintingWithoutSelectionGapsEnabled()) Right -- this conditional is always 'false' for non-EFL builds, so there's no point in having the preference, or calling that method (and performing that test) on any active WebKit port.
EWS
Comment 4 2022-05-06 14:11:55 PDT
Committed r293923 (250370@main): <https://commits.webkit.org/250370@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 458894 [details].
Radar WebKit Bug Importer
Comment 5 2022-05-06 14:12:13 PDT
<rdar://problem/92879882>
Note You need to log in before you can comment on or make changes to this bug.