Closed Bug 1806813 Opened 2 years ago Closed 1 year ago

Enhance tests for "browsingContext.print" as PDF command

Categories

(Remote Protocol :: WebDriver BiDi, task, P1)

task
Points:
5

Tracking

(firefox112 fixed)

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: whimboo, Assigned: Sasha)

References

(Blocks 2 open bugs)

Details

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

Attachments

(2 files)

With bug 1806810 we will get a basic set of tests to make sure that the command works. But we should extend the tests to actually validate the correct output.

James suggested to use pdf.js to verify the output. Mainly because there is already existing code for that purpose. But that would be not cross-browser compliant. So I wonder what other option we would have.

James, any thoughts from your side?

Flags: needinfo?(james)
Priority: -- → P1
Summary: Enhance tests for "browsingContext.printToPDF" command → Enhance tests for "browsingContext.print" as PDF command

It is cross browser; web-platform-tests vendors a copy of pdf.js for this purpose. We already use it in the WebDriver classic tests: https://searchfox.org/mozilla-central/source/testing/web-platform/tests/webdriver/tests/print/__init__.py Porting that for BiDi should be more or less trivial.

Flags: needinfo?(james)
Assignee: nobody → aborovova
Status: NEW → ASSIGNED
Blocks: 1818733
Pushed by aborovova@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/14201ceea9ee
[wdspec] Add fixture to render PDF to png. r=webdriver-reviewers,whimboo,jgraham,jdescottes
https://hg.mozilla.org/integration/autoland/rev/4dfc3d3988f1
[wdspec] Enhance tests for "browsingContext.print" as PDF command. r=webdriver-reviewers,whimboo
Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.
Whiteboard: [webdriver:m6] → [webdriver:m6], [wptsync upstream error]
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38931 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m6], [wptsync upstream error] → [webdriver:m6], [wptsync upstream]
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: