+++ This bug was initially created as a clone of Bug #908692 +++

Comment on attachment 796866 [details] [diff] [review] fix-getelement Review of attachment 796866 [details] [diff] [review]: ----------------------------------------------------------------- There are some other implementations of nsIContentPermissionRequest. Most of them are ok, but the one in DesktopNotification.cpp also throws. And there's http://mxr.mozilla.org/mozilla-central/source/dom/base/nsContentPermissionHelper.cpp#88 which throws under certain circumstances and I'm not sure if that one needs to be changed. Do you know when is that one used? ::: browser/components/nsBrowserGlue.js @@ +1734,5 @@ > QueryInterface: XPCOMUtils.generateQI([Ci.nsIContentPermissionPrompt]), > > _getBrowserForRequest: function (aRequest) { > + // "element" is only defined in e10s mode. > + var browser = aRequest.element; s/var/let/

(In reply to :Felipe Gomes from comment #2) > Comment on attachment 796866 [details] [diff] [review] > fix-getelement > > Review of attachment 796866 [details] [diff] [review]: > ----------------------------------------------------------------- > > There are some other implementations of nsIContentPermissionRequest. Most of > them are ok, but the one in DesktopNotification.cpp also throws. Ups don't know why I didn't fix. > And there's > http://mxr.mozilla.org/mozilla-central/source/dom/base/ > nsContentPermissionHelper.cpp#88 which throws under certain circumstances > and I'm not sure if that one needs to be changed. Do you know when is that > one used? This is actually the one we use in the e10s case. It proxies some of the stuff, and is the only thing that implements cross process requests. If we have no mParent doing anything with this object will throw, so we might as well do it early. > > ::: browser/components/nsBrowserGlue.js > @@ +1734,5 @@ > > QueryInterface: XPCOMUtils.generateQI([Ci.nsIContentPermissionPrompt]), > > > > _getBrowserForRequest: function (aRequest) { > > + // "element" is only defined in e10s mode. > > + var browser = aRequest.element; > > s/var/let/ :)