Fixes lint warning W004 in /web

This commit is contained in:
Tim van der Meij 2014-04-07 22:34:03 +02:00
parent 10f80bda23
commit b348243866
3 changed files with 16 additions and 13 deletions

View File

@ -58,17 +58,17 @@ if (typeof PDFJS === 'undefined') {
} }
function TypedArray(arg1) { function TypedArray(arg1) {
var result; var result, i, n;
if (typeof arg1 === 'number') { if (typeof arg1 === 'number') {
result = []; result = [];
for (var i = 0; i < arg1; ++i) { for (i = 0; i < arg1; ++i) {
result[i] = 0; result[i] = 0;
} }
} else if ('slice' in arg1) { } else if ('slice' in arg1) {
result = arg1.slice(0); result = arg1.slice(0);
} else { } else {
result = []; result = [];
for (var i = 0, n = arg1.length; i < n; ++i) { for (i = 0, n = arg1.length; i < n; ++i) {
result[i] = arg1[i]; result[i] = arg1[i];
} }
} }

View File

@ -211,10 +211,11 @@ var StepperManager = (function StepperManagerClosure() {
return stepper; return stepper;
}, },
selectStepper: function selectStepper(pageIndex, selectPanel) { selectStepper: function selectStepper(pageIndex, selectPanel) {
var i;
if (selectPanel) { if (selectPanel) {
this.manager.selectPanel(this); this.manager.selectPanel(this);
} }
for (var i = 0; i < steppers.length; ++i) { for (i = 0; i < steppers.length; ++i) {
var stepper = steppers[i]; var stepper = steppers[i];
if (stepper.pageIndex == pageIndex) { if (stepper.pageIndex == pageIndex) {
stepper.panel.removeAttribute('hidden'); stepper.panel.removeAttribute('hidden');
@ -223,7 +224,7 @@ var StepperManager = (function StepperManagerClosure() {
} }
} }
var options = stepperChooser.options; var options = stepperChooser.options;
for (var i = 0; i < options.length; ++i) { for (i = 0; i < options.length; ++i) {
var option = options[i]; var option = options[i];
option.selected = option.value == pageIndex; option.selected = option.value == pageIndex;
} }
@ -344,7 +345,7 @@ var Stepper = (function StepperClosure() {
var self = this; var self = this;
var chunk = document.createDocumentFragment(); var chunk = document.createDocumentFragment();
var operatorsToDisplay = Math.min(MAX_OPERATORS_COUNT, var operatorsToDisplay = Math.min(MAX_OPERATORS_COUNT,
operatorList.fnArray.length); operatorList.fnArray.length);
for (var i = this.operatorListIdx; i < operatorsToDisplay; i++) { for (var i = this.operatorListIdx; i < operatorsToDisplay; i++) {
var line = c('tr'); var line = c('tr');
line.className = 'line'; line.className = 'line';
@ -369,7 +370,7 @@ var Stepper = (function StepperClosure() {
if (fn in glyphCommands) { if (fn in glyphCommands) {
var glyphIndex = glyphCommands[fn]; var glyphIndex = glyphCommands[fn];
var glyphs = args[glyphIndex]; var glyphs = args[glyphIndex];
var decArgs = args.slice(); decArgs = args.slice();
var newArg; var newArg;
if (fn === 'showSpacedText') { if (fn === 'showSpacedText') {
newArg = []; newArg = [];

View File

@ -639,7 +639,7 @@ var PDFView = {
if (exception && exception.name === 'InvalidPDFException') { if (exception && exception.name === 'InvalidPDFException') {
// change error message also for other builds // change error message also for other builds
var loadingErrorMessage = mozL10n.get('invalid_file_error', null, loadingErrorMessage = mozL10n.get('invalid_file_error', null,
'Invalid or corrupted PDF file.'); 'Invalid or corrupted PDF file.');
//#if B2G //#if B2G
// window.alert(loadingErrorMessage); // window.alert(loadingErrorMessage);
@ -649,7 +649,7 @@ var PDFView = {
if (exception && exception.name === 'MissingPDFException') { if (exception && exception.name === 'MissingPDFException') {
// special message for missing PDF's // special message for missing PDF's
var loadingErrorMessage = mozL10n.get('missing_file_error', null, loadingErrorMessage = mozL10n.get('missing_file_error', null,
'Missing PDF file.'); 'Missing PDF file.');
//#if B2G //#if B2G
@ -1478,10 +1478,11 @@ var PDFView = {
} }
var alertNotReady = false; var alertNotReady = false;
var i, ii;
if (!this.pages.length) { if (!this.pages.length) {
alertNotReady = true; alertNotReady = true;
} else { } else {
for (var i = 0, ii = this.pages.length; i < ii; ++i) { for (i = 0, ii = this.pages.length; i < ii; ++i) {
if (!this.pages[i].pdfPage) { if (!this.pages[i].pdfPage) {
alertNotReady = true; alertNotReady = true;
break; break;
@ -1497,7 +1498,7 @@ var PDFView = {
var body = document.querySelector('body'); var body = document.querySelector('body');
body.setAttribute('data-mozPrintCallback', true); body.setAttribute('data-mozPrintCallback', true);
for (var i = 0, ii = this.pages.length; i < ii; ++i) { for (i = 0, ii = this.pages.length; i < ii; ++i) {
this.pages[i].beforePrint(); this.pages[i].beforePrint();
} }
}, },
@ -1511,14 +1512,15 @@ var PDFView = {
rotatePages: function pdfViewRotatePages(delta) { rotatePages: function pdfViewRotatePages(delta) {
var currentPage = this.pages[this.page - 1]; var currentPage = this.pages[this.page - 1];
var i, l;
this.pageRotation = (this.pageRotation + 360 + delta) % 360; this.pageRotation = (this.pageRotation + 360 + delta) % 360;
for (var i = 0, l = this.pages.length; i < l; i++) { for (i = 0, l = this.pages.length; i < l; i++) {
var page = this.pages[i]; var page = this.pages[i];
page.update(page.scale, this.pageRotation); page.update(page.scale, this.pageRotation);
} }
for (var i = 0, l = this.thumbnails.length; i < l; i++) { for (i = 0, l = this.thumbnails.length; i < l; i++) {
var thumb = this.thumbnails[i]; var thumb = this.thumbnails[i];
thumb.update(this.pageRotation); thumb.update(this.pageRotation);
} }