1444499 - Update pdf.js to version 2.0.447

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, task)

Description

[Ryan VanderMeulen [:RyanVM]]

+++ 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

Comment 1

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6405d9287056
Update pdf.js to version 2.0.426. r=bdahl

Comment 2

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/55dedf58f123
Backed out changeset 6405d9287056 for failing browser chrome at browser/extensions/pdfjs/test/browser_pdfjs_main.js on a CLOSED TREE

Comment 3

[Ryan VanderMeulen [:RyanVM]]

From IRC discussion, it sounds like bdahl is working on the test bustage.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

IIUC, this is blocked on upstream issue #9545.

Comment 5

[Brendan Dahl [:bdahl]]

Attachment: Bug 1444499 - Remove checks for global PDFJS.

Review commit: https://reviewboard.mozilla.org/r/229564/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/229564/

Comment 6

[Brendan Dahl [:bdahl]]

Attachment: Bug 1444499 - Update pdf.js to version 2.0.447.

Review commit: https://reviewboard.mozilla.org/r/229566/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/229566/

Comment 7

[Code Review Bot [:reviewbot]]

Comment on attachment 8960825 [details]
Bug 1444499 - Update pdf.js to version 2.0.447.

https://reviewboard.mozilla.org/r/229566/#review235294


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: http://bit.ly/2y9N9Vx


::: 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]

Comment 8

[Brendan Dahl [:bdahl]]

Comment on attachment 8960825 [details]
Bug 1444499 - Update pdf.js to version 2.0.447.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/229566/diff/1-2/

Comment 9

[Ryan VanderMeulen [:RyanVM]]

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

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Yury, any chance you'll be able to look at these reviews sometime soon? :)

Comment 11

[Yury Delendik (:yury)]

Comment on attachment 8960824 [details]
Bug 1444499 - Remove checks for global PDFJS.

https://reviewboard.mozilla.org/r/229564/#review237044

Comment 12

[Yury Delendik (:yury)]

Comment on attachment 8960825 [details]
Bug 1444499 - Update pdf.js to version 2.0.447.

https://reviewboard.mozilla.org/r/229566/#review237048

Looks good.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/974e81a794a1
Remove checks for global PDFJS. r=yury
https://hg.mozilla.org/integration/autoland/rev/7d8a99f5f759
Update pdf.js to version 2.0.447. r=yury

Comment 14

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/974e81a794a1
https://hg.mozilla.org/mozilla-central/rev/7d8a99f5f759