803253 - JS tests in Marionette are broken on TBPL

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Jonathan Griffin (:jgriffin)]

This is due to my patch in bug 802807.  That patch breaks JS tests for desktop Firefox, since it attempts to wait for isSystemMessageListenerReady() even when not running on B2G, and it breaks emulator JS tests on TBPL, because that event is not fired after updating gecko in the emulator.

Comment 1

[Jonathan Griffin (:jgriffin)]

Attachment: Only wait for system-message-listener-ready on emulator startup,

This patch fixes the problem by only waiting for this even on emulator startup.  Thus, it won't affect non-B2G tests, it won't affect tests that run on an already-running B2G device, and it won't break due to the fact that the message it listens for doesn't get fired after updating gecko in the emulator.

It also doesn't fail if the 30s timeout elapses; we just assume then that the even never got fired and continue anyway.  I _think_ this should take care of all the potential use cases.

Comment 2

[Malini Das [:mdas] - Away, not checking bugmail]

Comment on attachment 672929 [details] [diff] [review]
Only wait for system-message-listener-ready on emulator startup,

makes sense... I'd add a comment about why we're doing this in the code though, since it's not very obvious.

Comment 3

[Jonathan Griffin (:jgriffin)]

Comment added and pushed as https://hg.mozilla.org/mozilla-central/rev/85172b4eb961

Comment 4

[Jonathan Griffin (:jgriffin)]

https://hg.mozilla.org/projects/ash/rev/9023a301f13e

Comment 5

[Jonathan Griffin (:jgriffin)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/f9166576f834