Closed Bug 1444499 Opened 7 years ago Closed 6 years ago

Update pdf.js to version 2.0.447


(Firefox :: PDF Viewer, task)

Not set



Firefox 61
Tracking Status
firefox60 --- wontfix
firefox61 --- fixed


(Reporter: RyanVM, Assigned: bdahl)


(Blocks 1 open bug)



(2 files)

+++ This bug was initially created as a clone of Bug #1442785 +++

Changes since last update:
#9493 [api-major] Move the remaining options from, and completely remove, the global `PDFJS` object
#9508 Add paper size to document information/properties
#9509 Implement a single `getInheritableProperty` utility function
#9542 [CRX] Fix error that broke the Chrome extension
Pushed by
Update pdf.js to version 2.0.426. r=bdahl
Backout by
Backed out changeset 6405d9287056 for failing browser chrome at browser/extensions/pdfjs/test/browser_pdfjs_main.js on a CLOSED TREE
From IRC discussion, it sounds like bdahl is working on the test bustage.
Flags: needinfo?(bdahl)
IIUC, this is blocked on upstream issue #9545.
Comment on attachment 8960825 [details]
Bug 1444499 - Update pdf.js to version 2.0.447.

Code analysis found 1 defect in this patch:
 - 1 defect found by mozlint

You can run this analysis locally with:
 - `./mach lint path/to/file` (JS/Python)

If you see a problem in this automated review, please report it here:

::: browser/extensions/pdfjs/content/PdfJsDefaultPreferences.jsm:41
(Diff revision 1)
> + "renderer": "canvas",
> + "renderInteractiveForms": false,
> + "enablePrintAutoRotate": false,
> + "disablePageMode": false,
> + "disablePageLabels": false
> +});

Error: Newline required at end of file but not found. [eslint: eol-last]
Summary: Update pdf.js to version 2.0.426 → Update pdf.js to version 2.0.447
For perpetuity, additional changes from 426->447:
#9541 [CRX] Disable fetch in Chrome 60-
#9549 Attempt to handle corrupt PDF documents that inline Page dictionaries in a Kids array (issue 9540)
#9574 Update translations
#9577 [api-minor] Fix various issues related to pageSize, and display the size for the active page in the document properties dialog
#9571 [Firefox addon] Refactor bundling of the `default_preferences.json` file
#9587 Move the `cursorToolOnLoad` preference handling into `AppOptions` (PR 9493 follow-up)
#9565 Rename the globals to shorter names.
#9584 Update examples/ documentation for 2.0
Flags: needinfo?(bdahl)
Assignee: ryanvm → bdahl
Yury, any chance you'll be able to look at these reviews sometime soon? :)
Flags: needinfo?(ydelendik)
Comment on attachment 8960824 [details]
Bug 1444499 - Remove checks for global PDFJS.
Attachment #8960824 - Flags: review?(ydelendik) → review+
Comment on attachment 8960825 [details]
Bug 1444499 - Update pdf.js to version 2.0.447.

Looks good.
Attachment #8960825 - Flags: review?(ydelendik) → review+
Pushed by
Remove checks for global PDFJS. r=yury
Update pdf.js to version 2.0.447. r=yury
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Blocks: 1450369
Flags: needinfo?(ydelendik)
No longer blocks: 1450369
Type: defect → task
No longer depends on: 1442785
Regressions: 1658975
You need to log in before you can comment on or make changes to this bug.