Bug 1946467 - Treat multiple commas/periods as thousands separators not as decimal points. r=daleharvey

Differential Revision: https://phabricator.services.mozilla.com/D237629

Author: yazanmacki

browser/components/urlbar/UrlbarProviderCalculator.sys.mjs

@@ -504,8 +504,14 @@ Calculator.addNumberSystem({
       // Contains both a period and a comma and the comma came first
       // so strip the comma (ie 1,999.5).
       num = num.replace(/,/g, "");
+    } else if (firstComma != -1 && num.includes(",", firstComma + 1)) {
+      // Contains multiple commas and no periods, strip commas
+      num = num.replace(/,/g, "");
+    } else if (firstPeriod != -1 && num.includes(".", firstPeriod + 1)) {
+      // Contains multiple periods and no commas, strip periods
+      num = num.replace(/\./g, "");
     } else if (firstComma != -1) {
-      // Has commas but no periods so treat comma as decimal seperator
+      // Has a single comma and no periods, treat comma as decimal seperator
       num = num.replace(/,/g, ".");
     }
     return num;

browser/components/urlbar/tests/unit/test_calculator.js

@@ -39,6 +39,8 @@ const FORMULAS = [
   ["1/0", "undefined"],
   ["1/3", 0.3333333333333333],
   ["1/(3*10^10)", "3.33333333e-11"],
+  ["1,000,000+500", 1000500],
+  ["1.000.000+500", 1000500],
 ];
 
 add_task(function test() {