JS tests in Marionette are broken on TBPL

RESOLVED FIXED in Firefox 18

Status

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jgriffin, Assigned: jgriffin)

Tracking

unspecified
mozilla19
Points:
---

Firefox Tracking Flags

(firefox18 fixed, firefox19 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

7 years ago
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.
Assignee

Comment 1

7 years ago
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.
Attachment #672929 - Flags: review?(mdas)
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.
Attachment #672929 - Flags: review?(mdas) → review+
Assignee

Comment 3

7 years ago
Comment added and pushed as https://hg.mozilla.org/mozilla-central/rev/85172b4eb961
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [automation-needed-in-aurora]
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.