4a9994b54c
This patch will help reduce memory usage, especially for longer documents, when the user scrolls around in the thumbnailView (in the sidebar). Note how the `PDFPageProxy.cleanup` method will, assuming it's safe to do so, release main-thread resources associated with the page. These include things such as e.g. image data (which can be arbitrarily large), and also the operatorList (which can also be quite large). Hence when pages are evicted from the `PDFPageViewBuffer`, on the `BaseViewer`-instance, the `PDFPageView.destroy` method is invoked which will (among other things) call `PDFPageProxy.cleanup` in the API. However, looking at the `PDFThumbnailViewer`/`PDFThumbnailView` classes you'll notice that there's no attempt to ever call `PDFPageProxy.cleanup`, which implies that in certain circumstances we'll essentially keep all resources allocated permanently on the `PDFPageProxy`-instances in the API. In particular, this happens when the users opens the sidebar and starts scrolling around in the thumbnails. Generally speaking you obviously need to keep all thumbnail *images* around, since otherwise the thumbnailView is useless, but there's still room for improvement here. Please note that the case where a *rendered page* is used to create the thumbnail is (obviously) completely unaffected by the issues described above, and this rather only applies to thumbnails being explicitly rendered by the `PDFThumbnailView.draw` method. For the latter case, we can fix these issues simply by calling `PDFPageProxy.cleanup` once rendering has finished. To prevent *accidentally* pulling the rug out from under `PDFPageViewBuffer` in the viewer, which expects data to be available, this required adding a couple of new methods[1] to enable checking that it's indeed safe to call `PDFPageProxy.cleanup` from the `PDFThumbnailView.draw` method. It's really quite fascinating that no one has noticed this issue before, since it's been around since basically "forever". --- [1] While it should be *very* rare for `PDFThumbnailView.draw` to be called for a pageView that's also in the `PDFPageViewBuffer`, given that pages are rendered before thumbnails and that the *rendered page* is used to create the thumbnail, it can still happen since rendering is asynchronous. Furthermore, it's also possible for `PDFThumbnailView.setImage` to be disabled, in which case checking the `PDFPageViewBuffer` for active pageViews *really* matters.
246 lines
4.7 KiB
JavaScript
246 lines
4.7 KiB
JavaScript
/* Copyright 2018 Mozilla Foundation
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
/* eslint-disable getter-return */
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IPDFLinkService {
|
|
/**
|
|
* @type {number}
|
|
*/
|
|
get pagesCount() {}
|
|
|
|
/**
|
|
* @type {number}
|
|
*/
|
|
get page() {}
|
|
|
|
/**
|
|
* @param {number} value
|
|
*/
|
|
set page(value) {}
|
|
|
|
/**
|
|
* @type {number}
|
|
*/
|
|
get rotation() {}
|
|
|
|
/**
|
|
* @param {number} value
|
|
*/
|
|
set rotation(value) {}
|
|
|
|
/**
|
|
* @type {boolean}
|
|
*/
|
|
get externalLinkEnabled() {}
|
|
|
|
/**
|
|
* @param {boolean} value
|
|
*/
|
|
set externalLinkEnabled(value) {}
|
|
|
|
/**
|
|
* @param {string|Array} dest - The named, or explicit, PDF destination.
|
|
*/
|
|
async goToDestination(dest) {}
|
|
|
|
/**
|
|
* @param {number|string} val - The page number, or page label.
|
|
*/
|
|
goToPage(val) {}
|
|
|
|
/**
|
|
* @param dest - The PDF destination object.
|
|
* @returns {string} The hyperlink to the PDF object.
|
|
*/
|
|
getDestinationHash(dest) {}
|
|
|
|
/**
|
|
* @param hash - The PDF parameters/hash.
|
|
* @returns {string} The hyperlink to the PDF object.
|
|
*/
|
|
getAnchorUrl(hash) {}
|
|
|
|
/**
|
|
* @param {string} hash
|
|
*/
|
|
setHash(hash) {}
|
|
|
|
/**
|
|
* @param {string} action
|
|
*/
|
|
executeNamedAction(action) {}
|
|
|
|
/**
|
|
* @param {number} pageNum - page number.
|
|
* @param {Object} pageRef - reference to the page.
|
|
*/
|
|
cachePageRef(pageNum, pageRef) {}
|
|
|
|
/**
|
|
* @param {number} pageNumber
|
|
*/
|
|
isPageVisible(pageNumber) {}
|
|
|
|
/**
|
|
* @param {number} pageNumber
|
|
*/
|
|
isPageCached(pageNumber) {}
|
|
}
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IPDFHistory {
|
|
/**
|
|
* @param {Object} params
|
|
*/
|
|
initialize({ fingerprint, resetHistory = false, updateUrl = false }) {}
|
|
|
|
reset() {}
|
|
|
|
/**
|
|
* @param {Object} params
|
|
*/
|
|
push({ namedDest = null, explicitDest, pageNumber }) {}
|
|
|
|
/**
|
|
* @param {number} pageNumber
|
|
*/
|
|
pushPage(pageNumber) {}
|
|
|
|
pushCurrentPosition() {}
|
|
|
|
back() {}
|
|
|
|
forward() {}
|
|
}
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IRenderableView {
|
|
/**
|
|
* @type {string} - Unique ID for rendering queue.
|
|
*/
|
|
get renderingId() {}
|
|
|
|
/**
|
|
* @type {RenderingStates}
|
|
*/
|
|
get renderingState() {}
|
|
|
|
/**
|
|
* @returns {Promise} Resolved on draw completion.
|
|
*/
|
|
draw() {}
|
|
|
|
resume() {}
|
|
}
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IPDFTextLayerFactory {
|
|
/**
|
|
* @param {HTMLDivElement} textLayerDiv
|
|
* @param {number} pageIndex
|
|
* @param {PageViewport} viewport
|
|
* @param {boolean} enhanceTextSelection
|
|
* @param {EventBus} eventBus
|
|
* @returns {TextLayerBuilder}
|
|
*/
|
|
createTextLayerBuilder(
|
|
textLayerDiv,
|
|
pageIndex,
|
|
viewport,
|
|
enhanceTextSelection = false,
|
|
eventBus
|
|
) {}
|
|
}
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IPDFAnnotationLayerFactory {
|
|
/**
|
|
* @param {HTMLDivElement} pageDiv
|
|
* @param {PDFPage} pdfPage
|
|
* @param {AnnotationStorage} [annotationStorage] - Storage for annotation
|
|
* data in forms.
|
|
* @param {string} [imageResourcesPath] - Path for image resources, mainly
|
|
* for annotation icons. Include trailing slash.
|
|
* @param {boolean} renderInteractiveForms
|
|
* @param {IL10n} l10n
|
|
* @param {boolean} [enableScripting]
|
|
* @param {Promise<boolean>} [hasJSActionsPromise]
|
|
* @returns {AnnotationLayerBuilder}
|
|
*/
|
|
createAnnotationLayerBuilder(
|
|
pageDiv,
|
|
pdfPage,
|
|
annotationStorage = null,
|
|
imageResourcesPath = "",
|
|
renderInteractiveForms = true,
|
|
l10n = undefined,
|
|
enableScripting = false,
|
|
hasJSActionsPromise = null
|
|
) {}
|
|
}
|
|
|
|
/**
|
|
* @interface
|
|
*/
|
|
class IL10n {
|
|
/**
|
|
* @returns {Promise<string>} - Resolves to the current locale.
|
|
*/
|
|
async getLanguage() {}
|
|
|
|
/**
|
|
* @returns {Promise<string>} - Resolves to 'rtl' or 'ltr'.
|
|
*/
|
|
async getDirection() {}
|
|
|
|
/**
|
|
* Translates text identified by the key and adds/formats data using the args
|
|
* property bag. If the key was not found, translation falls back to the
|
|
* fallback text.
|
|
* @param {string} key
|
|
* @param {object} args
|
|
* @param {string} fallback
|
|
* @returns {Promise<string>}
|
|
*/
|
|
async get(key, args, fallback) {}
|
|
|
|
/**
|
|
* Translates HTML element.
|
|
* @param {HTMLElement} element
|
|
* @returns {Promise<void>}
|
|
*/
|
|
async translate(element) {}
|
|
}
|
|
|
|
export {
|
|
IPDFLinkService,
|
|
IPDFHistory,
|
|
IRenderableView,
|
|
IPDFTextLayerFactory,
|
|
IPDFAnnotationLayerFactory,
|
|
IL10n,
|
|
};
|