Closed Bug 821098 Opened 7 years ago Closed 7 years ago

browser_pluginnotification.js shouldn't nuke all permissions

Categories

(Firefox :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 20

People

(Reporter: keeler, Assigned: keeler)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
browser_pluginnotification.js calls Services.perms.removeAll() repeatedly. As it turns out, there are some default permissions installed by the test harness, so this shouldn't be done.
Attachment #691571 - Flags: review?(jaws)
Comment on attachment 691571 [details] [diff] [review]
patch

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

Thanks for fixing this.
Attachment #691571 - Flags: review?(jaws) → review+
https://hg.mozilla.org/mozilla-central/rev/821ceb9e2687
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
You need to log in before you can comment on or make changes to this bug.