Bug 167151

Summary:

Make calls to render() functions async

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Perf Dashboard

Assignee:

Ryosuke Niwa <rniwa>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cdumez, kling, koivisto, rniwa

Priority:

Version:

Other

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Improves efficiency	kling: review+

Description Ryosuke Niwa 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.

Comment 1 Ryosuke Niwa 2017-01-17 20:28:13 PST

Created attachment 299114 [details]
Improves efficiency

Comment 2 Ryosuke Niwa 2017-01-18 13:23:49 PST

Committed r210880: <http://trac.webkit.org/changeset/210880>