Merge pull request #8807 from Snuffleupagus/PDFViewer-scrollPageIntoView-rm-fallback-args

Remove handling of fallback arguments from `PDFViewer.scrollPageIntoView`
This commit is contained in:
Tim van der Meij 2017-08-23 00:24:24 +02:00 committed by GitHub
commit 07f64e064d

View File

@ -581,20 +581,13 @@ class PDFViewer {
* @param {ScrollPageIntoViewParameters} params * @param {ScrollPageIntoViewParameters} params
*/ */
scrollPageIntoView(params) { scrollPageIntoView(params) {
if (!this.pdfDocument) {
return;
}
if ((typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) && if ((typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) &&
(arguments.length > 1 || typeof params === 'number')) { (arguments.length > 1 || typeof params === 'number')) {
console.warn('Call of scrollPageIntoView() with obsolete signature.'); console.error('Call of scrollPageIntoView() with obsolete signature.');
let paramObj = {}; return;
if (typeof params === 'number') { }
paramObj.pageNumber = params; // pageNumber argument was found. if (!this.pdfDocument) {
} return;
if (arguments[1] instanceof Array) {
paramObj.destArray = arguments[1]; // destArray argument was found.
}
params = paramObj;
} }
let pageNumber = params.pageNumber || 0; let pageNumber = params.pageNumber || 0;
let dest = params.destArray || null; let dest = params.destArray || null;