Add test for CSS-only zoom above maxCanvasPixels
This commit introduces a test to ensure that: - When zooming below the maxCanvasPixels limit, the canvas is rendered with the correct size (the two sides are multiplied by the zoom factor). - When zooming above the maxCanvasPixels limit, the canvas size is capped.
This commit is contained in:
parent
95a7de9f98
commit
12692d2e39
@ -38,8 +38,13 @@ function loadAndWait(filename, selector, zoom, pageSetup, options) {
|
|||||||
|
|
||||||
let app_options = "";
|
let app_options = "";
|
||||||
if (options) {
|
if (options) {
|
||||||
|
const optionsObject =
|
||||||
|
typeof options === "function"
|
||||||
|
? await options(page, session.name)
|
||||||
|
: options;
|
||||||
|
|
||||||
// Options must be handled in app.js::_parseHashParams.
|
// Options must be handled in app.js::_parseHashParams.
|
||||||
for (const [key, value] of Object.entries(options)) {
|
for (const [key, value] of Object.entries(optionsObject)) {
|
||||||
app_options += `&${key}=${encodeURIComponent(value)}`;
|
app_options += `&${key}=${encodeURIComponent(value)}`;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -94,24 +94,6 @@ describe("PDF viewer", () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
describe("CSS-only zoom", () => {
|
describe("CSS-only zoom", () => {
|
||||||
let pages;
|
|
||||||
|
|
||||||
beforeAll(async () => {
|
|
||||||
pages = await loadAndWait(
|
|
||||||
"tracemonkey.pdf",
|
|
||||||
".textLayer .endOfContent",
|
|
||||||
null,
|
|
||||||
null,
|
|
||||||
{
|
|
||||||
maxCanvasPixels: 0,
|
|
||||||
}
|
|
||||||
);
|
|
||||||
});
|
|
||||||
|
|
||||||
afterAll(async () => {
|
|
||||||
await closePages(pages);
|
|
||||||
});
|
|
||||||
|
|
||||||
function createPromiseForFirstPageRendered(page) {
|
function createPromiseForFirstPageRendered(page) {
|
||||||
return createPromise(page, (resolve, reject) => {
|
return createPromise(page, (resolve, reject) => {
|
||||||
const controller = new AbortController();
|
const controller = new AbortController();
|
||||||
@ -129,50 +111,183 @@ describe("PDF viewer", () => {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
it("respects drawing delay when zooming out", async () => {
|
describe("forced (maxCanvasPixels: 0)", () => {
|
||||||
await Promise.all(
|
let pages;
|
||||||
pages.map(async ([browserName, page]) => {
|
|
||||||
const promise = await createPromiseForFirstPageRendered(page);
|
|
||||||
|
|
||||||
const start = await page.evaluate(() => {
|
beforeAll(async () => {
|
||||||
const startTime = performance.now();
|
pages = await loadAndWait(
|
||||||
window.PDFViewerApplication.pdfViewer.decreaseScale({
|
"tracemonkey.pdf",
|
||||||
drawingDelay: 100,
|
".textLayer .endOfContent",
|
||||||
scaleFactor: 0.9,
|
null,
|
||||||
|
null,
|
||||||
|
{ maxCanvasPixels: 0 }
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
afterAll(async () => {
|
||||||
|
await closePages(pages);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("respects drawing delay when zooming out", async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
const promise = await createPromiseForFirstPageRendered(page);
|
||||||
|
|
||||||
|
const start = await page.evaluate(() => {
|
||||||
|
const startTime = performance.now();
|
||||||
|
window.PDFViewerApplication.pdfViewer.decreaseScale({
|
||||||
|
drawingDelay: 100,
|
||||||
|
scaleFactor: 0.9,
|
||||||
|
});
|
||||||
|
return startTime;
|
||||||
});
|
});
|
||||||
return startTime;
|
|
||||||
});
|
|
||||||
|
|
||||||
const end = await awaitPromise(promise);
|
const end = await awaitPromise(promise);
|
||||||
|
|
||||||
expect(end - start)
|
expect(end - start)
|
||||||
.withContext(`In ${browserName}`)
|
.withContext(`In ${browserName}`)
|
||||||
.toBeGreaterThanOrEqual(100);
|
.toBeGreaterThanOrEqual(100);
|
||||||
})
|
})
|
||||||
);
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("respects drawing delay when zooming in", async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
const promise = await createPromiseForFirstPageRendered(page);
|
||||||
|
|
||||||
|
const start = await page.evaluate(() => {
|
||||||
|
const startTime = performance.now();
|
||||||
|
window.PDFViewerApplication.pdfViewer.increaseScale({
|
||||||
|
drawingDelay: 100,
|
||||||
|
scaleFactor: 1.1,
|
||||||
|
});
|
||||||
|
return startTime;
|
||||||
|
});
|
||||||
|
|
||||||
|
const end = await awaitPromise(promise);
|
||||||
|
|
||||||
|
expect(end - start)
|
||||||
|
.withContext(`In ${browserName}`)
|
||||||
|
.toBeGreaterThanOrEqual(100);
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
it("respects drawing delay when zooming in", async () => {
|
describe("triggers when going bigger than maxCanvasPixels", () => {
|
||||||
await Promise.all(
|
let pages;
|
||||||
pages.map(async ([browserName, page]) => {
|
|
||||||
const promise = await createPromiseForFirstPageRendered(page);
|
|
||||||
|
|
||||||
const start = await page.evaluate(() => {
|
const MAX_CANVAS_PIXELS = new Map();
|
||||||
const startTime = performance.now();
|
|
||||||
window.PDFViewerApplication.pdfViewer.increaseScale({
|
beforeAll(async () => {
|
||||||
drawingDelay: 100,
|
pages = await loadAndWait(
|
||||||
scaleFactor: 1.1,
|
"tracemonkey.pdf",
|
||||||
|
".textLayer .endOfContent",
|
||||||
|
null,
|
||||||
|
null,
|
||||||
|
async (page, browserName) => {
|
||||||
|
const ratio = await page.evaluate(() => window.devicePixelRatio);
|
||||||
|
const maxCanvasPixels = 1_000_000 * ratio ** 2;
|
||||||
|
MAX_CANVAS_PIXELS.set(browserName, maxCanvasPixels);
|
||||||
|
|
||||||
|
return { maxCanvasPixels };
|
||||||
|
}
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
await page.evaluate(() => {
|
||||||
|
window.PDFViewerApplication.pdfViewer.currentScale = 0.5;
|
||||||
});
|
});
|
||||||
return startTime;
|
})
|
||||||
});
|
);
|
||||||
|
});
|
||||||
|
|
||||||
const end = await awaitPromise(promise);
|
afterAll(async () => {
|
||||||
|
await closePages(pages);
|
||||||
|
});
|
||||||
|
|
||||||
expect(end - start)
|
function getCanvasSize(page) {
|
||||||
.withContext(`In ${browserName}`)
|
return page.evaluate(() => {
|
||||||
.toBeGreaterThanOrEqual(100);
|
const canvas = window.document.querySelector(".canvasWrapper canvas");
|
||||||
})
|
return canvas.width * canvas.height;
|
||||||
);
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
// MAX_CANVAS_PIXELS must be big enough that the originally rendered
|
||||||
|
// canvas still has enough space to grow before triggering CSS-only zoom
|
||||||
|
it("test correctly configured", async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
const canvasSize = await getCanvasSize(page);
|
||||||
|
|
||||||
|
expect(canvasSize)
|
||||||
|
.withContext(`In ${browserName}`)
|
||||||
|
.toBeLessThan(MAX_CANVAS_PIXELS.get(browserName) / 4);
|
||||||
|
expect(canvasSize)
|
||||||
|
.withContext(`In ${browserName}`)
|
||||||
|
.toBeGreaterThan(MAX_CANVAS_PIXELS.get(browserName) / 16);
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("does not trigger CSS-only zoom below maxCanvasPixels", async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
const originalCanvasSize = await getCanvasSize(page);
|
||||||
|
const factor = 2;
|
||||||
|
|
||||||
|
await page.evaluate(scaleFactor => {
|
||||||
|
window.PDFViewerApplication.pdfViewer.increaseScale({
|
||||||
|
drawingDelay: 0,
|
||||||
|
scaleFactor,
|
||||||
|
});
|
||||||
|
}, factor);
|
||||||
|
|
||||||
|
const canvasSize = await getCanvasSize(page);
|
||||||
|
|
||||||
|
expect(canvasSize)
|
||||||
|
.withContext(`In ${browserName}`)
|
||||||
|
.toBe(originalCanvasSize * factor ** 2);
|
||||||
|
|
||||||
|
expect(canvasSize)
|
||||||
|
.withContext(`In ${browserName}, MAX_CANVAS_PIXELS`)
|
||||||
|
.toBeLessThan(MAX_CANVAS_PIXELS.get(browserName));
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("triggers CSS-only zoom above maxCanvasPixels", async () => {
|
||||||
|
await Promise.all(
|
||||||
|
pages.map(async ([browserName, page]) => {
|
||||||
|
const originalCanvasSize = await getCanvasSize(page);
|
||||||
|
const factor = 4;
|
||||||
|
|
||||||
|
await page.evaluate(scaleFactor => {
|
||||||
|
window.PDFViewerApplication.pdfViewer.increaseScale({
|
||||||
|
drawingDelay: 0,
|
||||||
|
scaleFactor,
|
||||||
|
});
|
||||||
|
}, factor);
|
||||||
|
|
||||||
|
const canvasSize = await getCanvasSize(page);
|
||||||
|
|
||||||
|
expect(canvasSize)
|
||||||
|
.withContext(`In ${browserName}`)
|
||||||
|
.toBeLessThan(originalCanvasSize * factor ** 2);
|
||||||
|
|
||||||
|
// Disabled because `canvasSize` is `4_012_800`, which is
|
||||||
|
// close to the limit but somehow a bit more.
|
||||||
|
//
|
||||||
|
// expect(canvasSize)
|
||||||
|
// .withContext(`In ${browserName}, MAX_CANVAS_PIXELS`)
|
||||||
|
// .toBeLessThan(MAX_CANVAS_PIXELS.get(browserName));
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user