From 0fa44d7233bb4190099ac96e24efe21155014f93 Mon Sep 17 00:00:00 2001 From: Thorben Bochenek Date: Wed, 12 Mar 2014 09:51:40 +0100 Subject: [PATCH 1/2] refactor src/core/pdf_manager.js: rename pdfModel to pdfDocument Rename pdfModel to pdfDocument to let the name closer resemble what the variable actually holds --- src/core/worker.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/core/worker.js b/src/core/worker.js index a092b783a..69f284dba 100644 --- a/src/core/worker.js +++ b/src/core/worker.js @@ -294,7 +294,8 @@ var WorkerMessageHandler = PDFJS.WorkerMessageHandler = { handler.on('GetPageIndex', function wphSetupGetPageIndex(data, deferred) { var ref = new Ref(data.ref.num, data.ref.gen); - pdfManager.pdfModel.catalog.getPageIndex(ref).then(function (pageIndex) { + var catalog = pdfManager.pdfDocument.catalog; + catalog.getPageIndex(ref).then(function (pageIndex) { deferred.resolve(pageIndex); }, deferred.reject); }); From c547f17ee5eb086fa759472a441c1e97a8801f52 Mon Sep 17 00:00:00 2001 From: Thorben Bochenek Date: Wed, 19 Mar 2014 10:17:58 +0100 Subject: [PATCH 2/2] Add Test for PDFDocumentProxy_getPageIndex --- test/unit/api_spec.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/test/unit/api_spec.js b/test/unit/api_spec.js index eee5fcfe7..ff2286012 100644 --- a/test/unit/api_spec.js +++ b/test/unit/api_spec.js @@ -54,6 +54,14 @@ describe('api', function() { expect(true).toEqual(true); }); }); + it('gets page index', function() { + // reference to second page + var ref = {num: 17, gen: 0}; + var promise = doc.getPageIndex(ref); + waitsForPromise(promise, function(pageIndex) { + expect(pageIndex).toEqual(1); + }); + }); it('gets destinations', function() { var promise = doc.getDestinations(); waitsForPromise(promise, function(data) {