mirrors/pdf.js
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2025-04-19 14:48:08 +02:00
Code Activity

Move the reporting of page Stats into the API

Browse source 
This avoids having to add a couple of event listeners in the viewer, when debugging is enabled, and is consistent with the existing handling of `FontInspector` and `StepperManager` in the API.
This commit is contained in:
Jonas Jenwald 2024-05-10 12:13:14 +02:00
parent b6765403a1
commit 4a8d742592
2 changed files with 8 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/display/api.js
View file

@ -1510,8 +1510,14 @@ class PDFPageProxy {
internalRenderTask.capability.resolve();
}
this._stats?.timeEnd("Rendering");
this._stats?.timeEnd("Overall");
if (this._stats) {
this._stats.timeEnd("Rendering");
this._stats.timeEnd("Overall");
if (globalThis.Stats?.enabled) {
globalThis.Stats.add(this.pageNumber, this._stats);
}
}
};
const internalRenderTask = new InternalRenderTask({

14
web/app.js
View file

@ -1898,10 +1898,6 @@ const PDFViewerApplication = {
signal,
});
if (AppOptions.get("pdfBug")) {
eventBus._on("pagerendered", reportPageStatsPDFBug, { signal });
eventBus._on("pagechanging", reportPageStatsPDFBug, { signal });
}
if (typeof PDFJSDev === "undefined" || PDFJSDev.test("GENERIC")) {
eventBus._on("fileinputchange", webViewerFileInputChange, { signal });
eventBus._on("openfile", webViewerOpenFile, { signal });
@ -2187,16 +2183,6 @@ async function loadPDFBug(self) {
self._PDFBug = PDFBug;
}
function reportPageStatsPDFBug({ pageNumber }) {
if (!globalThis.Stats?.enabled) {
return;
}
const pageView = PDFViewerApplication.pdfViewer.getPageView(
/* index = */ pageNumber - 1
);
globalThis.Stats.add(pageNumber, pageView?.pdfPage?.stats);
}
function webViewerPageRender({ pageNumber }) {
// If the page is (the most) visible when it starts rendering,
// ensure that the page number input loading indicator is displayed.