Closed
Bug 1090925
Opened 11 years ago
Closed 11 years ago
Add keyboard support to actions
Categories
(Testing :: Marionette Client and Harness, defect, P1)
Testing
Marionette Client and Harness
Tracking
(firefox38 fixed)
RESOLVED
FIXED
mozilla38
Tracking | Status | |
---|---|---|
firefox38 | --- | fixed |
People
(Reporter: automatedtester, Assigned: chmanchester)
References
(Blocks 1 open bug, )
Details
(Keywords: pi-marionette-client, pi-marionette-server, Whiteboard: [marionette=1.0])
Attachments
(1 file, 1 obsolete file)
We need to add support for keyboard actions.
This was recently added to the webdriver spec.
Updated•11 years ago
|
Reporter | ||
Updated•11 years ago
|
Blocks: webdriver
Keywords: ateam-marionette-client,
ateam-marionette-server
Reporter | ||
Updated•11 years ago
|
Whiteboard: [marionette=1.0]
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
/r/1719 - Bug 1090925 - Add keyboard support to marionette action chains ("keyDown" and "keyUp")
Pull down this commit:
hg pull review -r 31b492e73d2c94bf9e5b5d49ed261afad5f36c2e
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Comment 5•11 years ago
|
||
/r/1719 - Bug 1090925 - Add keyboard support to marionette action chains ("keyDown" and "keyUp")
Pull down this commit:
hg pull review -r 9384f44cdb9a3be380137788effd5488ed58c843
Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 8541048 [details]
MozReview Request: bz://1090925/chmanchester
My main question at this point is whether these test cases are what is intended by the spec.
Attachment #8541048 -
Flags: feedback?(dburns)
Reporter | ||
Comment 7•11 years ago
|
||
Comment on attachment 8541048 [details]
MozReview Request: bz://1090925/chmanchester
Looks good. I think the main item that we want to test in the future is handling mouse down and mouse up while a key is pressed.
e.g. mousemove(top, left).keydown(shift).mousemove(bottom, right).keyup()
That part can be done at a later stage.
Attachment #8541048 -
Flags: feedback?(dburns) → feedback+
Assignee | ||
Comment 8•11 years ago
|
||
Assignee | ||
Comment 9•11 years ago
|
||
I updated the mozreview commits to pass modifier through to mouse events.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e9d46c52f14e
https://treeherder.mozilla.org/#/jobs?repo=try&revision=769fe5f4537a
Assignee | ||
Comment 10•11 years ago
|
||
Comment on attachment 8541048 [details]
MozReview Request: bz://1090925/chmanchester
/r/1719 - Bug 1090925 - Add keyboard support to marionette action chains ("keyDown" and "keyUp")
Pull down this commit:
hg pull review -r deabf306e480d5ca7137d8ac609ad7d6b4d8cb2c
Attachment #8541048 -
Flags: feedback+
Assignee | ||
Updated•11 years ago
|
Attachment #8541048 -
Flags: review?(dburns)
Assignee | ||
Comment 11•11 years ago
|
||
Comment on attachment 8541048 [details]
MozReview Request: bz://1090925/chmanchester
/r/1719 - Bug 1090925 - Add keyboard support to marionette action chains ("keyDown" and "keyUp")
Pull down this commit:
hg pull review -r 1404be612e5fe44c46aecf840d27e603e90feaa6
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → cmanchester
Reporter | ||
Comment 12•11 years ago
|
||
Comment on attachment 8541048 [details]
MozReview Request: bz://1090925/chmanchester
https://reviewboard.mozilla.org/r/1717/#review2813
This looks good!
Attachment #8541048 -
Flags: review?(dburns) → review+
Assignee | ||
Comment 13•11 years ago
|
||
Comment 14•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
status-firefox38:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Reporter | ||
Updated•10 years ago
|
Priority: -- → P1
Assignee | ||
Comment 17•10 years ago
|
||
Attachment #8541048 -
Attachment is obsolete: true
Attachment #8618498 -
Flags: review+
Assignee | ||
Comment 18•10 years ago
|
||
Updated•3 years ago
|
Product: Testing → Remote Protocol
Comment 19•3 years ago
|
||
Moving bugs for Marionette client due to component changes.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•