Refactors nextPageMatch (nits)

This commit is contained in:
Yury Delendik 2014-01-28 11:11:02 -06:00
parent 187c9007ef
commit 48f0d0a131

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) {