Merge pull request #21489 from Snuffleupagus/editor-tools-rm-testing-reset

Remove the `AnnotationEditorUIManager` and `IdManager` test-only `reset` methods (PR 19809 follow-up)
This commit is contained in:
Jonas Jenwald 2026-06-23 14:03:46 +02:00 committed by GitHub
commit a1acf4fc9f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -134,9 +134,6 @@ class IdManager {
constructor() {
if (typeof PDFJSDev !== "undefined" && PDFJSDev.test("TESTING")) {
Object.defineProperty(this, "reset", {
value: () => (this.#id = 0),
});
Object.defineProperty(this, "getNextId", {
value: () => this.#id,
});
@ -1067,13 +1064,6 @@ class AnnotationEditorUIManager {
commentManager?.setSidebarUiManager(this);
if (typeof PDFJSDev !== "undefined" && PDFJSDev.test("TESTING")) {
Object.defineProperty(this, "reset", {
value: () => {
this.selectAll();
this.delete();
this.#idManager.reset();
},
});
Object.defineProperty(this, "getNextEditorId", {
value: () => this.#idManager.getNextId(),
});