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

Fix the global variable definitions in test/resources/reftest-analyzer.js (issue 13862)

Browse Source 
It shouldn't be necessary to assign these variables to the global scope (as far as I can tell), either explicitly with `window` or implicitly with `var`, and this way we don't need to disable the ESLint `no-undef` rule; fixes another small part of issue 13862.

*Please note:* I wasn't going to put additional work into this code after PR 13869, however these changes looked so simple that I figured trying to get rid of the few remaining "Code scanning alerts" wouldn't hurt.
However, this file would still very much benefit from additional clean-up and re-factoring work, since it's quite old and currently contains some dead code (commented out).
This commit is contained in:
Jonas Jenwald 2021-08-06 11:25:24 +02:00
parent 849bab973c
commit df79b831f4
1 changed files with 14 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
test/resources/reftest-analyzer.js
View File

@ -28,21 +28,20 @@ the terms of any one of the MPL, the GPL or the LGPL.
Original author: L. David Baron <dbaron@dbaron.org> Original author: L. David Baron <dbaron@dbaron.org>
*/ */
/* eslint-disable no-undef */
// Global variables // Global variables
window.gPhases = null; const XLINK_NS = "http://www.w3.org/1999/xlink";
window.XLINK_NS = "http://www.w3.org/1999/xlink"; const SVG_NS = "http://www.w3.org/2000/svg";
window.SVG_NS = "http://www.w3.org/2000/svg"; let gPhases = null;
window.gMagPixPaths = []; // 2D array of array-of-two <path> objects used in the pixel magnifier const gMagPixPaths = []; // 2D array of array-of-two <path> objects used in the pixel magnifier
window.gMagWidth = 5; // number of zoomed in pixels to show horizontally const gMagWidth = 5; // number of zoomed in pixels to show horizontally
window.gMagHeight = 5; // number of zoomed in pixels to show vertically const gMagHeight = 5; // number of zoomed in pixels to show vertically
window.gMagZoom = 16; // size of the zoomed in pixels const gMagZoom = 16; // size of the zoomed in pixels
window.gImage1Data = null; // ImageData object for the test output image let gImage1Data = null; // ImageData object for the test output image
window.gImage2Data = null; // ImageData object for the reference image let gImage2Data = null; // ImageData object for the reference image
window.gFlashingPixels = []; // array of <path> objects that should be flashed due to pixel color mismatch const gFlashingPixels = []; // array of <path> objects that should be flashed due to pixel color mismatch
window.gPath = ""; // path taken from #web= and prepended to ref/snp urls let gPath = ""; // path taken from #web= and prepended to ref/snp urls
window.gSelected = null; // currently selected comparison let gSelected = null; // currently selected comparison
window.onload = function () { window.onload = function () {
load(); load();
@ -201,11 +200,10 @@ window.onload = function () {
processLog(log); processLog(log);
} }
let gTestItems; const gTestItems = [];
function processLog(contents) { function processLog(contents) {
const lines = contents.split(/[\r\n]+/); const lines = contents.split(/[\r\n]+/);
gTestItems = [];
for (const j in lines) { for (const j in lines) {
let line = lines[j]; let line = lines[j];
let match = line.match(/^(?:NEXT ERROR )?REFTEST (.*)$/); let match = line.match(/^(?:NEXT ERROR )?REFTEST (.*)$/);
@ -452,7 +450,7 @@ window.onload = function () {
const dy_hi = Math.floor(gMagHeight / 2); const dy_hi = Math.floor(gMagHeight / 2);
flashPixels(false); flashPixels(false);
gFlashingPixels = []; gFlashingPixels.length = 0;
for (let j = dy_lo; j <= dy_hi; j++) { for (let j = dy_lo; j <= dy_hi; j++) {
for (let i = dx_lo; i <= dx_hi; i++) { for (let i = dx_lo; i <= dx_hi; i++) {
const px = x + i; const px = x + i;