Bug 249092

Summary:	Get rid of a redundant conditional expression previousBoundary()
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	HTML Editing	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	rniwa, webkit-bug-importer, wenson_hsieh
Priority:	P2	Keywords:	InRadar
Version:	Safari Technology Preview
Hardware:	Unspecified
OS:	Unspecified

Description Ahmad Saleem 2022-12-11 05:17:15 PST

Hi Team,

This is just another clean-up of refactoring and simplification by removing redundant expression:

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/b2b9135664b0337e297d37bc50673504b0c6eb61

Webkit GitHub Source - https://github.com/WebKit/WebKit/blob/f44c987a95d10ab86bfa3e59275c00ecfdb9d42d/Source/WebCore/editing/VisibleUnits.cpp#LL573

Just wanted to raise this bug to do simplification.

Thanks!

Comment 1 EWS 2022-12-11 23:59:20 PST

Committed 257715@main (041584a3ee1d): <https://commits.webkit.org/257715@main>

Reviewed commits have been landed. Closing PR #7451 and removing active labels.

Comment 2 Radar WebKit Bug Importer 2022-12-12 00:00:19 PST

<rdar://problem/103242457>