Closed Bug 378685 Opened 17 years ago Closed 17 years ago

Permission manager should operate on innermost URI

Categories

(Core :: Networking: Cookies, defect)

Product:
Core
Component:
Networking: Cookies
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9alpha6

People

(Reporter: dveditz, Assigned: dcamp)

References

Details

Attachments

(1 file)

use the inner URI for permission tests
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().
Flags: blocking1.9?
Assignee: nobody → dcamp
Flags: blocking1.9? → blocking1.9+
Attachment #264538 - Flags: review?(dveditz)
Target Milestone: --- → mozilla1.9alpha5
Comment on attachment 264538 [details] [diff] [review]
use the inner URI for permission tests

r=dveditz
Attachment #264538 - Flags: review?(dveditz) → review+
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+
pushing to a6, please land when the tree reopens
Target Milestone: mozilla1.9alpha5 → mozilla1.9alpha6
dcamp, any plans to land this?
checked in
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: