As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 739898 - Stop using enablePrivilege in test_bug480509.html
: Stop using enablePrivilege in test_bug480509.html
Product: Core Graveyard
Classification: Graveyard
Component: Security: UI (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla14
Assigned To: Peter Van der Beken [:peterv]
Depends on:
Blocks: ParisBindings
  Show dependency treegraph
Reported: 2012-03-27 22:29 PDT by Peter Van der Beken [:peterv]
Modified: 2016-09-27 13:03 PDT (History)
4 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

v1 (2.07 KB, patch)
2012-03-27 22:29 PDT, Peter Van der Beken [:peterv]
bzbarsky: review+
Details | Diff | Splinter Review

Description User image Peter Van der Beken [:peterv] 2012-03-27 22:29:29 PDT
Created attachment 610020 [details] [diff] [review]

Stop using enablePrivilege, add an API to SpecialPowers to set a property to a value and use it to set
Comment 1 User image Boris Zbarsky [:bz] (still a bit busy) 2012-03-27 22:49:28 PDT
Comment on attachment 610020 [details] [diff] [review]

r=me.  I'm impressed there was no eval involved in the impl!  ;)
Comment 2 User image Josh Matthews [:jdm] 2012-03-28 01:18:22 PDT
Are you able to use |SpecialPowers.wrap(req).channel.notificationCallbacks = foo| instead?
Comment 3 User image Peter Van der Beken [:peterv] 2012-03-28 17:03:34 PDT

I'll file a new bug to try the SpecialPowers.wrap method.
Comment 4 User image Peter Van der Beken [:peterv] 2012-03-28 17:04:48 PDT
Comment 5 User image Marco Bonardo [::mak] 2012-03-29 08:54:15 PDT
Comment 6 User image Matt Brubeck (:mbrubeck) 2012-03-29 08:54:40 PDT

Note You need to log in before you can comment on or make changes to this bug.