Enhance tests for "browsingContext.print" as PDF command
Categories
(Remote Protocol :: WebDriver BiDi, task, P1)
Tracking
(firefox112 fixed)
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?
Reporter | ||
Updated•2 years ago
|
Reporter | ||
Updated•2 years ago
|
Comment 1•2 years ago
|
||
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.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 2•1 year ago
|
||
Depends on D170356
Assignee | ||
Comment 3•1 year ago
|
||
Depends on D171082
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.
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38931 for changes under testing/web-platform/tests
Comment 7•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/14201ceea9ee
https://hg.mozilla.org/mozilla-central/rev/4dfc3d3988f1
Description
•