Issue 3697 - Fall back to i18n.getUILanguage if @ui_locale isn't supported

lib/utils.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 26 matching lines @@
       };
       document.addEventListener("DOMContentLoaded", onDOMContentLoaded);
     }
     else
     {
       setTimeout(callback, 0);
     }
   },
   get appLocale()
   {
-    var locale = ext.i18n.getMessage("@@ui_locale").replace(/_/g, "-");
+    var locale;
+    try
+    {
+      locale = ext.i18n.getMessage("@@ui_locale");
+    }
+    catch (exception)
+    {
+      // Edge does not yet (?) support @@ui_locale, throwing an exception here.
+    }
+    if (!locale)
+      locale = ext.i18n.getUILanguage();

[Sebastian Noack, 2017/08/23 12:26:44]

It seems we can use chrome.i18n.getUILanguage() regardless, without checking for
getMessage("@@ui_locale") first, since all supported browsers and browser
versions seem to support it.

+    locale = locale.replace(/_/g, "-");
     Object.defineProperty(this, "appLocale", {value: locale, enumerable: true});
     return this.appLocale;
   },
+  get readingDirection()
+  {
+    var direction;
+    try
+    {
+      direction = ext.i18n.getMessage("@@bidi_dir");
+    }
+    catch (exception)
+    {
+      // Edge does not yet (?) support @@bidi_dir, throwing an exception here.
+    }
+    if (!direction)
+      direction = /^(ar|fa|he|ug|ur)(_|$)/.test(this.appLocale) ? "rtl" : "ltr";
+    Object.defineProperty(this, "readingDirection", {value: direction, enumerable: true});
+    return this.readingDirection;
+  },
   generateChecksum: function(lines)
   {
     // We cannot calculate MD5 checksums yet :-(
     return null;
   },
   checkLocalePrefixMatch: function(prefixes)
   {
     if (!prefixes)
       return null;
 
@@ skipping 48 matching lines @@
   {
     var Prefs = require("prefs").Prefs;
     var docLink = Prefs.documentation_link;
     return docLink.replace(/%LINK%/g, linkID).replace(/%LANG%/g, Utils.appLocale);
   },
 
   yield: function()
   {
   }
 };