1968587 - Mochitests should not start with a 1s wait

Status: RESOLVED FIXED

(Testing :: Mochitest, defect, P2)

Description

[Florian Quèze [:florian]]

This behavior was introduced in bug 431464, because of focus issues when "Running mochitests on SeaMonkey" ("It's possible the problem also occurs on Firefox").

Example profile showing that we wait for 1s before we start doing anything: https://share.firefox.dev/4dCqsKH

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jmaher, could you have a look please?

For more information, please visit BugBot documentation.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

I think the 1 second is here

Comment 3

[Florian Quèze [:florian]]

The profile in comment 0 shows "setTimeout handler with interval 1000ms: TestRunner._makeIframe/< (chrome://mochikit/content/tests/SimpleTest/TestRunner.js:421)" so the timer would be this one: https://searchfox.org/mozilla-central/rev/b6a437e0e51dbbfa65f07ee4e1db3864624ab5ea/testing/mochitest/tests/SimpleTest/TestRunner.js#421,423

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: Bug 1968587 - Remove unnecessary 1s wait for mochitests. r=florian

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/4cb18326ced3
https://hg.mozilla.org/integration/autoland/rev/b4ba3be26043
Remove unnecessary 1s wait for mochitests. r=florian

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/b4ba3be26043