Implement Switch To Parent Frame command

RESOLVED FIXED in Firefox 44

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: ato, Assigned: automatedtester)

Tracking

(Blocks 1 bug, {pi-marionette-server})

unspecified
mozilla44
Points:
---

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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

4 years ago
(Reporter)

Comment 1

4 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

4 years ago
Assignee: nobody → dburns
(Assignee)

Comment 3

4 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)
Attachment #8675889 - Flags: review?(jgriffin) → review+
Comment on attachment 8675889 [details]
MozReview Request: Bug 1206122: Add support for switchToParentFrame r?jgriffin

https://reviewboard.mozilla.org/r/22489/#review20167

lgtm
https://hg.mozilla.org/mozilla-central/rev/fa277d3857e2
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.