Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Zero the width/height of the temporary canvas used during JpegDecode (issue 10594)

Browse Source
This commit is contained in:
Jonas Jenwald 2019-02-28 12:05:11 +01:00
parent 21d70b19c2
commit 4687cc85ac
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/display/api.js
View File

@ -2068,15 +2068,14 @@ class WorkerTransport {
return new Promise(function(resolve, reject) {
const img = new Image();
img.onload = function() {
const width = img.width;
const height = img.height;
const { width, height, } = img;
const size = width * height;
const rgbaLength = size * 4;
const buf = new Uint8ClampedArray(size * components);
const tmpCanvas = document.createElement('canvas');
let tmpCanvas = document.createElement('canvas');
tmpCanvas.width = width;
tmpCanvas.height = height;
const tmpCtx = tmpCanvas.getContext('2d');
let tmpCtx = tmpCanvas.getContext('2d');
tmpCtx.drawImage(img, 0, 0);
const data = tmpCtx.getImageData(0, 0, width, height).data;
@ -2092,6 +2091,13 @@ class WorkerTransport {
}
}
resolve({ data: buf, width, height, });
// Zeroing the width and height cause Firefox to release graphics
// resources immediately, which can greatly reduce memory consumption.
tmpCanvas.width = 0;
tmpCanvas.height = 0;
tmpCanvas = null;
tmpCtx = null;
};
img.onerror = function() {
reject(new Error('JpegDecode failed to load image'));