Closed
Bug 1206122
Opened 9 years ago
Closed 9 years ago
Implement Switch To Parent Frame command
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(firefox44 fixed)
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: ato, Assigned: automatedtester)
References
(Blocks 1 open bug)
Details
(Keywords: pi-marionette-server)
Attachments
(1 file)
Marionette is missing the Switch To Parent Frame command that WebDriver specifies. Instead Marionette currently lets you switch to the parent frame by the same mechanism that Selenium has, which is to pass an empty string to switchToFrame. We can probably leave this behaviour there for the time being, until we’ve moved existing tests and client bindings over to this new command. This was originally reported by Alexei Barancev in https://github.com/jgraham/wires/issues/22.
Reporter | ||
Updated•9 years ago
|
Blocks: webdriver
Keywords: ateam-marionette-server
Reporter | ||
Comment 1•9 years ago
|
||
I was wrong in the description. Passing an empty string to Switch To Frame will switch to the top-most frame, and not the parent frame. This, then, is about implementing Switch To Parent Frame as defined in the specification: https://w3c.github.io/webdriver/webdriver-spec.html#switch-to-parent-frame
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → dburns
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4185f316af07
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1206122: Add support for switchToParentFrame r?jgriffin This allows us from a child frame to go up one level. This is documented at http://w3c.github.io/webdriver/webdriver-spec.html#switch-to-parent-frame
Attachment #8675889 -
Flags: review?(jgriffin)
Updated•9 years ago
|
Attachment #8675889 -
Flags: review?(jgriffin) → review+
Comment 4•9 years ago
|
||
Comment on attachment 8675889 [details] MozReview Request: Bug 1206122: Add support for switchToParentFrame r?jgriffin https://reviewboard.mozilla.org/r/22489/#review20167 lgtm
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa277d3857e280f1e82e012afc587ce428feb532 Bug 1206122: Add support for switchToParentFrame r=jgriffin
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fa277d3857e2
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•