251042 2023-01-23 14:27:13 -0800 Editing - Code refactoring to avoid several calls to hasEditableStyle() in loop 2023-01-24 01:45:30 -0800 1 1 1 Unclassified WebKit HTML Editing WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 ahmad.saleem792 webkit-unassigned webkit-bug-importer wenson_hsieh oldest_to_newest 1927591 0 ahmad.saleem792 2023-01-23 14:27:13 -0800 Hi Team, I came across this possible merge, which is to avoid looping and reduce calls to hasEditableStyle() in VisibleUnits.cpp: WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/editing/VisibleUnits.cpp#1121 & https://searchfox.org/wubkat/source/Source/WebCore/editing/VisibleUnits.cpp#1178 Blink Commit - https://chromium.googlesource.com/chromium/blink/+/85355b065acfc6ffbf960afa2d8e175104c556fa It is just a refactoring to reduce calls, so I think it would be fine to land without any dedicated testcase. Thanks! 1927753 1 ews-feeder 2023-01-24 01:44:40 -0800 Committed 259267@main (e2e91f349dc7): <https://commits.webkit.org/259267@main> Reviewed commits have been landed. Closing PR #8993 and removing active labels. 1927756 2 webkit-bug-importer 2023-01-24 01:45:30 -0800 <rdar://problem/104592394>