[Android WebView] Add rules to bundle localised string paks.

[Android WebView] Add rules to bundle localised string paks.

Add makefile rules to install the built webkit string paks into
the system image.



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

[benm (inactive), 2013-02-12 13:58:48]

ptal.

For reference: set of paks was generated with:

find out/ -name "webkit_strings*.pak" | cut -f9 -d'/' | sort -u > /tmp/input &&
for i in `cat /tmp/input`; do echo include \$\(CLEAR_VARS\); echo LOCAL_MODULE
:= webviewchromium_${i}; echo LOCAL_MODULE_STEM := `echo ${i} | cut -f3 -d'_' |
cut -f1 -d'.'`; echo LOCAL_BUILT_MODULE_STEM := webkit/${i}; echo include
\$\(LOCAL_PATH\)/webview_pak.mk; echo; done

[mnaganov (inactive), 2013-02-12 14:05:49]

https://codereview.chromium.org/12223093/diff/1/android_webview/Android.mk
File android_webview/Android.mk (right):

https://codereview.chromium.org/12223093/diff/1/android_webview/Android.mk#ne...
android_webview/Android.mk:77: include $(CLEAR_VARS)
Is it possible to somehow iterate over a list of locales instead of making this
massive duplication?

[benm (inactive), 2013-02-12 15:06:46]

thanks mikhail!

https://codereview.chromium.org/12223093/diff/1/android_webview/Android.mk
File android_webview/Android.mk (right):

https://codereview.chromium.org/12223093/diff/1/android_webview/Android.mk#ne...
android_webview/Android.mk:77: include $(CLEAR_VARS)
I don't think so, but maybe I'm missing some make magic... The plan is that at
build time, the WebView will only include the paks required for a particular
Android product so we need to have an individual rule for each different locale.

[mnaganov (inactive), 2013-02-12 15:10:44]

LGTM

[commit-bot: I haz the power, 2013-02-12 15:15:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benm@chromium.org/12223093/7001

[commit-bot: I haz the power, 2013-02-12 16:09:08]

Retried try job too often on mac_rel for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 16:13:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benm@chromium.org/12223093/7001

[commit-bot: I haz the power, 2013-02-12 18:19:24]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 18:23:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benm@chromium.org/12223093/7001

[commit-bot: I haz the power, 2013-02-12 19:29:19]

Change committed as 181976