Bug 237495

Summary: Replace IntRectExtent with RectEdges<int>
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: annulen, bfulgham, changseok, dino, esprehn+autocc, ews-watchlist, fred.wang, glenn, gyuyoung.kim, kondapallykalyan, pdr, ryuan.choi, sabouhallawa, sam, sergio, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
sam: review+
Patch none

Description Simon Fraser (smfr) 2022-03-04 20:53:17 PST
Replace IntRectExtent with RectEdges<int>
Comment 1 Simon Fraser (smfr) 2022-03-04 20:56:13 PST
Created attachment 453895 [details]
Patch
Comment 2 Sam Weinig 2022-03-05 08:51:21 PST
Comment on attachment 453895 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=453895&action=review

> Source/WebCore/platform/RectEdges.h:104
> +template<typename T>
> +inline RectEdges<T>& operator+=(RectEdges<T>& a, const RectEdges<T>& b)

I'd encourage you to write both += and + (you can usually have one call the other).
Comment 3 Simon Fraser (smfr) 2022-03-05 11:23:19 PST
Created attachment 453915 [details]
Patch
Comment 4 EWS 2022-03-05 22:51:29 PST
Committed r290872 (248103@main): <https://commits.webkit.org/248103@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453915 [details].
Comment 5 Radar WebKit Bug Importer 2022-03-05 22:52:21 PST
<rdar://problem/89868237>