With my patch on bug 503277 I have accidentally checked in a sleep call of 1000ms after we return from the waitFor function: + try { + // Wait until the page has been loaded + this.waitFor(function() { return self.loaded; }, aTimeout, aInterval); + this.sleep(1000); This sleep call should not be necessary, at least not with 1000ms. Right now it slows down our testing for each page load and will be more visible once we have moved all the external content to local test pages. We should investigate if we can completely remove this call or how much we can reduce the interval.

Need to do a test run before and after this change to ensure we're ok, but we should take this for 1.5.1 if nothing breaks when we take it out. if anything breaks, we'll lump this into 1.5.2

The timeout can safely removed for normal pages and should only take place for error pages. Those do not propagate the readyState=complete and we do not know when the DOM will be ready. But 1s should be more than enough. I can't see any new failures for test-runs but we are 100% faster with that change! Will be fixed with my patch on bug 602040.

Verified fixed with mozmill 1.5.1rc1