Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #4210 from yurydelendik/issue4131

Browse Source 
Refactors nextPageMatch (nits)
This commit is contained in:
Yury Delendik 2014-01-28 09:13:20 -08:00
commit c45c9a7ffa
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
web/pdf_find_controller.js
View File

@ -296,11 +296,10 @@ var PDFFindController = {
}, },
nextPageMatch: function() { nextPageMatch: function() {
if (this.resumePageIdx !== null) if (this.resumePageIdx !== null) {
console.error('There can only be one pending page.'); console.error('There can only be one pending page.');
// done boolean to remove do-while construct per review of PR 4131 }
var done = false; do {
while (!done) {
var pageIdx = this.offset.pageIdx; var pageIdx = this.offset.pageIdx;
var matches = this.pageMatches[pageIdx]; var matches = this.pageMatches[pageIdx];
if (!matches) { if (!matches) {
@ -309,8 +308,7 @@ var PDFFindController = {
this.resumePageIdx = pageIdx; this.resumePageIdx = pageIdx;
break; break;
} }
done = this.matchesReady(matches); } while (!this.matchesReady(matches));
}
}, },
advanceOffsetPage: function(previous) { advanceOffsetPage: function(previous) {