Reduce the number of locales for iOS.

Reduce the number of locales for iOS.

iOS only supports a limited amount of languages ( https://sites.google.com/a/google.com/mobile-effort/ios/i18n-and-l10n ). We must remove the unused languages to reduce the app's size.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=188336

[tony, 2013-03-11 17:55:14]

https://codereview.chromium.org/12686003/diff/3001/build/common.gypi
File build/common.gypi (left):

https://codereview.chromium.org/12686003/diff/3001/build/common.gypi#oldcode938
build/common.gypi:938: # NOTE: When these end up in the Mac bundle, we need to
replace '-' for '_'
Can we keep this list here and in the ios case, override locales using
'locales=' as the key name?  See
https://code.google.com/p/gyp/wiki/InputFormatReference#Merge_Basics_(=,_?,_+)
about replacing the destination list.

[jeanfrancoisg, 2013-03-13 13:22:01]

You are right. I can't seem to make it work though. The iOS locales list is
merged even if I put 'locales='.
doing:
http://p.prom.corp.google.com/2561062
yields on iOS (notice the 'pt' at the end of the list):
am ar bg bn ca cs da de el en-GB en-US es-419 es et fa fi fil fr gu he hi hr hu
id it ja kn ko lt lv ml mr ms nb nl pl pt-BR pt-PT ro ru sk sl sr sv sw ta te th
tr uk vi zh-CN zh-TW pt

Do you know how to fix this?

[tony, 2013-03-13 20:55:51]

Hmm, I can't seem to get it to work.  I guess your original patch LGTM.

Mark or Brad, do you know why we can't use locales= to override the locales list
for iOS?

[Mark Mentovai, 2013-03-13 21:05:36]

Depending on where the two 'locales' keys are relative to each other, there may
be more than one list merge operation. You might need something like 'locales='
or 'locales=='.

If GYP winds up merging the big list into the iOS list, then locales= on the iOS
list wouldn’t work at all, but you could specify locales? on the big list.

[tony, 2013-03-13 21:14:23]

On 2013/03/13 21:05:36, Mark Mentovai wrote:
> Depending on where the two 'locales' keys are relative to each other, there
may
> be more than one list merge operation. You might need something like
'locales='
> or 'locales=='.

Interesting.  If I use 'locales=' for the main list and 'locales==' in the
conditional, it seems to work for me.

[jeanfrancoisg, 2013-03-14 10:12:09]

Thanks a lot!
I ended up with 'locales' for the main list and 'locales==' in the conditional.
It minimizes the changes.
PTAL.

[tony, 2013-03-14 17:00:29]

LGTM, thanks!

[commit-bot: I haz the power, 2013-03-14 17:31:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jeanfrancoisg@google.com/12686003/13001

[commit-bot: I haz the power, 2013-03-15 11:15:45]

Change committed as 188336