github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-07 20:31:06 -05:00
Code Issues Projects Releases Packages Wiki Activity

use a single pdfjs worker, rotated to avoid leaks

Browse source
This commit is contained in:
Brian Gough 2016-10-13 11:26:24 +01:00
parent 85d8c2ed2e
commit c98f40b90b
1 changed files with 20 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
services/web/public/coffee/ide/pdfng/directives/pdfRenderer.coffee
View file

@ -6,6 +6,22 @@ define [
App.factory 'PDFRenderer', ['$q', '$timeout', 'pdfAnnotations', 'pdfTextLayer', 'pdfSpinner', ($q, $timeout, pdfAnnotations, pdfTextLayer, pdfSpinner) ->
# Have a single worker used by all rendering, to avoid reloading
RenderThread = { worker: null, count: 0}
getRenderThread = () ->
if RenderThread.count > 16 # recycle the worker periodically to avoid leaks
RenderThread.readyToDestroy = true
RenderThread = { worker: null, count: 0 }
RenderThread.worker ||= new PDFJS.PDFWorker('pdfjsworker')
RenderThread.count++
return RenderThread
resetWorker = (thread) ->
thread.worker.destroy() if thread.readyToDestroy
# The PDF page renderer
class PDFRenderer
JOB_QUEUE_INTERVAL: 25
PAGE_LOAD_TIMEOUT: 60*1000
@ -22,7 +38,8 @@ define [
# PDFJS.disableStream
# PDFJS.disableRange
@scale = @options.scale || 1
@pdfjs = PDFJS.getDocument {url: @url, rangeChunkSize: 2*65536}
@thread = getRenderThread()
@pdfjs = PDFJS.getDocument {url: @url, rangeChunkSize: 2*65536, worker: @thread.worker}
@pdfjs.onProgress = @options.progressCallback
@document = $q.when(@pdfjs)
@navigateFn = @options.navigateFn
@ -327,8 +344,9 @@ define [
destroy: () ->
@shuttingDown = true
@resetState()
@pdfjs.then (document) ->
@pdfjs.then (document) =>
document.cleanup()
document.destroy()
resetWorker(@thread)
]