[Webkit-unassigned] [Bug 39908] Reentrant Geolocation tests crash with an assertion

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Jonathan Dixon <joth at chromium.org>  2010-11-16 10:44:51 PST ---
(From update of attachment 74002)
Looks good, although hard to follow the logic behind it.
Maybe we could move the pending cache-using requests out of the vector copy and back into the set before calling SendError(). (But after clearing the set). We could do this for both oneshots and watchers (to avoid a cache-using watcher from going into limbo if a fatal error occurred before its cache-pump timer fires)

This way we'd keep the policy logic in one place, rather than half in SendError and half in the copy-back

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