[webkit-reviews] review requested: [Bug 177641] Web Inspector: Include Beacon and Ping requests in Network tab : [Attachment 322258] [PATCH] Proposed Fix

[bugzilla-daemon at webkit.org]

Joseph Pecoraro <joepeck at webkit.org> has asked	for review:
Bug 177641: Web Inspector: Include Beacon and Ping requests in Network tab
https://bugs.webkit.org/show_bug.cgi?id=177641

Attachment 322258: [PATCH] Proposed Fix

https://bugs.webkit.org/attachment.cgi?id=322258&action=review




--- Comment #23 from Joseph Pecoraro <joepeck at webkit.org> ---
Created attachment 322258

  --> https://bugs.webkit.org/attachment.cgi?id=322258&action=review

[PATCH] Proposed Fix

If ElCap fails, I'll put up a debug patch to see where the ElCap (non-network
session PingHandle.h impl) is completing the ping load. I removed
didReceiveBuffer since maybe that happens before didReceiveResponse and we
don't really need to complete it early.