Merge pull request #21491 from Snuffleupagus/readCmapTable-sort-last

Sort the mappings *last* in the `readCmapTable` function (PR 19321 follow-up)

src/core/fonts.js

@@ -1860,7 +1860,6 @@ class Font {
       }
 
       // removing duplicate entries
-      mappings.sort((a, b) => a.charCode - b.charCode);
       const finalMappings = [],
         seenCharCodes = new Set();
       for (const map of mappings) {
@@ -1876,7 +1875,7 @@ class Font {
       return {
         platformId: potentialTable.platformId,
         encodingId: potentialTable.encodingId,
-        mappings: finalMappings,
+        mappings: finalMappings.sort((a, b) => a.charCode - b.charCode),
         hasShortCmap,
       };
     }

src/display/editor/tools.js

@@ -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(),
       });