newSession is sometimes fired before listener is finished registering.

RESOLVED FIXED in Firefox 43

Status

Testing
Marionette
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: yzen, Assigned: yzen)

Tracking

({ateam-marionette-server})

Trunk
mozilla43
All
Windows
ateam-marionette-server
Points:
---

Firefox Tracking Flags

(firefox43 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
We need to to always pass along the flags (both B2G and the a11y flag) on registration of the listener to make sure that everything is set in time.
(Assignee)

Comment 1

3 years ago
Created attachment 8658793 [details]
MozReview Request: Bug 1203166 - ensuring that B2G and a11y flags are passed when listener is registring.

Bug 1203166 - ensuring that B2G and a11y flags are passed when listener is registring.

---
 testing/marionette/driver.js   | 15 +++++++--------
 testing/marionette/listener.js |  3 +++
 2 files changed, 10 insertions(+), 8 deletions(-)
Attachment #8658793 - Flags: review?(ato)
Comment on attachment 8658793 [details]
MozReview Request: Bug 1203166 - ensuring that B2G and a11y flags are passed when listener is registring.

https://reviewboard.mozilla.org/r/18689/#review16709
Attachment #8658793 - Flags: review?(ato) → review+
This looks good to me!  Can you do a try run before you land?
Status: NEW → ASSIGNED
Flags: needinfo?(yzenevich)
Keywords: ateam-marionette-server
(Assignee)

Updated

3 years ago
Flags: needinfo?(yzenevich)
https://hg.mozilla.org/mozilla-central/rev/37de8cac2687
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox43: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.