[Webkit-unassigned] [Bug 28702] Workers do not exit after async operations

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Andrew Wilson <atwilson at chromium.org>  2009-08-27 11:12:13 PDT ---
(In reply to comment #3)
> (In reply to comment #2)
> > That code wasn't removed - it was just moved out of WorkerScriptController
> > (since it's not needed for SharedWorkers) and into DedicatedWorkerContext.
> 
> The code reported pending activity after any JavaScript evaluation - but in its
> new place, it only reports after importScripts().

When I looked at the code, it did not appear to me that XHR and setTimeout()
callbacks were routed through WorkerScriptController. Only the initial script
load/evaluation went through that code - async callbacks went through the
normal message loop event processing mechanism.

If this is incorrect, I'll update the code to do the right thing - just point
me at where setTimeout handlers invoke WorkerScriptController::evaluate(). I
only found two calls to evaluate() - in WorkerThread::workerThread (initial
worker script load) and in WorkerContext::importScripts() so I don't see how
pending activity could have been reported in these other cases because
WorkerScriptController isn't involved.

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