See bug 659350 comment 31. Modal content windows and chrome windows are missing some interfaces that normal content windows have. Jonas, Ben: should chrome windows support indexeddb in classinfo?
Er, looks like in fact chrome and modal content windows _do_ support indexeddb. Unconditionally, unlike content windows. Doug, you made that change in bug 623316. Why is the support for modal content windows unconditional?
6 years ago
indexedDB should be available everywhere (except in third party windows, but we block that in other ways).
OK, and should it be unconditionally available in modal content windows? Or conditionally?
I don't see any reason why modal content windows should be any different from regular content windows... Is there something that sets them apart that I'm missing?
Making sure sicking agrees... sicking?
That was my feeling, but it didn't match our existing code, so I just wanted to make sure I wasn't missing anything.
I'm also not sure why modal makes a difference. I could however see that being able to open a new "toplevel" window from deeply within a frame hierarchy could be used as a way to circumvent the 3rd party checks that we have. We've been kind'a relying on the popup blocker and the fact that sites don't want to open popups in order to have them not work around our 3rd party block. Is the concern that modal windows somehow changes this?
To be clear, right now modal content windows unconditionally allow indexeddb. I was assuming that's for some reason, but it starting to sound like it was just a bug...
Created attachment 555505 [details] [diff] [review] Make the set of interfaces exposed by windows saner across different window types.
6 years ago
Comment on attachment 555505 [details] [diff] [review] Make the set of interfaces exposed by windows saner across different window types. I think jst, peterv or mrbkap would make better reviewers here
Comment on attachment 555505 [details] [diff] [review] Make the set of interfaces exposed by windows saner across different window types. >--- a/dom/base/nsDOMClassInfo.cpp >+++ b/dom/base/nsDOMClassInfo.cpp > // FIXME: Bug 680903. Should ModalContentWindow really not have > // touch, performance apis and have unconditional indexeddb?? Fixed?
Er, yes. Pushed a followup as http://hg.mozilla.org/integration/mozilla-inbound/rev/7ccfcddfed82