Re: Why does nsWindowMediator have a lock?

2014-10-13 Thread Bobby Holley
Windows in general shouldn't be touched OMT. Add a MOZ_RELEASE_ASSERT(NS_IsMainThread()) when it's touched an remove the lock. On Tue, Oct 7, 2014 at 1:29 AM, Neil wrote: > Josh Matthews wrote: > > As far as I can tell, nsWindowMediator::mListLock protects mOldestWindow >> and mTopmostWindow. H

Re: Why does nsWindowMediator have a lock?

2014-10-06 Thread Neil
Josh Matthews wrote: As far as I can tell, nsWindowMediator::mListLock protects mOldestWindow and mTopmostWindow. However, the only direct users of nsWindowMediator are the enumerators in nsAppShellWindowEnumerator.cpp, and all uses of nsIWindowMediator that I could find appear to be in JS (i

Why does nsWindowMediator have a lock?

2014-10-06 Thread Josh Matthews
As far as I can tell, nsWindowMediator::mListLock protects mOldestWindow and mTopmostWindow. However, the only direct users of nsWindowMediator are the enumerators in nsAppShellWindowEnumerator.cpp, and all uses of nsIWindowMediator that I could find appear to be in JS (ie. on the main thread)