Bug 1597751 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Thanks Ehsan! 

(In reply to :ehsan akhgari from comment #3)
> I think the correct solution is to modify [`nsPermissionManager::_DoImport()`](https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/extensions/permissions/nsPermissionManager.cpp#2913) to get it to save this permission for all of the combinations of {normalWindow, privateWindow}x{user's default containers}. 
I'll work on a patch for that.

I don't think this needs to be confidential since it only affects pre-release channels and can't be abused.
Revision 1 by
on 
Thanks Ehsan! 

(In reply to :ehsan akhgari from comment #3)
> I think the correct solution is to modify [`nsPermissionManager::_DoImport()`](https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/extensions/permissions/nsPermissionManager.cpp#2913) to get it to save this permission for all of the combinations of {normalWindow, privateWindow}x{user's default containers}. 

I'll work on a patch for that.

I don't think this needs to be confidential since it only affects pre-release channels and can't be abused.

Back to Bug 1597751 Comment 4