Revert "Merge pull request matrix-org#1801 from matrix-org/gsouquet/localecompare-perf"

This reverts commit e85038e, reversing
changes made to a878588.

Author: bradjones1

src/models/room.ts

@@ -2363,7 +2363,7 @@ export class Room extends EventEmitter {
                 return true;
             });
             // make sure members have stable order
-            otherMembers.sort((a, b) => utils.compare(a.userId, b.userId));
+            otherMembers.sort((a, b) => a.userId.localeCompare(b.userId));
             // only 5 first members, immitate summaryHeroes
             otherMembers = otherMembers.slice(0, 5);
             otherNames = otherMembers.map((m) => m.name);

src/utils.ts

@@ -677,16 +677,6 @@ export function lexicographicCompare(a: string, b: string): number {
     return (a < b) ? -1 : ((a === b) ? 0 : 1);
 }
 
-const collator = new Intl.Collator();
-/**
- * Performant language-sensitive string comparison
- * @param a the first string to compare
- * @param b the second string to compare
- */
-export function compare(a: string, b: string): number {
-    return collator.compare(a, b);
-}
-
 /**
  * This function is similar to Object.assign() but it assigns recursively and
  * allows you to ignore nullish values from the source