[Webkit-unassigned] [Bug 158239] Notify client immediately if network session doesn't exist for a synchronous XHR load.

[bugzilla-daemon at webkit.org]

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

--- Comment #6 from Yongjun Zhang <yongjun_zhang at apple.com> ---
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > Comment on attachment 280188 [details]
> > > Notify the client immediately when the network session doesn't exist for a
> > > sync XHR request.
> > > 
> > > Why is this only an issue for sync loads? It seems like it's an issue for
> > > async loads, too.
> > > 
> > > Sync loads cause a larger issue because the WebProcess is purposefully
> > > hanging for them, but isn't it also bad for the WebProcess to think an async
> > > load is happening.... forever...? Even though it will never happen?
> > 
> > I wasn't able to reproduce with async load. We could hit the point where the
> > network session is removed for an async load. However, since this only
> > happens when we closing a web page, Web Process will cancel the load
> > immediately and we won't have a long waiting WebProcess.
> 
> That's *currently* the only time a Session can be removed, but I foresee
> that could change in the future.
> 
> I don't see any downside to immediately signaling the error for all loads.
> 
> You've looked at the code more recently - Do you foresee any downside?

I don't see any downside either, and I agree signaling the error for all loads sounds more future-proof. I will submit another path. Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160601/92d7a68b/attachment.html>