(In reply to Boris Zbarsky [:bzbarsky, bz on IRC] from comment #0) > * I'm not sure what devtools is doing and why; needs investigation. Julian just asked about the result in actors/inspector/inspector.js in https://bugzilla.mozilla.org/show_bug.cgi?id=1604126#c3 - I believe that one should go away. The other DevTools check could probably just be changed to check system principal here https://searchfox.org/mozilla-central/rev/c61720a7d0c094d772059f9d6a7844eb7619f107/devtools/server/actors/inspector/utils.js#160-165. I'd have to double check but I think that condition was needed for something related to XBL (Bug 1187482) and I also odon't remember why we went with "xul document" and not "chrome document" in the first place.
Bug 1604299 Comment 1 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Boris Zbarsky [:bzbarsky, bz on IRC] from comment #0) > * I'm not sure what devtools is doing and why; needs investigation. Julian just asked about the result in actors/inspector/inspector.js in https://bugzilla.mozilla.org/show_bug.cgi?id=1604126#c3 - I believe that one should go away. The other DevTools check could probably just be changed to check system principal here https://searchfox.org/mozilla-central/rev/c61720a7d0c094d772059f9d6a7844eb7619f107/devtools/server/actors/inspector/utils.js#160-165. I'd have to double check but I think that condition was needed for something related to XBL (Bug 1187482) and I also don't remember why we went with "xul document" and not "chrome document" in the first place.