Closed Bug 958865 Opened 6 years ago Closed 6 years ago

extensions/cookie/test should use standard Services.perms

Categories

(Core :: Networking: Cookies, defect, trivial)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: mkmelin, Assigned: mkmelin)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch proposed fix (obsolete) — Splinter Review
extensions/cookie/test should use standard Services.perms 

Using a non-standard Services.permissions sure confused me...
Attachment #8358844 - Flags: review?(josh)
Comment on attachment 8358844 [details] [diff] [review]
proposed fix

Review of attachment 8358844 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense. Thanks!
Attachment #8358844 - Flags: review?(josh) → review+
With updated patch commit msg. For checkin.
Attachment #8358844 - Attachment is obsolete: true
Attachment #8358911 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/ab211cc6d9d6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.