Bug 258601

Summary: Speedometer 3: A lot of CPU time spent in Element::hasRunningTransitionForProperty
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: AnimationsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED WONTFIX    
Severity: Normal CC: cdumez, cmarcelo, dino, esprehn+autocc, ews-watchlist, graouts, kangil.han, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
WIP
ews-feeder: commit-queue-
WIP2
none
WIP3 ews-feeder: commit-queue-

Ryosuke Niwa
Reported 2023-06-27 21:16:11 PDT
We're spending a significant chunk of our time (~6%) of style resolution in this one function. Reduce the overhead of calling this function.
Attachments
WIP (35.18 KB, patch)
2023-06-27 21:19 PDT, Ryosuke Niwa 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
WIP2 (14.30 KB, patch)
2023-06-28 18:20 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
WIP3 (22.62 KB, patch)
2023-06-28 19:58 PDT, Ryosuke Niwa 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2023-06-27 21:19:13 PDT
Created attachment 466837 [details] WIP
Ryosuke Niwa
Comment 2 2023-06-28 18:20:20 PDT
Created attachment 466854 [details] WIP2
Ryosuke Niwa
Comment 3 2023-06-28 19:58:00 PDT
Created attachment 466856 [details] WIP3
Radar WebKit Bug Importer
Comment 4 2023-07-04 21:17:17 PDT
<rdar://problem/111764620>
Ryosuke Niwa
Comment 5 2023-07-07 15:06:18 PDT
This turned out to be not a fruitful improvement.
Note You need to log in before you can comment on or make changes to this bug.