Closed
Bug 378685
Opened 18 years ago
Closed 18 years ago
Permission manager should operate on innermost URI
Categories
(Core :: Networking: Cookies, defect)
Core
Networking: Cookies
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha6
People
(Reporter: dveditz, Assigned: dcamp)
References
Details
Attachments
(1 file)
959 bytes,
patch
|
dveditz
:
review+
mconnor
:
superreview+
|
Details | Diff | Splinter Review |
The permission manager should get out the innermost URI before checking per-site permissions. Currently any nested URI (e.g. offline app using jar:?) won't have a host and will get the nsIPermissionManager::UNKNOWN_ACTION value from TestPermssion().
![]() |
||
Updated•18 years ago
|
Flags: blocking1.9?
Updated•18 years ago
|
Assignee: nobody → dcamp
Flags: blocking1.9? → blocking1.9+
Assignee | ||
Comment 1•18 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Attachment #264538 -
Flags: review?(dveditz)
Assignee | ||
Updated•18 years ago
|
Target Milestone: --- → mozilla1.9alpha5
Reporter | ||
Comment 2•18 years ago
|
||
Comment on attachment 264538 [details] [diff] [review]
use the inner URI for permission tests
r=dveditz
Attachment #264538 -
Flags: review?(dveditz) → review+
Comment 3•18 years ago
|
||
Comment on attachment 264538 [details] [diff] [review]
use the inner URI for permission tests
sr=mconnor
can we get this landed, or push the bug to a6?
Attachment #264538 -
Flags: superreview+
Comment 4•18 years ago
|
||
pushing to a6, please land when the tree reopens
Target Milestone: mozilla1.9alpha5 → mozilla1.9alpha6
Comment 5•18 years ago
|
||
dcamp, any plans to land this?
Assignee | ||
Comment 6•18 years ago
|
||
checked in
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•