Closed Bug 1017820 Opened 7 years ago Closed 7 years ago
Remove the classinfo for DOMWindow
No description provided.
Comment on attachment 8431078 [details] [diff] [review] Remove the classinfo for DOMWindowUtils https://tbpl.mozilla.org/?tree=Try&rev=2739094b9ab1 I've fixed the test failures after pushing to try. Note that I removed test_bug397571.html because I don't think that test makes sense any more, as the DOMWindowUtils object should not be accessible to content.
Attachment #8431078 - Flags: review?(bzbarsky)
Comment on attachment 8431078 [details] [diff] [review] Remove the classinfo for DOMWindowUtils Nice. Sounds like that specialpowers stuff is working. ;) r=me
Attachment #8431078 - Flags: review?(bzbarsky) → review+
Either that, or I had an extremely lucky one here! :-)
So, can we now remove the security checks in nsDOMClassInfo?
I'm actually not sure. Bobby?
Flags: needinfo?(bzbarsky) → needinfo?(bobbyholley)
Probably. Which ones are you referring to? Feel free to write a patch and ask me for review.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
(In reply to :Ehsan Akhgari (lagging on bugmail, needinfo? me!) from comment #6) > So, can we now remove the security checks in nsDOMClassInfo? From bug 1018482, I'm assuming you meant nsDOMWindowUtils here.
You need to log in before you can comment on or make changes to this bug.