Closed
Bug 1197146
Opened 9 years ago
Closed 9 years ago
Convert assorted element commands in listener to use new dispatching technique
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(firefox42 affected, firefox43 fixed)
RESOLVED
FIXED
mozilla43
People
(Reporter: ato, Assigned: ato)
References
Details
(Keywords: pi-marionette-server)
Attachments
(8 files)
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
automatedtester
:
review+
|
Details |
Assorted element commands implemented by listener not yet converted:
- isElementDisplayed
- getElementValueOfCssProperty
- sendKeysToElement
- getElementLocation
- clearEleemnt
- findElementContent
- findElementsContent
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ato
Status: NEW → ASSIGNED
Assignee | ||
Updated•9 years ago
|
Summary: Convert listener assorted element commands to use new dispatching technique → Convert assorted element commands in listener to use new dispatching technique
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Bug 1197146: Part 1: Prime dispatcher in listener to resolve promises
Because of the asynchronous behaviour of some internal utilities, such
as ElementManager, we need to employ promises to create a bridge over
them and the synchronous returning functions.
r=dburns
Attachment #8651058 -
Flags: review?(dburns)
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1197146: Part 2: Use dispatcher for listener findElementsContent
r=dburns
Attachment #8651059 -
Flags: review?(dburns)
Assignee | ||
Comment 4•9 years ago
|
||
Bug 1197146: Part 3: Use dispatcher for listener findElementContent
r=dburns
Attachment #8651060 -
Flags: review?(dburns)
Assignee | ||
Comment 5•9 years ago
|
||
Bug 1197146: Part 4: Use dispatcher for listener isElementSelected
r=dburns
Attachment #8651061 -
Flags: review?(dburns)
Assignee | ||
Comment 6•9 years ago
|
||
Bug 1197146: Part 5: Use dispatcher for listener getElementLocation
r=dburns
Attachment #8651062 -
Flags: review?(dburns)
Assignee | ||
Comment 7•9 years ago
|
||
Bug 1197146: Part 6: Use dispatcher for listener clearElement
r=dburns
Attachment #8651063 -
Flags: review?(dburns)
Assignee | ||
Comment 8•9 years ago
|
||
Bug 1197146: Part 7: Use dispatcher for listener isElementDisplayed
r=dburns
Attachment #8651064 -
Flags: review?(dburns)
Assignee | ||
Comment 9•9 years ago
|
||
Bug 1197146: Part 8: Use dispatcher for listener getElementValueOfCssProperty
r=dburns
Attachment #8651065 -
Flags: review?(dburns)
Assignee | ||
Updated•9 years ago
|
Keywords: ateam-marionette-server
Updated•9 years ago
|
Attachment #8651058 -
Flags: review?(dburns) → review+
Comment 10•9 years ago
|
||
Comment on attachment 8651058 [details]
MozReview Request: Bug 1197146: Part 1: Prime dispatcher in listener to resolve promises
https://reviewboard.mozilla.org/r/16833/#review15065
Ship It!
Comment 11•9 years ago
|
||
Comment on attachment 8651059 [details]
MozReview Request: Bug 1197146: Part 2: Use dispatcher for listener findElementsContent
https://reviewboard.mozilla.org/r/16835/#review15067
Ship It!
Attachment #8651059 -
Flags: review?(dburns) → review+
Comment 12•9 years ago
|
||
Comment on attachment 8651060 [details]
MozReview Request: Bug 1197146: Part 3: Use dispatcher for listener findElementContent
https://reviewboard.mozilla.org/r/16837/#review15069
Ship It!
Attachment #8651060 -
Flags: review?(dburns) → review+
Comment 13•9 years ago
|
||
Comment on attachment 8651061 [details]
MozReview Request: Bug 1197146: Part 4: Use dispatcher for listener isElementSelected
https://reviewboard.mozilla.org/r/16839/#review15071
Ship It!
Attachment #8651061 -
Flags: review?(dburns) → review+
Comment 14•9 years ago
|
||
Comment on attachment 8651062 [details]
MozReview Request: Bug 1197146: Part 5: Use dispatcher for listener getElementLocation
https://reviewboard.mozilla.org/r/16841/#review15073
Ship It!
Attachment #8651062 -
Flags: review?(dburns) → review+
Comment 15•9 years ago
|
||
Comment on attachment 8651063 [details]
MozReview Request: Bug 1197146: Part 6: Use dispatcher for listener clearElement
https://reviewboard.mozilla.org/r/16843/#review15075
Ship It!
Attachment #8651063 -
Flags: review?(dburns) → review+
Comment 16•9 years ago
|
||
Comment on attachment 8651064 [details]
MozReview Request: Bug 1197146: Part 7: Use dispatcher for listener isElementDisplayed
https://reviewboard.mozilla.org/r/16845/#review15077
Ship It!
Attachment #8651064 -
Flags: review?(dburns) → review+
Comment 17•9 years ago
|
||
Comment on attachment 8651065 [details]
MozReview Request: Bug 1197146: Part 8: Use dispatcher for listener getElementValueOfCssProperty
https://reviewboard.mozilla.org/r/16847/#review15079
Ship It!
Attachment #8651065 -
Flags: review?(dburns) → review+
Comment 18•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1e582796c14f
https://hg.mozilla.org/integration/mozilla-inbound/rev/dda01c42607b
https://hg.mozilla.org/integration/mozilla-inbound/rev/6517558635f0
https://hg.mozilla.org/integration/mozilla-inbound/rev/673255adf942
https://hg.mozilla.org/integration/mozilla-inbound/rev/031cdb1945f6
https://hg.mozilla.org/integration/mozilla-inbound/rev/2775f73389e7
https://hg.mozilla.org/integration/mozilla-inbound/rev/550872c7e38a
https://hg.mozilla.org/integration/mozilla-inbound/rev/12e2ea054a03
Comment 19•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1e582796c14f
https://hg.mozilla.org/mozilla-central/rev/dda01c42607b
https://hg.mozilla.org/mozilla-central/rev/6517558635f0
https://hg.mozilla.org/mozilla-central/rev/673255adf942
https://hg.mozilla.org/mozilla-central/rev/031cdb1945f6
https://hg.mozilla.org/mozilla-central/rev/2775f73389e7
https://hg.mozilla.org/mozilla-central/rev/550872c7e38a
https://hg.mozilla.org/mozilla-central/rev/12e2ea054a03
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Updated•9 years ago
|
Flags: needinfo?(ryanvm)
Comment 21•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/e4069d54d4b1
https://hg.mozilla.org/releases/mozilla-aurora/rev/463f3471e857
https://hg.mozilla.org/releases/mozilla-aurora/rev/f71ee054aab4
https://hg.mozilla.org/releases/mozilla-aurora/rev/6e0802b05f7b
https://hg.mozilla.org/releases/mozilla-aurora/rev/62fa60029584
https://hg.mozilla.org/releases/mozilla-aurora/rev/33d9c59ab315
https://hg.mozilla.org/releases/mozilla-aurora/rev/9aec50a751b3
https://hg.mozilla.org/releases/mozilla-aurora/rev/95ea1b781b0f
status-firefox42:
--- → fixed
Comment 22•9 years ago
|
||
Had to back this one out for Marionette test_selected.py failures.
https://treeherder.mozilla.org/logviewer.html#?job_id=1072517&repo=mozilla-aurora
https://hg.mozilla.org/releases/mozilla-aurora/rev/66e9b94c4066
Flags: needinfo?(ato)
Assignee | ||
Comment 23•9 years ago
|
||
This probably needs http://hg.mozilla.org/integration/mozilla-inbound/rev/ba43a48d3c52 (bug 1191432) uplifted as well to work.
Flags: needinfo?(ato)
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•