Bug 1344901 - Fix a regression introduced in bug 1337551. r?jfkthame draft
authorZibi Braniecki <gandalf@mozilla.com>
Mon, 06 Mar 2017 13:54:30 -0800
changeset 494233 ddd6c5808209a274b54111e9d37b828f1704742d
parent 494079 517c553ad64746c479456653ce11b04ab8e4977f
child 548052 bfd330404b38f869bcd6afe430a0f35e9238bd78
push id47981
push userzbraniecki@mozilla.com
push dateMon, 06 Mar 2017 21:55:17 +0000
reviewersjfkthame
bugs1344901, 1337551
milestone54.0a1
Bug 1344901 - Fix a regression introduced in bug 1337551. r?jfkthame MozReview-Commit-ID: 8SP0ucROJRF
browser/modules/DirectoryLinksProvider.jsm
toolkit/components/telemetry/TelemetryEnvironment.jsm
toolkit/modules/Locale.jsm
toolkit/mozapps/extensions/AddonManager.jsm
--- a/browser/modules/DirectoryLinksProvider.jsm
+++ b/browser/modules/DirectoryLinksProvider.jsm
@@ -190,17 +190,17 @@ var DirectoryLinksProvider = {
   get locale() {
     let matchOS;
     try {
       matchOS = Services.prefs.getBoolPref(PREF_MATCH_OS_LOCALE);
     } catch (e) {}
 
     if (matchOS) {
       return Cc["@mozilla.org/intl/ospreferences;1"].
-             getService(Ci.mozIOSPreferences).getSystemLocale();
+             getService(Ci.mozIOSPreferences).systemLocale;
     }
 
     try {
       let locale = Services.prefs.getComplexValue(PREF_SELECTED_LOCALE,
                                                   Ci.nsIPrefLocalizedString);
       if (locale) {
         return locale.data;
       }
--- a/toolkit/components/telemetry/TelemetryEnvironment.jsm
+++ b/toolkit/components/telemetry/TelemetryEnvironment.jsm
@@ -239,17 +239,17 @@ function getBrowserLocale() {
 /**
  * Get the current OS locale.
  * @return a string with the OS locale or null on failure.
  */
 function getSystemLocale() {
   try {
     return Cc["@mozilla.org/intl/ospreferences;1"].
              getService(Ci.mozIOSPreferences).
-             getSystemLocale();
+             systemLocale;
   } catch (e) {
     return null;
   }
 }
 
 /**
  * Safely get a sysinfo property and return its value. If the property is not
  * available, return aDefault.
--- a/toolkit/modules/Locale.jsm
+++ b/toolkit/modules/Locale.jsm
@@ -16,17 +16,17 @@ this.Locale = {
   /**
    * Gets the currently selected locale for display.
    * @return  the selected locale or "en-US" if none is selected
    */
   getLocale() {
     if (Preferences.get(PREF_MATCH_OS_LOCALE, false)) {
       const osPrefs =
         Cc["@mozilla.org/intl/ospreferences;1"].getService(Ci.mozIOSPreferences);
-      return osPrefs.getSystemLocale();
+      return osPrefs.systemLocale;
     }
     try {
       let locale = Preferences.get(PREF_SELECTED_LOCALE, null, Ci.nsIPrefLocalizedString);
       if (locale)
         return locale;
     } catch (e) {}
     return Preferences.get(PREF_SELECTED_LOCALE, "en-US");
   },
--- a/toolkit/mozapps/extensions/AddonManager.jsm
+++ b/toolkit/mozapps/extensions/AddonManager.jsm
@@ -327,17 +327,17 @@ function promiseCallProvider(aProvider, 
  * Gets the currently selected locale for display.
  * @return  the selected locale or "en-US" if none is selected
  */
 function getLocale() {
   try {
     if (Services.prefs.getBoolPref(PREF_MATCH_OS_LOCALE)) {
       const osPrefs =
         Cc["@mozilla.org/intl/ospreferences;1"].getService(Ci.mozIOSPreferences);
-      return osPrefs.getSystemLocale();
+      return osPrefs.systemLocale;
     }
   } catch (e) { }
 
   try {
     let locale = Services.prefs.getComplexValue(PREF_SELECTED_LOCALE,
                                                 Ci.nsIPrefLocalizedString);
     if (locale)
       return locale;