Bug 104323 - 10% createNodes regression.
Summary: 10% createNodes regression.
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Nobody
URL:
Keywords: PerfRegression
Depends on:
Blocks:
 
Reported: 2012-12-06 17:03 PST by Ojan Vafai
Modified: 2023-12-19 17:10 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Adam Barth 2012-12-06 17:28:34 PST
I'm sorry that I won't be available to follow up on these regressions.  Please feel free to roll out any patches of mine that you suspect as causing a regression.
Comment 2 Kentaro Hara 2012-12-06 17:57:13 PST
I'll take it over. Enjoy your vacation:)
Comment 3 Kentaro Hara 2012-12-06 18:12:19 PST
(In reply to comment #0)
> 10% regression: http://build.chromium.org/f/chromium/perf/linux-release-webkit-latest/dom_perf/report.html?rev=171397&graph=CreateNodes&trace=score&history=100
> 
> Hard to be sure, but the regression range looks like http://trac.webkit.org/log/?verbose=on&rev=136605&stop_rev=136590.
> 
> Could it possibly be http://trac.webkit.org/changeset/136590/ ?

As far as I see, r136590 won't be the culprit. The patch removes one member variable and if checks, which will slightly improve performance in theory. Also the patch won't affect performance of CreateNodes.

It looks a bit hard to identify a regression range.
Comment 4 Emil A Eklund 2013-01-07 16:38:32 PST
Any luck tracking this down? We are still worse of than before :(
Comment 5 Ahmad Saleem 2023-12-19 17:10:46 PST
This report is too old to be actionable at this point. We might have already actioned / fixed this. Apple will also have internal performance tracking as well (which might have this coverage).

Marking this as 'RESOLVED WONTFIX'. Please reopen if you think we still can do something here.