Summary: | Web Inspector: about protocol loads can show up on Cookies list as Local Files | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Weinstein <bweinstein> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Brian Weinstein <bweinstein> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | alice.barraclough, aroben, joepeck, rik, timothy | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Brian Weinstein
2009-10-15 20:07:00 PDT
Created attachment 41261 [details]
Fix
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.
Dan Bernstein sent me a test case that shows that cookies are allowed on file URLs, so it seems that this bug was mistitled, and the code check is doing the wrong thing, we should also support file:// loads to show the cookies from. Created attachment 41295 [details]
Fix for HTTP + File URLs
r=me Landed in http://trac.webkit.org/changeset/49883. |