Bug 1352343 - Rename LocaleService::On* methods to LocaleService::*. r=jfkthame draft
authorZibi Braniecki <zbraniecki@mozilla.com>
Tue, 10 Oct 2017 10:39:12 -0700
changeset 678625 870688e801beb0b6181a37cd86da61bd48650df4
parent 678624 30df702016a6266e23cceb14c843feb6c8ccc1ce
child 735383 b003a90897dae57ff2c1c99042f65b48a0224031
push id83984
push userbmo:gandalf@aviary.pl
push dateWed, 11 Oct 2017 17:32:13 +0000
reviewersjfkthame
bugs1352343
milestone58.0a1
Bug 1352343 - Rename LocaleService::On* methods to LocaleService::*. r=jfkthame MozReview-Commit-ID: LvEW9GcOodb
chrome/nsChromeRegistryChrome.cpp
intl/locale/LocaleService.h
--- a/chrome/nsChromeRegistryChrome.cpp
+++ b/chrome/nsChromeRegistryChrome.cpp
@@ -734,17 +734,17 @@ nsChromeRegistryChrome::ManifestLocale(M
   nsresult rv = OverrideLocalePackage(NS_LITERAL_CSTRING("global"), mainPackage);
   if (NS_FAILED(rv)) {
     return;
   }
 
   if (mainPackage.Equals(package)) {
     // We should refresh the LocaleService, since the available
     // locales changed.
-    LocaleService::GetInstance()->OnAvailableLocalesChanged();
+    LocaleService::GetInstance()->AvailableLocalesChanged();
   }
 }
 
 void
 nsChromeRegistryChrome::ManifestSkin(ManifestProcessingContext& cx, int lineno,
                                      char *const * argv, int flags)
 {
   char* package = argv[0];
--- a/intl/locale/LocaleService.h
+++ b/intl/locale/LocaleService.h
@@ -208,19 +208,19 @@ public:
    * LocaleService itself, but in a couple special cases, we have the
    * other component call this manually instead of sending a global event.
    *
    * If the result differs from the previous list, it will additionally
    * trigger a corresponding event
    *
    * This code should be called only in the server mode..
    */
-  void OnAvailableLocalesChanged();
-  void OnRequestedLocalesChanged();
-  void OnLocalesChanged();
+  void AvailableLocalesChanged();
+  void RequestedLocalesChanged();
+  void LocalesChanged();
 
   /**
    * Negotiates the best locales out of an ordered list of requested locales and
    * a list of available locales.
    *
    * Internally it uses the following naming scheme:
    *
    *  Requested - locales requested by the user