Comment on attachment 631837 [details] [diff] [review] fix ted should review the special powers API changes.

try looks good: https://tbpl.mozilla.org/?tree=Try&rev=04f82dc63a45

Comment on attachment 631837 [details] [diff] [review] fix Review of attachment 631837 [details] [diff] [review]: ----------------------------------------------------------------- I'm not wild about the SpecialPowers changes (seems like you could just expose the thread manager to accomplish some of this stuff), but I'll let ted be the final judge. ::: dom/file/test/test_lockedfile_lifetimes_nested.html @@ +23,5 @@ > > let lockedFile = fileHandle.open(); > > let lockedFile2; > + SpecialPowers.executeSoonSync(function() { SimpleTest has an executeSoon you can use. Then you won't have to make a new one in SpecialPowers.

(In reply to ben turner [:bent] from comment #4) > ::: dom/file/test/test_lockedfile_lifetimes_nested.html > @@ +23,5 @@ > > > > let lockedFile = fileHandle.open(); > > > > let lockedFile2; > > + SpecialPowers.executeSoonSync(function() { > > SimpleTest has an executeSoon you can use. Then you won't have to make a new > one in SpecialPowers. Ben, this one is special, see the "Sync". It waits for the event to be processed. I'm willing to rework it if we decide to just expose the thread manager.

(In reply to Jan Varga [:janv] from comment #5) > I'm willing to rework it if we decide to just expose the thread manager. or even the Indexed Database Manager

this adds only one tiny method to SpecialPowers