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

Better errors capturing at the core and stop rendering on error.

Browse Source
This commit is contained in:
Yury Delendik 2016-03-11 07:59:09 -06:00
parent 4784863ef7
commit 8ba413e761
2 changed files with 58 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

90
src/core/evaluator.js
View File

@ -755,7 +755,16 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
var preprocessor = new EvaluatorPreprocessor(stream, xref, stateManager); var preprocessor = new EvaluatorPreprocessor(stream, xref, stateManager);
var timeSlotManager = new TimeSlotManager(); var timeSlotManager = new TimeSlotManager();
return new Promise(function next(resolve, reject) { return new Promise(function promiseBody(resolve, reject) {
var next = function (promise) {
promise.then(function () {
try {
promiseBody(resolve, reject);
} catch (ex) {
reject(ex);
}
}, reject);
};
task.ensureNotTerminated(); task.ensureNotTerminated();
timeSlotManager.reset(); timeSlotManager.reset();
var stop, operation = {}, i, ii, cs; var stop, operation = {}, i, ii, cs;
@ -798,13 +807,13 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
if (type.name === 'Form') { if (type.name === 'Form') {
stateManager.save(); stateManager.save();
return self.buildFormXObject(resources, xobj, null, next(self.buildFormXObject(resources, xobj, null,
operatorList, task, operatorList, task,
stateManager.state.clone()). stateManager.state.clone()).
then(function () { then(function () {
stateManager.restore(); stateManager.restore();
next(resolve, reject); }));
}, reject); return;
} else if (type.name === 'Image') { } else if (type.name === 'Image') {
self.buildPaintImageXObject(resources, xobj, false, self.buildPaintImageXObject(resources, xobj, false,
operatorList, name, imageCache); operatorList, name, imageCache);
@ -823,13 +832,13 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case OPS.setFont: case OPS.setFont:
var fontSize = args[1]; var fontSize = args[1];
// eagerly collect all fonts // eagerly collect all fonts
return self.handleSetFont(resources, args, null, operatorList, next(self.handleSetFont(resources, args, null, operatorList,
task, stateManager.state). task, stateManager.state).
then(function (loadedName) { then(function (loadedName) {
operatorList.addDependency(loadedName); operatorList.addDependency(loadedName);
operatorList.addOp(OPS.setFont, [loadedName, fontSize]); operatorList.addOp(OPS.setFont, [loadedName, fontSize]);
next(resolve, reject); }));
}, reject); return;
case OPS.endInlineImage: case OPS.endInlineImage:
var cacheKey = args[0].cacheKey; var cacheKey = args[0].cacheKey;
if (cacheKey) { if (cacheKey) {
@ -929,10 +938,9 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case OPS.setFillColorN: case OPS.setFillColorN:
cs = stateManager.state.fillColorSpace; cs = stateManager.state.fillColorSpace;
if (cs.name === 'Pattern') { if (cs.name === 'Pattern') {
return self.handleColorN(operatorList, OPS.setFillColorN, next(self.handleColorN(operatorList, OPS.setFillColorN, args,
args, cs, patterns, resources, task, xref).then(function() { cs, patterns, resources, task, xref));
next(resolve, reject); return;
}, reject);
} }
args = cs.getRgb(args, 0); args = cs.getRgb(args, 0);
fn = OPS.setFillRGBColor; fn = OPS.setFillRGBColor;
@ -940,10 +948,9 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case OPS.setStrokeColorN: case OPS.setStrokeColorN:
cs = stateManager.state.strokeColorSpace; cs = stateManager.state.strokeColorSpace;
if (cs.name === 'Pattern') { if (cs.name === 'Pattern') {
return self.handleColorN(operatorList, OPS.setStrokeColorN, next(self.handleColorN(operatorList, OPS.setStrokeColorN, args,
args, cs, patterns, resources, task, xref).then(function() { cs, patterns, resources, task, xref));
next(resolve, reject); return;
}, reject);
} }
args = cs.getRgb(args, 0); args = cs.getRgb(args, 0);
fn = OPS.setStrokeRGBColor; fn = OPS.setStrokeRGBColor;
@ -975,10 +982,9 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
} }
var gState = extGState.get(dictName.name); var gState = extGState.get(dictName.name);
return self.setGState(resources, gState, operatorList, task, next(self.setGState(resources, gState, operatorList, task, xref,
xref, stateManager).then(function() { stateManager));
next(resolve, reject); return;
}, reject);
case OPS.moveTo: case OPS.moveTo:
case OPS.lineTo: case OPS.lineTo:
case OPS.curveTo: case OPS.curveTo:
@ -1023,9 +1029,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
operatorList.addOp(fn, args); operatorList.addOp(fn, args);
} }
if (stop) { if (stop) {
deferred.then(function () { next(deferred);
next(resolve, reject);
}, reject);
return; return;
} }
// Some PDFs don't close all restores inside object/form. // Some PDFs don't close all restores inside object/form.
@ -1301,7 +1305,16 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
var timeSlotManager = new TimeSlotManager(); var timeSlotManager = new TimeSlotManager();
return new Promise(function next(resolve, reject) { return new Promise(function promiseBody(resolve, reject) {
var next = function (promise) {
promise.then(function () {
try {
promiseBody(resolve, reject);
} catch (ex) {
reject(ex);
}
}, reject);
};
task.ensureNotTerminated(); task.ensureNotTerminated();
timeSlotManager.reset(); timeSlotManager.reset();
var stop, operation = {}, args = []; var stop, operation = {}, args = [];
@ -1323,9 +1336,8 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case OPS.setFont: case OPS.setFont:
flushTextContentItem(); flushTextContentItem();
textState.fontSize = args[1]; textState.fontSize = args[1];
return handleSetFont(args[0].name).then(function() { next(handleSetFont(args[0].name));
next(resolve, reject); return;
}, reject);
case OPS.setTextRise: case OPS.setTextRise:
flushTextContentItem(); flushTextContentItem();
textState.textRise = args[0]; textState.textRise = args[0];
@ -1496,18 +1508,17 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
stateManager.transform(matrix); stateManager.transform(matrix);
} }
return self.getTextContent(xobj, task, next(self.getTextContent(xobj, task,
xobj.dict.get('Resources') || resources, stateManager, xobj.dict.get('Resources') || resources, stateManager,
normalizeWhitespace).then(function (formTextContent) { normalizeWhitespace).then(function (formTextContent) {
Util.appendToArray(textContent.items, formTextContent.items); Util.appendToArray(textContent.items, formTextContent.items);
Util.extendObj(textContent.styles, formTextContent.styles); Util.extendObj(textContent.styles, formTextContent.styles);
stateManager.restore(); stateManager.restore();
xobjsCache.key = name; xobjsCache.key = name;
xobjsCache.texts = formTextContent; xobjsCache.texts = formTextContent;
}));
next(resolve, reject); return;
}, reject);
case OPS.setGState: case OPS.setGState:
flushTextContentItem(); flushTextContentItem();
var dictName = args[0]; var dictName = args[0];
@ -1527,17 +1538,14 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
} }
if (gsStateFont) { if (gsStateFont) {
textState.fontSize = gsStateFont[1]; textState.fontSize = gsStateFont[1];
return handleSetFont(gsStateFont[0]).then(function() { next(handleSetFont(gsStateFont[0]));
next(resolve, reject); return;
}, reject);
} }
break; break;
} // switch } // switch
} // while } // while
if (stop) { if (stop) {
deferred.then(function () { next(deferred);
next(resolve, reject);
}, reject);
return; return;
} }
flushTextContentItem(); flushTextContentItem();

9
src/display/api.js
View File

@ -1716,11 +1716,20 @@ var WorkerTransport = (function WorkerTransportClosure() {
var page = this.pageCache[data.pageNum - 1]; var page = this.pageCache[data.pageNum - 1];
var intentState = page.intentStates[data.intent]; var intentState = page.intentStates[data.intent];
if (intentState.displayReadyCapability) { if (intentState.displayReadyCapability) {
intentState.displayReadyCapability.reject(data.error); intentState.displayReadyCapability.reject(data.error);
} else { } else {
error(data.error); error(data.error);
} }
if (intentState.operatorList) {
// Mark operator list as complete.
intentState.operatorList.lastChunk = true;
for (var i = 0; i < intentState.renderTasks.length; i++) {
intentState.renderTasks[i].operatorListChanged();
}
}
}, this); }, this);
messageHandler.on('UnsupportedFeature', messageHandler.on('UnsupportedFeature',