[webkit-reviews] review requested: [Bug 30421] Web Inspector: Local Files can show up on cookies list : [Attachment 41261] Fix

[bugzilla-daemon at webkit.org]

Brian Weinstein <bweinstein at apple.com> has asked  for review:
Bug 30421: Web Inspector: Local Files can show up on cookies list
https://bugs.webkit.org/show_bug.cgi?id=30421

Attachment 41261: Fix
https://bugs.webkit.org/attachment.cgi?id=41261&action=review

------- Additional Comments from Brian Weinstein <bweinstein at apple.com>
I am not sure if file URLs can create cookies/should show up in the Cookies
list, if they should, then the if statement should be changed to:

if (protocolInHTTPFamily() || protocolIs("file")), but I'm not sure which of
these is correct, this patch works, but will not show cookies from a file URL,
I'm not sure if that is correct behavior or not.