Closed
Bug 1211501
Opened 9 years ago
Closed 9 years ago
Support for Marionette protocol level 3 in Marionette Node.js client
Categories
(Testing Graveyard :: JSMarionette, defect)
Testing Graveyard
JSMarionette
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ato, Assigned: ato)
References
Details
(Keywords: pi-marionette-client)
Attachments
(1 file)
To future-proof the JSMarionette client it should have support for level 3 of the Marionette protocol outlined in bug 1211489.
The changes required on the client side should this time be minimal as the command result data structures are not changing. However, the client should expect to receive "Response" messages and send "Command" messages as outlined in the message format description in bug 1211489.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ato
Status: NEW → ASSIGNED
Assignee | ||
Updated•9 years ago
|
Summary: Support for Marionette protocol level 3 → Support for Marionette protocol level 3 in Marionette Node.js client
Assignee | ||
Updated•9 years ago
|
Keywords: ateam-marionette-client
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8669730 -
Flags: review?(gaye)
Comment 2•9 years ago
|
||
Comment on attachment 8669730 [details] [review]
[gaia] andreastt:bug_1211501 > mozilla-b2g:master
LGTM though you seem to have a merge conflict now.
Attachment #8669730 -
Flags: review?(gaye) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 3•9 years ago
|
||
Expanded the commit message somewhat in https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=8239a716a17223f04bc641eaedb7c417f9562cef and fixed the failing tests in https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=78f56c00edeba90897f85a4ade06cb86026bcda2.
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 4•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•