From 6d53fc4db7022cc3bade725a360da2def09adc78 Mon Sep 17 00:00:00 2001 From: pramodhkp Date: Thu, 14 Aug 2014 00:29:28 +0530 Subject: [PATCH] Minor changes for api.js, font_loader.js and svg.js --- examples/svgviewer/viewer.js | 11 ++--------- src/display/api.js | 2 +- src/display/font_loader.js | 3 +++ src/display/svg.js | 9 +++++---- 4 files changed, 11 insertions(+), 14 deletions(-) diff --git a/examples/svgviewer/viewer.js b/examples/svgviewer/viewer.js index e05807a94..90d8de3d0 100644 --- a/examples/svgviewer/viewer.js +++ b/examples/svgviewer/viewer.js @@ -46,17 +46,10 @@ PDFJS.getDocument(url).then(function(pdf) { container.style.height = viewport.height + 'px'; anchor.appendChild(container); - var renderContext = { - viewport: viewport, - pageNum: pageNum, - container: container - }; - // the next page fetch will start only after this page rendering is done return page.getOperatorList().then(function (opList) { - var svgGfx = new SVGGraphics(page.commonObjs, page.objs); + var svgGfx = new PDFJS.SVGGraphics(page.commonObjs, page.objs); return svgGfx.loadDependencies(opList).then(function (values) { - return svgGfx.beginDrawing(renderContext.viewport, - renderContext.pageNum, renderContext.container, opList); + container.appendChild(svgGfx.getSVG(viewport, pageNum, opList)); }); }); }); diff --git a/src/display/api.js b/src/display/api.js index ed5865714..a2d98239c 100644 --- a/src/display/api.js +++ b/src/display/api.js @@ -992,7 +992,7 @@ var WorkerTransport = (function WorkerTransportClosure() { messageHandler.on('PageError', function transportError(data) { var page = this.pageCache[data.pageNum - 1]; var intentState = page.intentStates[data.intent]; - if (intentState.displayReadyCapability.promise) { + if (intentState.displayReadyCapability) { intentState.displayReadyCapability.reject(data.error); } else { error(data.error); diff --git a/src/display/font_loader.js b/src/display/font_loader.js index 2083ea732..105ed8322 100644 --- a/src/display/font_loader.js +++ b/src/display/font_loader.js @@ -91,6 +91,9 @@ var FontLoader = { return true; } // TODO other browsers + if (userAgent === 'node') { + return true; + } return false; })(), diff --git a/src/display/svg.js b/src/display/svg.js index 52bd23035..124861a50 100644 --- a/src/display/svg.js +++ b/src/display/svg.js @@ -431,10 +431,8 @@ var SVGGraphics = (function SVGGraphicsClosure() { this.tgrp.setAttributeNS(null, 'transform', pm(this.transformMatrix)); }, - beginDrawing: function SVGGraphics_beginDrawing(viewport, pageNum, - container, operatorList) { + getSVG: function SVGGraphics_getSVG(viewport, pageNum, operatorList) { this.svg = createScratchSVG(viewport.width, viewport.height); - this.container = container; this.viewport = viewport; this.transformMatrix = IDENTITY_MATRIX; this.pgrp = document.createElementNS(NS, 'svg:g'); // Parent group @@ -445,9 +443,9 @@ var SVGGraphics = (function SVGGraphicsClosure() { this.pgrp.appendChild(this.defs); this.pgrp.appendChild(this.tgrp); this.svg.appendChild(this.pgrp); - this.container.appendChild(this.svg); var opTree = this.convertOpList(operatorList); this.executeOpTree(opTree); + return this.svg; }, convertOpList: function SVGGraphics_convertOpList(operatorList) { @@ -1114,3 +1112,6 @@ var SVGGraphics = (function SVGGraphicsClosure() { }; return SVGGraphics; })(); + +PDFJS.SVGGraphics = SVGGraphics; +