Explorar el Código

use new pdfjs getDestination method in pdfRenderer

gives faster lookup of link destinations in pdf file
Brian Gough hace 11 años
padre
commit
b893446b9c

+ 2 - 14
services/web/public/coffee/ide/pdfng/directives/pdfRenderer.coffee

@@ -60,22 +60,10 @@ define [
 					pdfDocument.getDestinations()
 
 			getDestination: (dest) ->
-				# There is a direct method for this in pdf.js but it is not
-				# available in pdf.js-1.0.712. Use the following workaround of
-				# getting all the destinations and returning only the one we
-				# want.
-				@destinations = @document.then (pdfDocument) ->
-					pdfDocument.getDestinations()
-				return @destinations.then (all) ->
-					all[dest]
+				@document.then (pdfDocument) ->
+					pdfDocument.getDestination(dest)
 				, (error) =>
 					@errorCallback?(error)
-				# When we upgrade we can switch to using the following direct
-				# code.
-				# @document.then (pdfDocument) ->
-				# 	pdfDocument.getDestination(dest)
-				# , (error) ->
-				# 	console.log 'ERROR', error
 
 			getPageIndex: (ref) ->
 				@document.then (pdfDocument) =>