WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
30421
Web Inspector: about protocol loads can show up on Cookies list as Local Files
https://bugs.webkit.org/show_bug.cgi?id=30421
Summary
Web Inspector: about protocol loads can show up on Cookies list as Local Files
Brian Weinstein
Reported
2009-10-15 20:07:00 PDT
Go to www.google.com, you will see a Local Files entry on the Cookies list of the Web Inspector that has all the same Cookies as Google. Google creates an about:blank in generating their page, and that is being added to the Cookies domain list, when we should only be adding resources that were loaded from HTTP protocols.
Attachments
Fix
(2.19 KB, patch)
2009-10-15 20:50 PDT
,
Brian Weinstein
no flags
Details
Formatted Diff
Diff
Fix for HTTP + File URLs
(2.30 KB, patch)
2009-10-16 10:42 PDT
,
Brian Weinstein
alice.barraclough
: review+
bweinstein
: commit-queue-
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Brian Weinstein
Comment 1
2009-10-15 20:50:07 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.
Brian Weinstein
Comment 2
2009-10-16 10:24:16 PDT
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.
Brian Weinstein
Comment 3
2009-10-16 10:42:24 PDT
Created
attachment 41295
[details]
Fix for HTTP + File URLs
Alice Liu
Comment 4
2009-10-19 12:29:27 PDT
r=me
Brian Weinstein
Comment 5
2009-10-20 13:59:18 PDT
Landed in
http://trac.webkit.org/changeset/49883
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug