Convert region code to uppercase when generating translation file URL on Safari

safari/ext/common.js

Index: safari/ext/common.js
===================================================================
--- a/safari/ext/common.js
+++ b/safari/ext/common.js
@@ -85,17 +85,16 @@
     var candidates = [];
     var defaultLocale = "en_US";
 
-    var bits, i;
-    for (i = (bits = navigator.language.split("-")).length; i > 0; i--)
-    {
-      var locale = bits.slice(0, i).join("_");
-      candidates.push(locale);
+    var match = navigator.language.match(/(.*?)(?:-(.*))?$/);
+    var language = match[1];
+    var region = match[2];

[Wladimir Palant, 2014/04/10 08:05:34]

It seems that you are still trying to split the string, the following should be
more obvious:

var bits = navigator.language.split("-", 2);
var language = bits[0];
var region = bits.length > 1 ? bits[1] : null;

[Sebastian Noack, 2014/04/10 08:55:52]

Done, but I used ES6 array unpacking for better readability.

 
-      if (locale == defaultLocale)
-        return candidates;
-    }
+    if (region)
+      candidates.push(language + "_" + region.toUpperCase());
 
-    candidates.push(defaultLocale);
+    candidates.push(language, defaultLocale);
+    candidates.splice(candidates.indexOf(defaultLocale) + 1);

[Wladimir Palant, 2014/04/10 08:05:34]

I had some trouble understanding the purpose of this. So if navigator.language
is "en-us" we will return ["en_US"] rather than ["en_US", "en"]? That's
inconsistent (and - yes, it looks like the original code got it wrong as well).

I'd suggest not trying to be clever, the following code will produce consistent
results and also happens to be obvious:

if (candidates[0] != defaultLocale)
  candidates.push(defaultLocale);

[Sebastian Noack, 2014/04/10 08:55:52]

Done, but I used candidates.indexOf(defaultLocale) instead. So the code desn't
assume that defaultLocale includes a region suffix.

+
     return candidates;
   };