Skip to content

Test main-thread and worker rendering paths - #22096

Open
calixteman wants to merge 1 commit into
mozilla:masterfrom
calixteman:renderer-worker-test-parity
Open

calixteman wants to merge 1 commit into
mozilla:masterfrom
calixteman:renderer-worker-test-parity

Conversation

@calixteman

Copy link
Copy Markdown
Contributor

Add main-thread variants for transfer functions, soft masks, knockout groups, and partial rendering via a disableWorkerRendering manifest option. Run the transfer-function unit test with worker rendering enabled and disabled.

Use canvas for the optimized partial-render pass so it can run in the renderer worker, matching the full-render pass.

Add main-thread variants for transfer functions, soft masks, knockout
groups, and partial rendering via a disableWorkerRendering manifest
option. Run the transfer-function unit test with worker rendering enabled
and disabled.

Use canvas for the optimized partial-render pass so it can run in the
renderer worker, matching the full-render pass.
@codecov-commenter

codecov-commenter commented Oct 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.46%. Comparing base (61b7f9a) to head (e06a8ba).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #22096      +/-   ##
==========================================
- Coverage   90.03%   89.46%   -0.57%     
==========================================
  Files         271      271              
  Lines       67770    67770              
==========================================
- Hits        61015    60629     -386     
- Misses       6755     7141     +386     
Flag Coverage Δ
unittest 59.99% <ø> (+0.12%) ⬆️
unittestcli 58.28% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Reference tests

Results for e06a8ba (run, references from mozilla/pdf.js.refs@05be816b55).

Platform Status Tests Total runtime Errors Different FBF No reference Report
Linux ⚠️ 1393 9m 47s 0 1 0 26 view differences
Windows ⚠️ 1393 18m 16s 0 1 0 26 view differences

The report is kept for 30 days and replaced by the next run.

@timvandermeij timvandermeij left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me, with passing browser tests. Thank you for extending the coverage of this logic!

sync-pdfs-for-pdf-js Bot pushed a commit to mozilla/pdf.js.refs that referenced this pull request Oct 8, 2026

This branch was successfully deployed

1 active deployment
sync_pdfs — e06a8ba5 Deployed Oct 8, 2026 by timvandermeij via request #328
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants