Closed Bug 1202663 Opened 5 years ago Closed 5 years ago

Convert screen capture commands in listener to use new dispatching technique


(Testing :: Marionette, defect)

Not set


(firefox44 fixed)

Tracking Status
firefox44 --- fixed


(Reporter: ato, Assigned: ato)


(Blocks 1 open bug)


(Keywords: pi-marionette-server, Whiteboard: [qa-automation-blocked])


(1 file)

No description provided.
Assignee: nobody → ato
Blocks: 1197130, 1202576
Bug 1202663: Use dispatcher for screen capture command in listener

Attachment #8658184 - Flags: review?(dburns)
Comment on attachment 8658184 [details]
MozReview Request: Bug 1202663: Use dispatcher for screen capture command in listener
Attachment #8658184 - Flags: review?(dburns) → review+
Andreas, this problem is currently our #1 hang in the functional firefox-ui-tests. Sadly a fix was not part of the yesterdays 1.0.0 release via bug 1209698.

Would you mind to tell us if this is something you have the time to get fixed anytime soon? I would appreciate it. Thanks.
Flags: needinfo?(ato)
Whiteboard: [qa-automation-blocked]
I might get around to this tomorrow or early next week.
Flags: needinfo?(ato)
Scratch that, doing a new try run with both debug and optimised builds:
Depends on: 1213800
Depends on: 1213797
Flags: needinfo?(ato)
Blocks: 1213797
No longer depends on: 1213797
No longer blocks: 1213797
Depends on: 1213797
Andreas, are you going to release new versions of client and driver or do we need something else still?
Flags: needinfo?(ato)
Andreas, I wonder if we also can get this backported at least down to beta.
dburns has a better overview than me if we can do that.
Flags: needinfo?(ato) → needinfo?(dburns)
Flags: needinfo?(dburns)
Whiteboard: [qa-automation-blocked] → [checkin-needed-aurora][checkin-needed-beta][qa-automation-blocked]
Whiteboard: [checkin-needed-aurora][checkin-needed-beta][qa-automation-blocked] → [checkin-needed-beta][qa-automation-blocked]
for beta this failed to uplift :

grafting 309559:c615a2e6b269 "Bug 1202663 -  Use dispatcher for screen capture command in listener. r=AutomatedTester, r=jgriffin, a=test-only"
merging testing/marionette/driver.js
warning: conflicts during merge.
merging testing/marionette/driver.js incomplete! (edit conflicts, then use 'hg resolve --mark')
merging testing/marionette/listener.js
warning: conflicts during merge.
merging testing/marionette/listener.js incomplete! (edit conflicts, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use hg resolve and hg graft --continue)
Flags: needinfo?(ato)
clearing checkin-needed to remove it from our checkin-needed query till we have a solution for this bug. Please add this again when this bug/patch is ready for checkin, thanks!
Whiteboard: [checkin-needed-beta][qa-automation-blocked] → [qa-automation-blocked]
I’m not convinced we should put in the extra work to backport this for uplift to Aurora, but if anyone wants to they’re more than welcome to do it.

Essentially you’d have to figure out what dependencies (other patches) are required to be uplifted to Aurora before this patch can be cleanly applied.
Flags: needinfo?(ato)
You need to log in before you can comment on or make changes to this bug.