Merge pull request #20060 from noritaka1166/fix-typos

timvandermeij · web-flow · commit b999d5ce58a2 · 2025-07-08T18:28:52.000+02:00
Fix typos across the codebase
diff --git a/extensions/chromium/pdfHandler.js b/extensions/chromium/pdfHandler.js
@@ -69,7 +69,7 @@ async function registerPdfRedirectRule() {
     },
   };
 
-  // Rules in order of prority (highest priority rule first).
+  // Rules in order of priority (highest priority rule first).
   // The required "id" fields will be auto-generated later.
   const addRules = [
     {
diff --git a/src/core/annotation.js b/src/core/annotation.js
@@ -2391,7 +2391,7 @@ class WidgetAnnotation extends Annotation {
 
     if (encodingError && intent & RenderingIntentFlag.SAVE) {
       // We don't have a way to render the field, so we just rely on the
-      // /NeedAppearances trick to let the different sofware correctly render
+      // /NeedAppearances trick to let the different software correctly render
       // this pdf.
       return { needAppearances: true };
     }
diff --git a/src/core/evaluator.js b/src/core/evaluator.js
@@ -2745,7 +2745,7 @@ class PartialEvaluator {
           // This is not a 0, 90, 180, 270 rotation so:
           //  - remove the scale factor from the matrix to get a rotation matrix
           //  - apply the inverse (which is the transposed) to the positions
-          // and we can then compare positions of the glyphes to detect
+          // and we can then compare positions of the glyphs to detect
           // a whitespace.
           [posX, posY] = applyInverseRotation(posX, posY, currentTransform);
           [lastPosX, lastPosY] = applyInverseRotation(
@@ -3823,7 +3823,7 @@ class PartialEvaluator {
     // According to the spec if the font is a simple font we should only map
     // to unicode if the base encoding is MacRoman, MacExpert, or WinAnsi or
     // the differences array only contains adobe standard or symbol set names,
-    // in pratice it seems better to always try to create a toUnicode map
+    // in practice it seems better to always try to create a toUnicode map
     // based of the default encoding.
     if (!properties.composite /* is simple font */) {
       return new ToUnicodeMap(this._simpleFontToUnicode(properties));
diff --git a/src/core/fonts.js b/src/core/fonts.js
@@ -2506,7 +2506,7 @@ class Font {
             }
           }
         }
-        // Adjusting stack not extactly, but just enough to get function id
+        // Adjusting stack not exactly, but just enough to get function id
         if (!inFDEF && !inELSE) {
           let stackDelta = 0;
           if (op <= 0x8e) {
diff --git a/src/core/image.js b/src/core/image.js
@@ -237,8 +237,8 @@ class PDFImage {
         this.jpxDecoderOptions.numComponents = hasColorSpace
           ? this.numComps
           : 0;
-        // If the jpx image has a color space then it musn't be used in order to
-        // be able to use the color space that comes from the pdf.
+        // If the jpx image has a color space then it mustn't be used in order
+        // to be able to use the color space that comes from the pdf.
         this.jpxDecoderOptions.isIndexedColormap =
           this.colorSpace.name === "Indexed";
       }
diff --git a/src/core/image_resizer.js b/src/core/image_resizer.js
@@ -83,7 +83,7 @@ class ImageResizer {
 
     // TODO: the computation can be a bit long because we potentially allocate
     // some large canvas, so in the Firefox case this value (and MAX_DIM) can be
-    // infered from prefs (MAX_AREA = gfx.max-alloc-size / 4, 4 is because of
+    // inferred from prefs (MAX_AREA = gfx.max-alloc-size / 4, 4 is because of
     // RGBA).
     this.#goodSquareLength = this._guessMax(
       this.#goodSquareLength,
diff --git a/src/core/xfa/formcalc_parser.js b/src/core/xfa/formcalc_parser.js
@@ -241,7 +241,7 @@ const OPERAND = false;
 // operation ('... * - ...' can't be a subtraction).
 // Each time an operator is met its precedence is compared with the one of the
 // operator on top of operators stack:
-//  - if top has precendence on operator then top is applied to the operands
+//  - if top has precedence on operator then top is applied to the operands
 //    on their stack;
 //  - else just push the operator.
 // For example: 1 + 2 * 3
diff --git a/src/display/canvas.js b/src/display/canvas.js
@@ -1027,7 +1027,7 @@ class CanvasGraphics {
     );
     const fillCtx = fillCanvas.context;
 
-    // The offset will be the top-left cordinate mask.
+    // The offset will be the top-left coordinate mask.
     // If objToCanvas is [a,b,c,d,e,f] then:
     //   - offsetX = min(a, c) + e
     //   - offsetY = min(b, d) + f
diff --git a/src/display/editor/annotation_editor_layer.js b/src/display/editor/annotation_editor_layer.js
@@ -530,7 +530,7 @@ class AnnotationEditorLayer {
 
   /**
    * An editor can have a different parent, for example after having
-   * being dragged and droped from a page to another.
+   * being dragged and dropped from a page to another.
    * @param {AnnotationEditor} editor
    */
   changeParent(editor) {
diff --git a/src/shared/util.js b/src/shared/util.js
@@ -1185,7 +1185,7 @@ function _isValidExplicitDest(validRef, validName, dest) {
   return true;
 }
 
-// TOOD: Replace all occurrences of this function with `Math.clamp` once
+// TODO: Replace all occurrences of this function with `Math.clamp` once
 //       https://github.com/tc39/proposal-math-clamp/ is generally available.
 function MathClamp(v, min, max) {
   return Math.min(Math.max(v, min), max);
diff --git a/test/driver.js b/test/driver.js
@@ -377,7 +377,7 @@ class Rasterize {
           height: height / pageHeight,
         });
       }
-      // We set the borderWidth to 0.001 to slighly increase the size of the
+      // We set the borderWidth to 0.001 to slightly increase the size of the
       // boxes so that they can be merged together.
       const outliner = new HighlightOutliner(boxes, /* borderWidth = */ 0.001);
       // We set the borderWidth to 0.0025 in order to have an outline which is
diff --git a/test/integration/stamp_editor_spec.mjs b/test/integration/stamp_editor_spec.mjs
@@ -1367,7 +1367,7 @@ describe("Stamp Editor", () => {
     });
   });
 
-  describe("A stamp musn't be on top of the secondary toolbar", () => {
+  describe("A stamp mustn't be on top of the secondary toolbar", () => {
     let pages;
 
     beforeEach(async () => {
diff --git a/web/event_utils.js b/web/event_utils.js
@@ -170,7 +170,7 @@ class EventBus {
 }
 
 /**
- * NOTE: Only used in the Firefox build-in pdf viewer.
+ * NOTE: Only used in the Firefox built-in pdf viewer.
  */
 class FirefoxEventBus extends EventBus {
   #externalServices;
diff --git a/web/pdf_find_controller.js b/web/pdf_find_controller.js
@@ -722,7 +722,7 @@ class PDFFindController {
         // kind of whitespaces are replaced by a single " ".
 
         if (p1) {
-          // Escape characters like *+?... to not interfer with regexp syntax.
+          // Escape characters like *+?... to not interfere with regexp syntax.
           return `[ ]*\\${p1}[ ]*`;
         }
         if (p2) {

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ async function registerPdfRedirectRule() {`
`69`	`69`	`},`
`70`	`70`	`};`
`71`	`71`
`72`		`- // Rules in order of prority (highest priority rule first).`
	`72`	`+ // Rules in order of priority (highest priority rule first).`
`73`	`73`	`// The required "id" fields will be auto-generated later.`
`74`	`74`	`const addRules = [`
`75`	`75`	`{`
Original file line number	Diff line number	Diff line change
`@@ -2391,7 +2391,7 @@ class WidgetAnnotation extends Annotation {`
`2391`	`2391`
`2392`	`2392`	`if (encodingError && intent & RenderingIntentFlag.SAVE) {`
`2393`	`2393`	`// We don't have a way to render the field, so we just rely on the`
`2394`		`- // /NeedAppearances trick to let the different sofware correctly render`
	`2394`	`+ // /NeedAppearances trick to let the different software correctly render`
`2395`	`2395`	`// this pdf.`
`2396`	`2396`	`return { needAppearances: true };`
`2397`	`2397`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2506,7 +2506,7 @@ class Font {`
`2506`	`2506`	`}`
`2507`	`2507`	`}`
`2508`	`2508`	`}`
`2509`		`- // Adjusting stack not extactly, but just enough to get function id`
	`2509`	`+ // Adjusting stack not exactly, but just enough to get function id`
`2510`	`2510`	`if (!inFDEF && !inELSE) {`
`2511`	`2511`	`let stackDelta = 0;`
`2512`	`2512`	`if (op <= 0x8e) {`
Original file line number	Diff line number	Diff line change
`@@ -1185,7 +1185,7 @@ function _isValidExplicitDest(validRef, validName, dest) {`
`1185`	`1185`	`return true;`
`1186`	`1186`	`}`
`1187`	`1187`
`1188`		-// TOOD: Replace all occurrences of this function with `Math.clamp` once
	`1188`	+// TODO: Replace all occurrences of this function with `Math.clamp` once
`1189`	`1189`	`// https://github.com/tc39/proposal-math-clamp/ is generally available.`
`1190`	`1190`	`function MathClamp(v, min, max) {`
`1191`	`1191`	`return Math.min(Math.max(v, min), max);`