Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions gulpfile.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -589,6 +589,18 @@ function createWorkerBundle(defines) {
.pipe(webpack2Stream(workerFileConfig));
}

function createRendererWorkerBundle(defines) {
const rendererWorkerFileConfig = createWebpackConfig(defines, {
filename: defines.MINIFIED ? "pdf.renderer.min.mjs" : "pdf.renderer.mjs",
library: {
type: "module",
},
});
return gulp
.src("./src/pdf.renderer.js", { encoding: false })
.pipe(webpack2Stream(rendererWorkerFileConfig));
}

function createWebBundle(defines, options) {
const viewerFileConfig = createWebpackConfig(
defines,
Expand Down Expand Up @@ -1103,6 +1115,7 @@ function buildGeneric(defines, dir) {
return ordered([
createMainBundle(defines).pipe(gulp.dest(dir + "build")),
createWorkerBundle(defines).pipe(gulp.dest(dir + "build")),
createRendererWorkerBundle(defines).pipe(gulp.dest(dir + "build")),
createSandboxBundle(defines).pipe(gulp.dest(dir + "build")),
createWebBundle(defines, {
defaultPreferencesDir: defines.SKIP_BABEL
Expand Down Expand Up @@ -1293,6 +1306,7 @@ function buildMinified(defines, dir) {
return ordered([
createMainBundle(defines).pipe(gulp.dest(dir + "build")),
createWorkerBundle(defines).pipe(gulp.dest(dir + "build")),
createRendererWorkerBundle(defines).pipe(gulp.dest(dir + "build")),
createSandboxBundle(defines).pipe(gulp.dest(dir + "build")),
createImageDecodersBundle({ ...defines, IMAGE_DECODERS: true }).pipe(
gulp.dest(dir + "image_decoders")
Expand Down Expand Up @@ -1438,6 +1452,9 @@ gulp.task(
createWorkerBundle(defines).pipe(
gulp.dest(MOZCENTRAL_CONTENT_DIR + "build")
),
createRendererWorkerBundle(defines).pipe(
gulp.dest(MOZCENTRAL_CONTENT_DIR + "build")
),
createWebBundle(defines, { defaultPreferencesDir: "mozcentral/" }).pipe(
gulp.dest(MOZCENTRAL_CONTENT_DIR + "web")
),
Expand Down Expand Up @@ -1535,6 +1552,9 @@ gulp.task(
createWorkerBundle(defines).pipe(
gulp.dest(CHROME_BUILD_CONTENT_DIR + "build")
),
createRendererWorkerBundle(defines).pipe(
gulp.dest(CHROME_BUILD_CONTENT_DIR + "build")
),
createSandboxBundle(defines).pipe(
gulp.dest(CHROME_BUILD_CONTENT_DIR + "build")
),
Expand Down
16 changes: 11 additions & 5 deletions src/core/document.js
Original file line number Diff line number Diff line change
Expand Up @@ -127,10 +127,11 @@ class Page {
};
}

#createPartialEvaluator(handler) {
#createPartialEvaluator(handler, rendererHandler) {
return new PartialEvaluator({
xref: this.xref,
handler,
rendererHandler,
pageIndex: this.pageIndex,
idFactory: this._localIdFactory,
fontCache: this.fontCache,
Expand Down Expand Up @@ -436,6 +437,7 @@ class Page {

async getOperatorList({
handler,
rendererHandler,
sink,
task,
intent,
Expand All @@ -446,7 +448,10 @@ class Page {
const contentStreamPromise = this.getContentStream();
const resourcesPromise = this.loadResources(RESOURCES_KEYS_OPERATOR_LIST);

const partialEvaluator = this.#createPartialEvaluator(handler);
const partialEvaluator = this.#createPartialEvaluator(
handler,
rendererHandler
);

const newAnnotsByPage = !this.xfaFactory
? getNewAnnotationsMap(annotationStorage)
Expand Down Expand Up @@ -1269,7 +1274,7 @@ class PDFDocument {
this.xfaFactory.setImages(xfaImages);
}

async #loadXfaFonts(handler, task) {
async #loadXfaFonts(handler, task, rendererHandler) {
const acroForm = await this.pdfManager.ensureCatalog("acroForm");
if (!acroForm) {
return;
Expand All @@ -1295,6 +1300,7 @@ class PDFDocument {
const partialEvaluator = new PartialEvaluator({
xref: this.xref,
handler,
rendererHandler,
pageIndex: -1,
idFactory: this._globalIdFactory,
fontCache,
Expand Down Expand Up @@ -1407,9 +1413,9 @@ class PDFDocument {
this.xfaFactory.appendFonts(pdfFonts, reallyMissingFonts);
}

loadXfaResources(handler, task) {
loadXfaResources(handler, task, rendererHandler) {
return Promise.all([
this.#loadXfaFonts(handler, task).catch(() => {
this.#loadXfaFonts(handler, task, rendererHandler).catch(() => {
// Ignore errors, to allow the document to load.
}),
this.#loadXfaImages(),
Expand Down
31 changes: 23 additions & 8 deletions src/core/evaluator.js
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,7 @@ class PartialEvaluator {
constructor({
xref,
handler,
rendererHandler,
pageIndex,
idFactory,
fontCache,
Expand All @@ -233,6 +234,7 @@ class PartialEvaluator {
}) {
this.xref = xref;
this.handler = handler;
this.rendererHandler = rendererHandler;
this.pageIndex = pageIndex;
this.idFactory = idFactory;
this.fontCache = fontCache;
Expand Down Expand Up @@ -552,13 +554,19 @@ class PartialEvaluator {
const transfers = imgData ? [imgData.bitmap || imgData.data.buffer] : null;

if (this.parsingType3Font || cacheGlobally) {
return this.handler.send(
this.handler.send("commonobj", [objId, "Image", imgData], transfers);
return this.rendererHandler.send(
"commonobj",
[objId, "Image", imgData],
transfers
);
}
return this.handler.send(
this.handler.send(
"obj",
[objId, this.pageIndex, "Image", imgData],
transfers
);
return this.rendererHandler.send(
"obj",
[objId, this.pageIndex, "Image", imgData],
transfers
Expand Down Expand Up @@ -786,11 +794,10 @@ class PartialEvaluator {
// globally, check if the image is still cached locally on the main-thread
// to avoid having to re-parse the image (since that can be slow).
if (w * h > 250000 || hasMask) {
const localLength = await this.handler.sendWithPromise("commonobj", [
objId,
"CopyLocalImage",
{ imageRef },
]);
const localLength = await this.rendererHandler.sendWithPromise(
"commonobj",
[objId, "CopyLocalImage", { imageRef }]
);

if (localLength) {
this.globalImageCache.setData(imageRef, globalCacheData);
Expand Down Expand Up @@ -1020,6 +1027,7 @@ class PartialEvaluator {

state.font = translated.font;
translated.send(this.handler);
translated.send(this.rendererHandler);
return translated.loadedName;
}

Expand All @@ -1039,7 +1047,7 @@ class PartialEvaluator {
PartialEvaluator.buildFontPaths(
font,
glyphs,
this.handler,
this.rendererHandler,
this.options
);
}
Expand Down Expand Up @@ -1517,8 +1525,15 @@ class PartialEvaluator {

if (this.parsingType3Font) {
this.handler.send("commonobj", [id, "Pattern", patternIR]);
this.rendererHandler.send("commonobj", [id, "Pattern", patternIR]);
} else {
this.handler.send("obj", [id, this.pageIndex, "Pattern", patternIR]);
this.rendererHandler.send("obj", [
id,
this.pageIndex,
"Pattern",
patternIR,
]);
}
return id;
}
Expand Down
24 changes: 19 additions & 5 deletions src/core/worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,8 @@ class WorkerMessageHandler {

static setup(handler, port) {
let testMessageProcessed = false;
let rendererHandler = null;

handler.on("test", data => {
if (testMessageProcessed) {
return; // we already processed 'test' message once
Expand All @@ -94,12 +96,19 @@ class WorkerMessageHandler {

handler.on("configure", data => {
setVerbosityLevel(data.verbosity);
rendererHandler = new MessageHandler(
"worker-channel",
"renderer-channel",
data.channelPort
);
});

handler.on("GetDocRequest", data => this.createDocumentHandler(data, port));
handler.on("GetDocRequest", data =>
this.createDocumentHandler(data, port, rendererHandler)
);
}

static createDocumentHandler(docParams, port) {
static createDocumentHandler(docParams, port, rendererHandler) {
// This context is actually holds references on pdfManager and handler,
// until the latter is destroyed.
let pdfManager;
Expand Down Expand Up @@ -173,7 +182,11 @@ class WorkerMessageHandler {
const task = new WorkerTask("loadXfaResources");
startWorkerTask(task);

await pdfManager.ensureDoc("loadXfaResources", [handler, task]);
await pdfManager.ensureDoc("loadXfaResources", [
handler,
task,
rendererHandler,
]);
finishWorkerTask(task);
}

Expand Down Expand Up @@ -725,6 +738,7 @@ class WorkerMessageHandler {
page
.getOperatorList({
handler,
rendererHandler,
sink,
task,
intent: data.intent,
Expand Down Expand Up @@ -808,8 +822,8 @@ class WorkerMessageHandler {
.then(page => pdfManager.ensure(page, "getStructTree"));
});

handler.on("FontFallback", function (data) {
return pdfManager.fontFallback(data.id, handler);
rendererHandler.on("FontFallback", function (data) {
return pdfManager.fontFallback(data.id, rendererHandler);
});

handler.on("Cleanup", function (data) {
Expand Down
Loading
Loading