483581 - Revert changes to region.properties on l10n-central and l10n-mozilla-1.9.1

Status: RESOLVED FIXED

Description

[Staś Małolepszy :stas]

I noticed two pushes to l10n-central and l10n-mozilla-1.9.1 that landed some changes to region.properties without a review. Please revert them:

http://hg.mozilla.org/l10n-central/vi/diff/a1ed9e0752bc/browser/chrome/browser-region/region.properties

We don't need this as we're not shipping Yahoo in Vietnamese builds.

http://hg.mozilla.org/l10n-central/vi/diff/59670faf0850/browser/chrome/browser-region/region.properties

Mibbit will be mass-added in bug 479030.

Please note: please don't land changes to region.properties or searchplugins/ without a review from the l10n-drivers team (even on the dev branch). These are sensitive pieces of localization that we watch over to make sure that the default settings are in line with the decisions taken with the localizers in the productization bugs.

Thanks.

Comment 1

[Hùng. NGUYỄN Mạnh]

My mistake. I sync our vi repository with our translations on Narro which in turn use the layouts of the en-US files. Next times, I'll remember to delete this file from the exported archive. (Btw, I'm still curious how sensitive that would affect, because uhm.. the Yahoo plugin is already not shipped in Vietnamese builds.)

I have reverted the changes:
http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/vi/rev/df72e8b54d60
http://hg.mozilla.org/l10n-central/vi/rev/1f54cf65d974

Comment 2

[Staś Małolepszy :stas]

Thanks Hùng! Marking as fixed. 

Regarding your question: as we don't ship Yahoo, there's no need to specify the second search engine in region.properties. Alpha ordering is enough in this case. The setting was obsolete, so we could safely remove it.