[Webkit-unassigned] [Bug 100791] ResourceLoader can start itself in cancel()

[bugzilla-daemon at webkit.org]

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





--- Comment #9 from Nate Chapin <japhet at chromium.org>  2012-11-02 11:17:51 PST ---
(In reply to comment #8)
> Actually I don't like the way how CachedResourceLoader::loadDone() triggers ResourceLoadScheduler::servePendingRequests(). In the case user just closes a page or stops loading by clicking on stop button, this just kicks off network jobs that will soon be cancelled (or give bigger troubles if not).
> 
> Should we just make CachedResourceLoader::loadDone() schedule a timer in ResourceLoadScheduler rather than serving the requests right away?

It may actually be unnecessary. ResourceLoader::releaseResources() calls ResourceLoadScheduler::remove(), which if I'm reading it correctly should start a timer for servePendingRequests().

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