1350102 - Remove nsILocaleService

Status: RESOLVED DUPLICATE of bug 1356263

(Core :: Internationalization, enhancement, P3)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

We're getting close to be ready to deprecate nsILocaleService as all of its functionality has been migrated to mozILocaleService.

This is a tracker for remaining items.

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

The remaining occurrences:

1) addon-sdk/source/lib/sdk/l10n/locale.js - fix in bug 1334772
2) browser/base/content/utilityOverlay.js - fix in bug 1347272
3) chrome/nsChromeRegistryChrome.cpp - fix in bug 1347306
4) dom/media/webspeech/synth/nsSynthVoiceRegistry.cpp - no fix yet, will fix with removal (with LocaleService::GetAppLocale)
5) chrome/test/unit/test_bug519468.js - fix in bug 1347306

So, we have 4 out of 5 already fixed in patches that are in the review process.

Comment 2

[Makoto Kato [:m_kato]]

Since a lot of addons still use nsILocaleService, so we cannot remove this until 57.

https://dxr.mozilla.org/addons/search?q=nsILocaleService&redirect=false

From 57, we supports WebExtension API only on release build, so we can remove this.  Also, after comment #1's all code is replaced, we should add deprecated attribute on IDL before 57.