[Webkit-unassigned] [Bug 106937] Cannot abort multiple XHR POSTs made to same url

[bugzilla-daemon at webkit.org]

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


Nate Chapin <japhet at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|webkit-unassigned at lists.web |japhet at chromium.org
                   |kit.org                     |




--- Comment #2 from Nate Chapin <japhet at chromium.org>  2013-01-16 11:06:02 PST ---
(In reply to comment #1)
> Confirmed with trunk.
> 
> What I'm seeing is that XHR.abort() gets down to CachedResource::removeClient(). For first two requests, inCache() is false, so we don't call allClientsRemoved(), and thus don't actually cancel ResourceHandle. Only the third request has inCache() return true, and gets actually canceled.
> 
> Nate, is this something you'd be willing to look into? To me, it just looks like allClientsRemoved() call should be made regardless of whether the resource is in cache, but perhaps there was a reason to make it conditional.

I agree, it seems obvious that allClientsRemoved() should be called even if the resource is in cache, though it's not immediately obvious whether that holds true for everything else in the if (!deleted && !hasClients() && inCache()) block.

I'll look into it.

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