Closed
Bug 1590064
Opened 5 years ago
Closed 5 years ago
"Webdriver:TakeScreenshot" will fail if captured area exceeds upper maximum bound
Categories
(Remote Protocol :: Marionette, defect, P3)
Tracking
(firefox72 fixed)
RESOLVED
FIXED
mozilla72
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: whimboo, Assigned: thomas510111, Mentored)
Details
(Whiteboard: [lang=js])
User Story
Please read the following documentation in how to get started: https://firefox-source-docs.mozilla.org/testing/marionette/NewContributors.html
Attachments
(1 file, 3 obsolete files)
Right now Marionette only checks for the horizontal and vertical bounds when creating a screenshot:
But there is also an upper limit for the area too which I recently implemented for the CDP protocol:
We should add such a check to Marionette.
Comment 1•5 years ago
|
||
The priority flag is not set for this bug.
:automatedtester, could you have a look please?
For more information, please visit auto_nag documentation.
Flags: needinfo?(dburns)
Updated•5 years ago
|
Flags: needinfo?(dburns)
Priority: -- → P3
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D51680
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D51680
Updated•5 years ago
|
Attachment #9106285 -
Attachment is obsolete: true
Updated•5 years ago
|
Attachment #9106285 -
Attachment is obsolete: false
Assignee | ||
Comment 5•5 years ago
|
||
Updated•5 years ago
|
Attachment #9106285 -
Attachment is obsolete: true
Updated•5 years ago
|
Attachment #9107224 -
Attachment is obsolete: true
Updated•5 years ago
|
Attachment #9107027 -
Attachment is obsolete: true
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/33fb2c19be36
[marionette] Restrict maximum screenshot area. r=whimboo
Comment 7•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox72:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Updated•5 years ago
|
Assignee: nobody → thomas510111
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
•