getUrl not matching webdriver command getCurrentUrl

RESOLVED FIXED in Firefox 28

Status

defect
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: ato, Assigned: ato)

Tracking

(Blocks 1 bug)

Trunk
mozilla29
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox28 fixed, firefox29 fixed)

Details

Attachments

(1 attachment, 3 obsolete attachments)

Assignee

Description

6 years ago
Marionette uses "getUrl" for getting the current browser window's URL
while the expected webdriver command is "getCurrentUrl".

This incompatibility makes it harder to use Marionette directly
without maintaining a WebDriver-to-Marionette command conversion set.
Raised https://www.w3.org/Bugs/Public/show_bug.cgi?id=24131 as its not currently in the specification
Assignee

Updated

5 years ago
Status: NEW → ASSIGNED
QA Contact: ato
Assignee

Comment 3

5 years ago
Comment on attachment 8362490 [details] [diff] [review]
0001-Bug-941136-getUrl-not-matching-webdriver-command-get.patch

Forgot to update the Python client.  There's nothing functionally wrong with the current patch, but I'll upload a new one that covers everything in one go.
Attachment #8362490 - Flags: review?(dburns)
Assignee

Comment 4

5 years ago
Uploaded new patch that also updates the Python client.

new try: https://tbpl.mozilla.org/?tree=Try&rev=5c2a5766db49
Attachment #8362490 - Attachment is obsolete: true
Attachment #8362556 - Flags: review?(dburns)
Comment on attachment 8362556 [details] [diff] [review]
0001-Bug-941136-getUrl-not-matching-webdriver-command-get.patch

Review of attachment 8362556 [details] [diff] [review]:
-----------------------------------------------------------------

r+ if you can update the with the comment

::: testing/marionette/marionette-listener.js
@@ +1236,4 @@
>    curFrame.location = msg.json.url;
>  }
>  
> +/** Get URL of the top level browsing context. */

Can we have this multi line again
Attachment #8362556 - Flags: review?(dburns) → review+
Assignee

Comment 6

5 years ago
Attachment #8362556 - Attachment is obsolete: true
Attachment #8362905 - Flags: review+
Assignee

Updated

5 years ago
Keywords: checkin-needed
This doesn't apply on top of the other patches you requested checkin on. Please rebase and indicate the proper landing order.
Keywords: checkin-needed
Assignee

Comment 8

5 years ago
Rebased patch.  Please apply this first, then the updated patch from bug 941132.
Attachment #8362905 - Attachment is obsolete: true
Attachment #8363075 - Flags: review+
Assignee

Updated

5 years ago
Keywords: checkin-needed
Assignee

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/df11842197ff
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Assignee

Updated

5 years ago
Keywords: checkin-needed
Whiteboard: checking-needed-aurora
Assignee

Updated

4 years ago
QA Contact: ato
You need to log in before you can comment on or make changes to this bug.