Update PDF.js to new version 762d86a59ebd7cc2f7733f519de70610089344e6 from 2023-07-12 19:44:34
Categories
(Firefox :: PDF Viewer, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox117 | --- | fixed |
People
(Reporter: update-bot, Assigned: update-bot)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [3pl-filed][task_id: KgZ2EfotSeaeEYzsKKoU2w])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
This update covers 20 commits.. Here are the overall diff statistics, and then the commit information.
toolkit/components/pdfjs/content/build/pdf.js | 109 ++++++++++++--
toolkit/components/pdfjs/content/build/pdf.scripting.js | 4 +-
toolkit/components/pdfjs/content/build/pdf.worker.js | 54 ++++---
toolkit/components/pdfjs/content/web/viewer-geckoview.js | 23 +-
toolkit/components/pdfjs/content/web/viewer.html | 2 +-
toolkit/components/pdfjs/content/web/viewer.js | 31 ++-
toolkit/components/pdfjs/moz.yaml | 4 +-
7 files changed, 160 insertions(+), 67 deletions(-)
944c68ee8545eae5e6360985b5ef89ea30d10d37 by Calixte Denizet
https://github.com/mozilla/pdf.js/commit/944c68ee8545eae5e6360985b5ef89ea30d10d37
Authored: 2023-07-05 19:46:21 +0200
Committed: 2023-07-12 19:53:57 +0200
[Editor] Try to make the position of an edited FreeText the more accurated as possible
- Take into account the page translation,
- Take into account the correct translation for the editor border,
- Take into account the position of the first glyph in the annotation,
- Take into account the rotation of the editor.
Close #16633.
Files Added:
- test/pdfs/issue16633.pdf
- test/pdfs/rotated_freetexts.pdf
Files Modified:
- package-lock.json
- package.json
- src/core/annotation.js
- src/display/annotation_layer.js
- src/display/editor/editor.js
- src/display/editor/freetext.js
- test/integration/freetext_editor_spec.js
- test/integration/test_utils.js
- test/pdfs/.gitignore
7e04ca395fb12ee261dd0611d17c89ca1774ee6c by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/7e04ca395fb12ee261dd0611d17c89ca1774ee6c
Authored: 2023-07-12 17:33:20 +0200
Committed: 2023-07-12 17:33:20 +0200
Access the isOffscreenCanvasSupported
option once in PDFViewerApplication._initializeViewerComponents
There's no good reason for getting this option multiple times in the same method. Also, we can slightly re-factor how the editorStampButton
is made visible.
Files Modified:
- web/app.js
- web/viewer.html
5cfe792349016ffd02642738e9f78a6583ae37d1 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/5cfe792349016ffd02642738e9f78a6583ae37d1
Authored: 2023-07-12 13:24:02 +0200
Committed: 2023-07-12 13:24:02 +0200
Tweak the writeStream
implementation slightly
- Do the /Filter and /DecodeParms lookup in parallel, since that ought to be a tiny bit more efficient.
- Avoid code-duplication when
CompressionStream
isn't supported, since we already have a fallback code-path at the end of the function.
Files Modified:
- src/core/writer.js
7daa6b5fd886136c7aaf8bf58cdfebe4dabeeda9 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/7daa6b5fd886136c7aaf8bf58cdfebe4dabeeda9
Authored: 2023-07-11 22:26:23 +0200
Committed: 2023-07-11 22:34:20 +0200
Re-enable editing, if necessary, when exiting PresentationMode (PR 16659 follow-up)
This regressed in PR 16659, when the signature of the PDFViewer.annotationEditorMode
-setter was changed, and it currently leads to an Error being thrown when exiting PresentationMode.
Files Modified:
- web/pdf_presentation_mode.js
51742323260ae4c9040f392d11a70dbb40522775 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/51742323260ae4c9040f392d11a70dbb40522775
Authored: 2023-07-08 16:07:54 +0200
Committed: 2023-07-11 11:38:59 +0200
[ESM] Convert the external/builder/
-folder to use standard modules
Files Modified:
- gulpfile.mjs
Files Changed:
- R097 external/builder/builder.js
- R097 external/builder/preprocessor2.js
- R088 external/builder/test-fixtures.js
- R086 external/builder/test-fixtures_esprima.js
- R085 external/webpack/pdfjsdev-loader.js
414ea4a365f50650e3850dbb8a7b96d81322c608 by Calixte Denizet
https://github.com/mozilla/pdf.js/commit/414ea4a365f50650e3850dbb8a7b96d81322c608
Authored: 2023-07-10 21:17:48 +0200
Committed: 2023-07-10 21:18:05 +0200
[Editor] Cheat the stamp editor icon to make a linter in m-c happy
At some point we won't use this icon anymore in the build-in pdf viewer
but we'll the one already in the m-c tree.
Files Modified:
- web/images/toolbarButton-editorStamp.svg
0bc9012b8f3b0dd5ce3c2ba5393ec032b899de28 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/0bc9012b8f3b0dd5ce3c2ba5393ec032b899de28
Authored: 2023-07-08 15:33:03 +0200
Committed: 2023-07-09 15:58:19 +0200
[ESM] Convert the gulpfile
to use standard modules
This is a necessary intermediate step to allow converting files in the external/builder/
folder.
Files Changed:
- R098 gulpfile.js
6c601d3922f761d73c503833d16312e7ab00355e by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/6c601d3922f761d73c503833d16312e7ab00355e
Authored: 2023-07-09 11:37:00 +0200
Committed: 2023-07-09 11:54:11 +0200
[ESM] Convert the "default preferences"-handling to use import()
syntax
Unfortunately I wasn't able to come up with a simple way to just replace the synchronous require
-call, since we need to ensure that the default preferences are available when bundling starts.
Hence this patch adds a new intermediate parsing-step in all the relevant gulp-tasks, but this shouldn't affect build-times noticeable since the amount of extra parsing is very small.
Please note: It's very possible that there's a better way to handle this, however I figured that unblocking further ESM-work is more important than a "perfect" solution.
Files Modified:
- gulpfile.js
f012fc5e70d8723846d3e05d28bbd08311c4a3fa by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/f012fc5e70d8723846d3e05d28bbd08311c4a3fa
Authored: 2023-07-08 14:35:15 +0200
Committed: 2023-07-08 18:52:58 +0200
[ESM] Convert the "cmaps"-task to use import()
syntax
Files Modified:
- gulpfile.js
Files Changed:
- R098 external/cmapscompress/compress.js
- R099 external/cmapscompress/optimize.js
- R097 external/cmapscompress/parse.js
ac3661972bac968c053e80e470cd7aafbd835e46 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/ac3661972bac968c053e80e470cd7aafbd835e46
Authored: 2023-07-08 13:50:19 +0200
Committed: 2023-07-08 13:50:19 +0200
[ESM] Convert the "wintersmith"-task to use import()
syntax
Files Modified:
- gulpfile.js
a209ce811d3e9f042797fce8c06bb582d836885f by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/a209ce811d3e9f042797fce8c06bb582d836885f
Authored: 2023-07-08 12:44:37 +0200
Committed: 2023-07-08 13:13:04 +0200
[ESM] Convert most of test
-folder to use standard modules
Files Modified:
- gulpfile.js
Files Changed:
- R094 test/add_test.js
- R094 test/downloadutils.js
- R090 test/font/ttxdriver.js
- R095 test/integration-boot.js
- R096 test/test.js
- R083 test/testutils.js
- R098 test/webserver.js
adfee2a7238226fbedfc657eacd893471ac3cb6a by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/adfee2a7238226fbedfc657eacd893471ac3cb6a
Authored: 2023-07-08 10:01:16 +0200
Committed: 2023-07-08 10:01:16 +0200
[ESM] Convert parseMinified
to use import()
syntax
Files Modified:
- gulpfile.js
cec2a9a03fce65b464366137b3c58e05ef28c5ee by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/cec2a9a03fce65b464366137b3c58e05ef28c5ee
Authored: 2023-07-08 09:36:47 +0200
Committed: 2023-07-08 09:36:47 +0200
Update l10n files
Files Modified:
- l10n/ach/viewer.properties
- l10n/af/viewer.properties
- l10n/an/viewer.properties
- l10n/ar/viewer.properties
- l10n/ast/viewer.properties
- l10n/az/viewer.properties
- l10n/be/viewer.properties
- l10n/bg/viewer.properties
- l10n/bn/viewer.properties
- l10n/bo/viewer.properties
- l10n/br/viewer.properties
- l10n/brx/viewer.properties
- l10n/bs/viewer.properties
- l10n/ca/viewer.properties
- l10n/cak/viewer.properties
- l10n/ckb/viewer.properties
- l10n/cs/viewer.properties
- l10n/cy/viewer.properties
- l10n/da/viewer.properties
- l10n/de/viewer.properties
- l10n/dsb/viewer.properties
- l10n/el/viewer.properties
- l10n/en-CA/viewer.properties
- l10n/en-GB/viewer.properties
- l10n/eo/viewer.properties
- l10n/es-AR/viewer.properties
- l10n/es-CL/viewer.properties
- l10n/es-ES/viewer.properties
- l10n/es-MX/viewer.properties
- l10n/et/viewer.properties
- l10n/eu/viewer.properties
- l10n/fa/viewer.properties
- l10n/ff/viewer.properties
- l10n/fi/viewer.properties
- l10n/fr/viewer.properties
- l10n/fur/viewer.properties
- l10n/fy-NL/viewer.properties
- l10n/ga-IE/viewer.properties
- l10n/gd/viewer.properties
- l10n/gl/viewer.properties
- l10n/gn/viewer.properties
- l10n/gu-IN/viewer.properties
- l10n/he/viewer.properties
- l10n/hi-IN/viewer.properties
- l10n/hr/viewer.properties
- l10n/hsb/viewer.properties
- l10n/hu/viewer.properties
- l10n/hy-AM/viewer.properties
- l10n/hye/viewer.properties
- l10n/ia/viewer.properties
- l10n/id/viewer.properties
- l10n/is/viewer.properties
- l10n/it/viewer.properties
- l10n/ka/viewer.properties
- l10n/kab/viewer.properties
- l10n/kk/viewer.properties
- l10n/km/viewer.properties
- l10n/kn/viewer.properties
- l10n/ko/viewer.properties
- l10n/lij/viewer.properties
- l10n/lo/viewer.properties
- l10n/lt/viewer.properties
- l10n/ltg/viewer.properties
- l10n/lv/viewer.properties
- l10n/meh/viewer.properties
- l10n/mk/viewer.properties
- l10n/mr/viewer.properties
- l10n/ms/viewer.properties
- l10n/my/viewer.properties
- l10n/nb-NO/viewer.properties
- l10n/ne-NP/viewer.properties
- l10n/nl/viewer.properties
- l10n/nn-NO/viewer.properties
- l10n/oc/viewer.properties
- l10n/pa-IN/viewer.properties
- l10n/pl/viewer.properties
- l10n/pt-BR/viewer.properties
- l10n/pt-PT/viewer.properties
- l10n/rm/viewer.properties
- l10n/ro/viewer.properties
- l10n/ru/viewer.properties
- l10n/sat/viewer.properties
- l10n/sc/viewer.properties
- l10n/sco/viewer.properties
- l10n/si/viewer.properties
- l10n/sk/viewer.properties
- l10n/skr/viewer.properties
- l10n/sl/viewer.properties
- l10n/son/viewer.properties
- l10n/sq/viewer.properties
- l10n/sr/viewer.properties
- l10n/sv-SE/viewer.properties
- l10n/szl/viewer.properties
- l10n/ta/viewer.properties
- l10n/te/viewer.properties
- l10n/tg/viewer.properties
- l10n/th/viewer.properties
- l10n/tl/viewer.properties
- l10n/tr/viewer.properties
- l10n/trs/viewer.properties
- l10n/uk/viewer.properties
- l10n/ur/viewer.properties
- l10n/uz/viewer.properties
- l10n/vi/viewer.properties
- l10n/wo/viewer.properties
- l10n/xh/viewer.properties
- l10n/zh-CN/viewer.properties
- l10n/zh-TW/viewer.properties
a650fcd6341ad40a9d4108592ce254e363204ee8 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/a650fcd6341ad40a9d4108592ce254e363204ee8
Authored: 2023-07-08 09:29:51 +0200
Committed: 2023-07-08 09:36:32 +0200
[ESM] Convert the external/importL10n
-folder to use standard modules
Files Modified:
- extensions/firefox/.eslintrc
- gulpfile.js
Files Changed:
- R095 external/importL10n/locales.js
559ea194f28ab93c192dc6eb025e307017449de2 by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/559ea194f28ab93c192dc6eb025e307017449de2
Authored: 2023-07-07 19:13:48 +0200
Committed: 2023-07-07 19:13:48 +0200
Don't try to hide an already invisible PopupAnnotation (issue 16660)
Files Modified:
- src/display/annotation_layer.js
5c5f9af803187d616703c19987eca5d7d39d9420 by Calixte Denizet
https://github.com/mozilla/pdf.js/commit/5c5f9af803187d616703c19987eca5d7d39d9420
Authored: 2023-07-07 15:51:48 +0200
Committed: 2023-07-07 17:44:45 +0200
[Editor] Edit an existing FreeText annotation in double-clicking on it (bug 1787298)
Files Modified:
- src/display/annotation_layer.js
- src/display/editor/editor.js
- src/display/editor/freetext.js
- src/display/editor/tools.js
- test/integration/freetext_editor_spec.js
- web/app.js
- web/pdf_presentation_mode.js
- web/pdf_viewer.js
506bca5e6da00fc4b29d7f1fa3686fe1b235c0fa by Jonas Jenwald
https://github.com/mozilla/pdf.js/commit/506bca5e6da00fc4b29d7f1fa3686fe1b235c0fa
Authored: 2023-07-07 12:36:21 +0200
Committed: 2023-07-07 12:36:21 +0200
Add unit-tests to check that more PDF.js APIs expose the expected functionality
Similar to e.g. PR 16587, let's ensure that the pdf.worker.js
and pdf.image_decoders.js
files expose the expected functionality.
Files Added:
- test/unit/pdf.image_decoders_spec.js
- test/unit/pdf.worker_spec.js
Files Modified:
- gulpfile.js
- src/pdf.image_decoders.js
- test/unit/clitests.json
- test/unit/jasmine-boot.js
70db938fe7223d5cd5f4b3c1639ed3dd8b79c077 by Rob Wu
https://github.com/mozilla/pdf.js/commit/70db938fe7223d5cd5f4b3c1639ed3dd8b79c077
Authored: 2023-07-02 17:10:17 +0200
Committed: 2023-07-02 17:23:18 +0200
[CRX] Drop code supporting ancient Chrome versions
Files Modified:
- extensions/chromium/contentscript.js
- extensions/chromium/contentstyle.css
- extensions/chromium/pdfHandler.js
- extensions/chromium/preserve-referer.js
- extensions/chromium/telemetry.js
f2753d62204b1a41f2ece87ab854c0b919013686 by Rob Wu
https://github.com/mozilla/pdf.js/commit/f2753d62204b1a41f2ece87ab854c0b919013686
Authored: 2023-07-02 15:31:35 +0200
Committed: 2023-07-02 15:31:35 +0200
[CRX] Avoid encoding the fragment in file key
Semantically, it is more correct to encode the fragment in the URL
instead of the URL-encoded file
query parameter. This shouldn't matter
in practice, because rewriteUrlClosure
in chromecom.js
decodes the
file
parameter and restores the fragment. However, as #16625 shows,
there was a case where this did not work as expected.
Files Modified:
- extensions/chromium/pdfHandler.js
1d07ef401ec16580360252ebe0bf218e979f7513 by Rob Wu
https://github.com/mozilla/pdf.js/commit/1d07ef401ec16580360252ebe0bf218e979f7513
Authored: 2023-07-02 15:28:33 +0200
Committed: 2023-07-02 15:28:33 +0200
[CRX] Re-initialize initialBookmark after URL rewrite
PDFViewerApplication
reads from location.hash
to initialize
initialBookmark
. But when extensions/chromium/pdfHandler.js prepares
the redirect URL, the reference fragment is encoded instead of bare.
rewriteUrlClosure
in chromecom.js
is responsible for decoding the
URL, but that currently runs too late.
To fix this, update initialBookmark
after rewriting the URL.
This was not a problem in the past because rewriteUrlClosure
in
chromecom.js
executed before the initialization of initialBookmark
.
Files Modified:
- web/chromecom.js
Assignee | ||
Comment 1•1 year ago
|
||
KgZ2EfotSeaeEYzsKKoU2w |
I've submitted a try run for this commit: https://treeherder.mozilla.org/jobs?repo=try&revision=0a226cbde77cf992b6ad345281ece326281ad6a7
Assignee | ||
Comment 2•1 year ago
|
||
Assignee | ||
Comment 3•1 year ago
|
||
Leaukt1eTbW_IeILvmFQwg |
Updatebot encountered an error while trying to submit to try. It appears that the Decision task did not succeed.
Updatebot will be unable to do anything more for this library version.
Comment 4•1 year ago
|
||
Updated•1 year ago
|
Comment 6•1 year ago
|
||
bugherder |
Description
•