[Webkit-unassigned] [Bug 58123] Document source preload scanned repeatedly

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=58123





--- Comment #3 from Antti Koivisto <koivisto at iki.fi>  2011-04-11 10:39:06 PST ---
(In reply to comment #2)
> The fix looks good, but I'm slightly sad that we can't add a test to make sure this doesn't regress. I can't think of any directly observable effects. Do you think it would make any sense to try to add an order of magnitude perf test? http://trac.webkit.org/changeset/65614

It is kinda hard since you need to run against a (throttled) web server to trigger the preload scanner, need a huge test content to make tokenization time observable and need to construct measurement so that the load throttling itself won't dominate. I don't think it would be worth the effort.

On the other hand the issue is easily observable with a sampling profiler on real world web sites. We would probably notice if it regressed.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.