Skip to content

Comments

[pull] master from mozilla:master#226

Merged
pull[bot] merged 10 commits intocode:masterfrom
mozilla:master
Feb 21, 2026
Merged

[pull] master from mozilla:master#226
pull[bot] merged 10 commits intocode:masterfrom
mozilla:master

Conversation

@pull
Copy link

@pull pull bot commented Feb 21, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Snuffleupagus and others added 10 commits February 14, 2026 15:49
…us method

This is not only shorter, but (in my opinion) it also simplifies the code.

*Note:* In order to keep the *five* different `BasePDFStreamRangeReader` implementations consistent, we purposely don't re-factor the `PDFWorkerStreamRangeReader` class to support `for await...of` iteration.
…FromUrl` function

Currently we essentially "duplicate" the same code for parsing the `values` and `keys` of the `searchParams`, which seems a little unnecessary.

To be able to parse the `searchParams` from the end, we currently create an Array (from the Iterator) and then reverse it before finally looping through it. Here the latter two steps can be replaced with the `Array.prototype.findLast()` method instead.

*Please note:* I completely understand if this patch is rejected, on account of being less readable than the current code.
…Request

Convert `ChunkedStreamManager.prototype.sendRequest` to an asynchronous method
Update the page count in the UI when some changes are done (bug 2018125)
…archParams-findLast

Reduce duplication when parsing `searchParams` in the `getPdfFilenameFromUrl` function
Add some unit tests for functions in image_utils.js
@pull pull bot locked and limited conversation to collaborators Feb 21, 2026
@pull pull bot added the ⤵️ pull label Feb 21, 2026
@pull pull bot merged commit f548a15 into code:master Feb 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants