Closed Bug 1791819 Opened 2 years ago Closed 1 year ago

WebDriver:Print does not support the orientation parameter

Categories

(Remote Protocol :: Marionette, defect, P1)

Default
defect
Points:
2

Tracking

(firefox113 fixed)

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: jdescottes, Assigned: Sasha)

References

(Blocks 2 open bugs, )

Details

(Whiteboard: [webdriver:m6], [wptsync upstream] [webdriver:relnote])

Attachments

(2 files)

See https://github.com/mozilla/geckodriver/issues/2001

The webdriver print command supports an "orientation" parameter, which can either be "landscape" or "portrait", but it seems that marionette expects a "landscape" boolean instead.

The severity field is not set for this bug.
:whimboo, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(hskupin)

It would be nice to handle it at the same time as handling print for BiDi.

Severity: -- → S3
Priority: -- → P3

Setting to P2 given that we most likely want to work on that feature for BiDi in the next milestone.

Depends on: 1783086
Flags: needinfo?(hskupin)
Priority: P3 → P2
Whiteboard: [webdriver:backlog]
Product: Testing → Remote Protocol
Blocks: 1818733
No longer blocks: 1806807
Points: --- → 2
Priority: P2 → P1
Whiteboard: [webdriver:backlog] → [webdriver:m6]
Assignee: nobody → aborovova
Status: NEW → ASSIGNED
Pushed by aborovova@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d1b666a9db5
[marionette] Add support for the orientation parameter in WebDriver:Print. r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/ebcf39c1897e
[wdspec] Add tests for the orientation parameter in WebDriver:Print. r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/39164 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m6] → [webdriver:m6], [wptsync upstream]
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
Upstream PR merged by moz-wptsync-bot
Whiteboard: [webdriver:m6], [wptsync upstream] → [webdriver:m6], [wptsync upstream] [webdriver:relnote]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: