diff --git a/web/pdf_find_controller.js b/web/pdf_find_controller.js index 92416330e..530735d04 100644 --- a/web/pdf_find_controller.js +++ b/web/pdf_find_controller.js @@ -86,7 +86,7 @@ class PDFFindController { }); } - normalize(text) { + _normalize(text) { return text.replace(this.normalizationRegex, function (ch) { return CHARACTERS_TO_NORMALIZE[ch]; }); @@ -143,7 +143,7 @@ class PDFFindController { } } - calcFindPhraseMatch(query, pageIndex, pageContent) { + _calculatePhraseMatch(query, pageIndex, pageContent) { let matches = []; let queryLen = query.length; let matchIdx = -queryLen; @@ -157,7 +157,7 @@ class PDFFindController { this.pageMatches[pageIndex] = matches; } - calcFindWordMatch(query, pageIndex, pageContent) { + _calculateWordMatch(query, pageIndex, pageContent) { let matchesWithLength = []; // Divide the query into pieces and search for text in each piece. let queryArray = query.match(/\S+/g); @@ -192,9 +192,9 @@ class PDFFindController { this.pageMatchesLength[pageIndex]); } - calcFindMatch(pageIndex) { - let pageContent = this.normalize(this.pageContents[pageIndex]); - let query = this.normalize(this.state.query); + _calculateMatch(pageIndex) { + let pageContent = this._normalize(this.pageContents[pageIndex]); + let query = this._normalize(this.state.query); let caseSensitive = this.state.caseSensitive; let phraseSearch = this.state.phraseSearch; let queryLen = query.length; @@ -210,25 +210,25 @@ class PDFFindController { } if (phraseSearch) { - this.calcFindPhraseMatch(query, pageIndex, pageContent); + this._calculatePhraseMatch(query, pageIndex, pageContent); } else { - this.calcFindWordMatch(query, pageIndex, pageContent); + this._calculateWordMatch(query, pageIndex, pageContent); } - this.updatePage(pageIndex); + this._updatePage(pageIndex); if (this.resumePageIdx === pageIndex) { this.resumePageIdx = null; - this.nextPageMatch(); + this._nextPageMatch(); } // Update the match count. if (this.pageMatches[pageIndex].length > 0) { this.matchCount += this.pageMatches[pageIndex].length; - this.updateUIResultsCount(); + this._updateUIResultsCount(); } } - extractText() { + _extractText() { if (this.startedTextExtraction) { return; } @@ -266,23 +266,23 @@ class PDFFindController { this.dirtyMatch = true; } this.state = state; - this.updateUIState(FindState.PENDING); + this._updateUIState(FindState.PENDING); this._firstPagePromise.then(() => { - this.extractText(); + this._extractText(); clearTimeout(this.findTimeout); if (cmd === 'find') { // Trigger the find action with a small delay to avoid starting the // search when the user is still typing (saving resources). - this.findTimeout = setTimeout(this.nextMatch.bind(this), FIND_TIMEOUT); + this.findTimeout = setTimeout(this._nextMatch.bind(this), FIND_TIMEOUT); } else { - this.nextMatch(); + this._nextMatch(); } }); } - updatePage(index) { + _updatePage(index) { if (this.selected.pageIdx === index) { // If the page is selected, scroll the page into view, which triggers // rendering the page, which adds the textLayer. Once the textLayer is @@ -296,7 +296,7 @@ class PDFFindController { } } - nextMatch() { + _nextMatch() { let previous = this.state.findPrevious; let currentPageIndex = this.pdfViewer.currentPageNumber - 1; let numPages = this.pdfViewer.pagesCount; @@ -317,14 +317,14 @@ class PDFFindController { for (let i = 0; i < numPages; i++) { // Wipe out any previously highlighted matches. - this.updatePage(i); + this._updatePage(i); // Start finding the matches as soon as the text is extracted. if (!(i in this.pendingFindMatches)) { this.pendingFindMatches[i] = true; this.extractTextPromises[i].then((pageIdx) => { delete this.pendingFindMatches[pageIdx]; - this.calcFindMatch(pageIdx); + this._calculateMatch(pageIdx); }); } } @@ -332,7 +332,7 @@ class PDFFindController { // If there's no query there's no point in searching. if (this.state.query === '') { - this.updateUIState(FindState.FOUND); + this._updateUIState(FindState.FOUND); return; } @@ -355,18 +355,18 @@ class PDFFindController { this.hadMatch = true; offset.matchIdx = (previous ? offset.matchIdx - 1 : offset.matchIdx + 1); - this.updateMatch(true); + this._updateMatch(/* found = */ true); return; } // We went beyond the current page's matches, so we advance to // the next page. - this.advanceOffsetPage(previous); + this._advanceOffsetPage(previous); } // Start searching through the page. - this.nextPageMatch(); + this._nextPageMatch(); } - matchesReady(matches) { + _matchesReady(matches) { let offset = this.offset; let numMatches = matches.length; let previous = this.state.findPrevious; @@ -375,16 +375,16 @@ class PDFFindController { // There were matches for the page, so initialize `matchIdx`. this.hadMatch = true; offset.matchIdx = (previous ? numMatches - 1 : 0); - this.updateMatch(true); + this._updateMatch(/* found = */ true); return true; } // No matches, so attempt to search the next page. - this.advanceOffsetPage(previous); + this._advanceOffsetPage(previous); if (offset.wrapped) { offset.matchIdx = null; if (this.pagesToSearch < 0) { // No point in wrapping again, there were no matches. - this.updateMatch(false); + this._updateMatch(/* found = */ false); // While matches were not found, searching for a page // with matches should nevertheless halt. return true; @@ -414,7 +414,7 @@ class PDFFindController { } } - nextPageMatch() { + _nextPageMatch() { if (this.resumePageIdx !== null) { console.error('There can only be one pending page.'); } @@ -424,15 +424,15 @@ class PDFFindController { let pageIdx = this.offset.pageIdx; matches = this.pageMatches[pageIdx]; if (!matches) { - // The matches don't exist yet for processing by `matchesReady`, + // The matches don't exist yet for processing by `_matchesReady`, // so set a resume point for when they do exist. this.resumePageIdx = pageIdx; break; } - } while (!this.matchesReady(matches)); + } while (!this._matchesReady(matches)); } - advanceOffsetPage(previous) { + _advanceOffsetPage(previous) { let offset = this.offset; let numPages = this.extractTextPromises.length; offset.pageIdx = (previous ? offset.pageIdx - 1 : offset.pageIdx + 1); @@ -446,7 +446,7 @@ class PDFFindController { } } - updateMatch(found = false) { + _updateMatch(found = false) { let state = FindState.NOT_FOUND; let wrapped = this.offset.wrapped; this.offset.wrapped = false; @@ -459,23 +459,23 @@ class PDFFindController { // Update the currently selected page to wipe out any selected matches. if (previousPage !== -1 && previousPage !== this.selected.pageIdx) { - this.updatePage(previousPage); + this._updatePage(previousPage); } } - this.updateUIState(state, this.state.findPrevious); + this._updateUIState(state, this.state.findPrevious); if (this.selected.pageIdx !== -1) { - this.updatePage(this.selected.pageIdx); + this._updatePage(this.selected.pageIdx); } } - updateUIResultsCount() { + _updateUIResultsCount() { if (this.onUpdateResultsCount) { this.onUpdateResultsCount(this.matchCount); } } - updateUIState(state, previous) { + _updateUIState(state, previous) { if (this.onUpdateState) { this.onUpdateState(state, previous, this.matchCount); }