Bug 167151

Summary: Make calls to render() functions async
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Perf DashboardAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, kling, koivisto, rniwa
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Improves efficiency kling: review+

Ryosuke Niwa
Reported 2017-01-17 19:43:58 PST
Right now, perf dashboard synchronously calls render() functions in various places. This turns out to be problematic when the top-level component keeps updating descendent components. For example, during a single load of the summary page on our internal dashboard, we called render() function 9444 times! Make these calls async so that we don't end up doing O(n^2) work.
Attachments
Improves efficiency (67.39 KB, patch)
2017-01-17 20:28 PST, Ryosuke Niwa 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2017-01-17 20:28:13 PST
Created attachment 299114 [details] Improves efficiency
Ryosuke Niwa
Comment 2 2017-01-18 13:23:49 PST
Committed r210880: <http://trac.webkit.org/changeset/210880>
Note You need to log in before you can comment on or make changes to this bug.