Cristian Fogel [:cfogel] Reporter
	Description • 6 years ago

[Affected versions]:
- Firefox: 63.0.1, 64.0b7, 65.0a1 (2018-11-05)

[Affected platforms]:
- win 10x64; macOS 10.12; Ubuntu16.04;

[Steps to reproduce]:
1. Launch Firefox;
2. Open DevTools-Console;
3. Access any pdf file, ex: http://www.nwu.ac.za/files/images/Basic_Curriculum_Vitae_example.pdf
4. Click inside the current page button;
5. Input a text not numeric value;
6. Press the Enter key;

[Expected result]:
- Error message displayed in the console;

[Actual result]:
- PDFViewer_setCurrentPageNumber: "0" is out of bounds. 
- the fact that it's not a numeric value in the input, a proper message could be displayed in the logs;

[Regression range]:
marking dates on when this log message was added;
- last good: 2016-08-18
- first bad: 2016-08-19

[Additional notes]:
- attached screenshot with the issue:
* the first message was for a numeric value;
* the second message was for a text value;
- older versions did not post any error message and set the value to 1;
- this can be viewed more as an enhancement suggestion;

	Nicolas Chevobbe [:nchevobbe]
	Comment 1 • 6 years ago

The message is printed from https://searchfox.org/mozilla-central/rev/17f55aee76b7c4610a974cffd3453454e0c8de7b/browser/extensions/pdfjs/content/web/viewer.js#6590 

Here's the stacktrace: 

_setCurrentPageNumber (viewer.js#6590)
currentPageNumber (viewer.js#6580)
currentPageLabel (viewer.js#6614)
webViewerPageNumberChanged (viewer.js#1498)
dispatch (viewer.js#2309)
dispatch (viewer.js#2308)
_bindListeners (viewer.js#8558)


This comes from a "change" event listener https://searchfox.org/mozilla-central/rev/17f55aee76b7c4610a974cffd3453454e0c8de7b/browser/extensions/pdfjs/content/web/viewer.js#8558 in a input[type=number]