Browse Source

Add the number of `textDivs` to the `textlayerrendered` event, to avoid having to manually get that information in the event handler

Jonas Jenwald 9 years ago
parent
commit
885ba83624
  1. 7
      web/app.js
  2. 6
      web/text_layer_builder.js

7
web/app.js

@ -1632,18 +1632,13 @@ function webViewerPageRendered(e) {
} }
function webViewerTextLayerRendered(e) { function webViewerTextLayerRendered(e) {
var pageIndex = e.pageNumber - 1;
var pageView = PDFViewerApplication.pdfViewer.getPageView(pageIndex);
//#if !PRODUCTION //#if !PRODUCTION
if (true) { if (true) {
return; return;
} }
//#endif //#endif
//#if (FIREFOX || MOZCENTRAL) //#if (FIREFOX || MOZCENTRAL)
if (pageView && pageView.textLayer && pageView.textLayer.textDivs && if (e.numTextDivs > 0 && !PDFViewerApplication.supportsDocumentColors) {
pageView.textLayer.textDivs.length > 0 &&
!PDFViewerApplication.supportsDocumentColors) {
console.error(mozL10n.get('document_colors_not_allowed', null, console.error(mozL10n.get('document_colors_not_allowed', null,
'PDF documents are not allowed to use their own colors: ' + 'PDF documents are not allowed to use their own colors: ' +
'\'Allow pages to choose their own colors\' ' + '\'Allow pages to choose their own colors\' ' +

6
web/text_layer_builder.js

@ -65,6 +65,9 @@ var TextLayerBuilder = (function TextLayerBuilderClosure() {
} }
TextLayerBuilder.prototype = { TextLayerBuilder.prototype = {
/**
* @private
*/
_finishRendering: function TextLayerBuilder_finishRendering() { _finishRendering: function TextLayerBuilder_finishRendering() {
this.renderingDone = true; this.renderingDone = true;
@ -76,7 +79,8 @@ var TextLayerBuilder = (function TextLayerBuilderClosure() {
this.eventBus.dispatch('textlayerrendered', { this.eventBus.dispatch('textlayerrendered', {
source: this, source: this,
pageNumber: this.pageNumber pageNumber: this.pageNumber,
numTextDivs: this.textDivs.length,
}); });
}, },

Loading…
Cancel
Save