Fix shell_apptest for android.

Fix shell_apptest for android.

This CL introduces mojo/tools/embed/rules.gni that allows to embed a
file in the data section of an object file. It refactors icu_data to use
this, and uses it in shell_apptest to bundle the ping application.

R=abarth@chromium.org, davemoore@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/dc3d193b9cac268e3adfc22f2750a82bf9558376

[qsr, 2015-03-05 14:21:52]

Adam, can you look at icu_data and tell me if that's fine with you.

Dave, can you look to the shell app tests changes.

Thanks.

[qsr, 2015-03-06 09:05:08]

gentle ping?

[DaveMoore, 2015-03-06 17:36:35]

test changes LGTM, I don't know much about the rest

[abarth-chromium, 2015-03-06 21:41:14]

lgtm

https://codereview.chromium.org/979043003/diff/1/mojo/tools/embed/embed_data.py
File mojo/tools/embed/embed_data.py (right):

https://codereview.chromium.org/979043003/diff/1/mojo/tools/embed/embed_data....
mojo/tools/embed/embed_data.py:53: f.write('namespace embed {\n')
Seems odd to put this all into the same namespace.  Maybe we should be able to
control the namespace from GN?

[qsr, 2015-03-09 10:17:20]

On 2015/03/06 21:41:14, abarth wrote:
> lgtm
> 
>
https://codereview.chromium.org/979043003/diff/1/mojo/tools/embed/embed_data.py
> File mojo/tools/embed/embed_data.py (right):
> 
>
https://codereview.chromium.org/979043003/diff/1/mojo/tools/embed/embed_data....
> mojo/tools/embed/embed_data.py:53: f.write('namespace embed {\n')
> Seems odd to put this all into the same namespace.  Maybe we should be able to
> control the namespace from GN?

 Done.

[qsr, 2015-03-09 10:17:42]

Committed patchset #2 (id:20001) manually as
dc3d193b9cac268e3adfc22f2750a82bf9558376 (presubmit successful).

[jamesr, 2015-03-09 17:58:56]

jamesr@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2015-03-09 17:58:56]

apptests on the android tester are failing after this patch:

http://build.chromium.org/p/client.mojo/builders/Mojo%20Android%20Builder%20T...

Failed command: "/b/build/slave/mojo/build/src/mojo/tools/android_mojo_shell.py
'--origin=http://127.0.0.1:11347/' '--args-for=mojo:shell_apptests
--gtest_filter=ShellHTTPAppTest.Http' 'mojo:shell_apptests'"
------------------------------------------------------------------------
[1002/140313:ERROR:simple_index_file.cc(223)] Failed to write the temporary
index file
[INFO:network_fetcher.cc(70)] Caching mojo app
http://127.0.0.1:11347/tracing.mojo at
/data/data/org.chromium.mojo.shell/cache/tmp/abc04b542a7ecdc1e81c3f30106084c27926726280fdec7ee721b1fe7aaec1be.mojo
[INFO:network_fetcher.cc(70)] Caching mojo app
http://127.0.0.1:11347/shell_apptests.mojo at
/data/data/org.chromium.mojo.shell/cache/tmp/8515f065f37aa32e7163eef8d02bc5385b7593c86439119fe1de5a746f24cb00.mojo
Note: Google Test filter = ShellHTTPAppTest.Http
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ShellHTTPAppTest
[ RUN      ] ShellHTTPAppTest.Http