Bug 250574

Summary:	Make functions expecting ComputedEffectTiming or BasicEffectTiming parameters use a const reference
Product:	WebKit	Reporter:	Antoine Quint <graouts>
Component:	Animations	Assignee:	Antoine Quint <graouts>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	dino, graouts, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	Safari Technology Preview
Hardware:	Unspecified
OS:	Unspecified

Antoine Quint

Reported 2023-01-13 07:32:32 PST

There are a few methods taking in a ComputedEffectTiming or BasicEffectTiming as parameter and not using a reference thus making unnecessary copies. These should be const references.

Attachments
Add attachment proposed patch, testcase, etc.

Antoine Quint

Comment 1 2023-01-13 07:34:00 PST

Pull request: https://github.com/WebKit/WebKit/pull/8622

EWS

Comment 2 2023-01-13 12:01:12 PST

Committed 258883@main (2e13a8bc0a16): <https://commits.webkit.org/258883@main> Reviewed commits have been landed. Closing PR #8622 and removing active labels.

Radar WebKit Bug Importer

Comment 3 2023-01-13 12:02:17 PST

<rdar://problem/104234493>

Note You need to log in before you can comment on or make changes to this bug.