Currently they are completely disabled but many of them appear to work and there are probably common problems for the ones that don't. Enable the ones that work and either fix or file specific bugs on the ones that don't.

So this is a good first step. Mostly simple fixes that gets the majority of the tests working. In some cases we were relying on executeSoon to allow callbacks to be called so now we wait for specific events from content. All other test failures look warrant their own bugs.

Comment on attachment 8507175 [details] [diff] [review] patch Review of attachment 8507175 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/mozapps/extensions/test/xpinstall/browser_enabled.js @@ +6,5 @@ > gBrowser.selectedTab = gBrowser.addTab(); > gBrowser.selectedBrowser.addEventListener("load", function() { > gBrowser.selectedBrowser.removeEventListener("load", arguments.callee, true); > + gBrowser.contentWindow.addEventListener("PageLoaded", function() { > + gBrowser.contentWindow.removeEventListener("PageLoaded", arguments.callee, false); We shouldn't be adding new usage of arguments.callee - name the listener instead. Few other cases of this in the patch to fix up.