Bug 249092 - Get rid of a redundant conditional expression previousBoundary()
Summary: Get rid of a redundant conditional expression previousBoundary()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-12-11 05:17 PST by Ahmad Saleem
Modified: 2022-12-12 00:00 PST (History)
3 users (show)

See Also:

Attachments
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 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>