[Webkit-unassigned] [Bug 63531] Turn PreloadScanner implementation to a thread based solution

[bugzilla-daemon at webkit.org]

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





--- Comment #23 from Gabor Loki <loki at webkit.org>  2011-06-30 02:50:53 PST ---
> Can somebody verify my results please?

I tried your patches on Gtk, and I can confirm the difference between trunk and disabled preloadScanner on PerformanceTests/Parser/html-parser.html .
Trunk:
  avg 2801.2
  median 2801
  stdev 9.57914401186244
  min 2791
  max 2828

PreloadScanner is disabled
  avg 37.6
  median 37.5
  stdev 0.8602325267042628
  min 36
  max 40

I can also confirm the same on the threaded preloadScanner, but we should take care about the jobs which are dedicated to the worker thread. The current approach does not care about the lifetime of dedicated tasks. We should cancel the pending actions on worker thread if there is no need for them (for example the parser is finished its work).

PreloadScanner is in a dedicated thread
  avg 227.05
  median 174
  stdev 143.21503936388805
  min 158
  max 692

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