[NaCl SDK] Fix create_nmf to include arm/glibc ELF loader

[NaCl SDK] Fix create_nmf to include arm/glibc ELF loader

Add tests for ARM glibc nmfs.

CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_nacl_sdk;tryserver.chromium.mac:mac_nacl_sdk;tryserver.chromium.win:win_nacl_sdk
BUG=505885
Committed: https://crrev.com/5afc24479ba6b45f12caaa1fb805930004db2596
Cr-Commit-Position: refs/heads/master@{#354536}

[Sam Clegg, 2015-10-15 00:04:21]

sbc@chromium.org changed reviewers:
+ binji@chromium.org, mcgrathr@chromium.org

[Sam Clegg, 2015-10-15 01:09:00]

sbc@chromium.org changed reviewers:
+ bradnelson@chromium.org

[Sam Clegg, 2015-10-15 02:14:56]

The CQ bit was checked by sbc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-15 02:15:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1398883004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1398883004/40001

[commit-bot: I haz the power, 2015-10-15 04:29:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-15 04:29:45]

Dry run: Try jobs failed on following builders:
  mac_nacl_sdk on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_nacl_sdk/buil...)

[bradn, 2015-10-15 04:48:07]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2015-10-15 04:48:08]

Oh, that makes more sense :-), I wasn't sure how the loader worked for arm.
lgtm

[binji, 2015-10-15 17:06:07]

lgtm

https://codereview.chromium.org/1398883004/diff/40001/native_client_sdk/src/t...
File native_client_sdk/src/tools/tests/create_nmf_test.py (right):

https://codereview.chromium.org/1398883004/diff/40001/native_client_sdk/src/t...
native_client_sdk/src/tools/tests/create_nmf_test.py:307:
self.assertEqual(strip_manifest['program'], expected['program'])
why separate this out?

[Sam Clegg, 2015-10-15 17:27:23]

https://codereview.chromium.org/1398883004/diff/40001/native_client_sdk/src/t...
File native_client_sdk/src/tools/tests/create_nmf_test.py (right):

https://codereview.chromium.org/1398883004/diff/40001/native_client_sdk/src/t...
native_client_sdk/src/tools/tests/create_nmf_test.py:307:
self.assertEqual(strip_manifest['program'], expected['program'])
On 2015/10/15 17:06:07, binji wrote:
> why separate this out?

It makes debugging the failures a lot easier.  The builtin diff display for
large python dictionaries is almost impossible to read when the tests fail.  
Then change just breaks the huge assertion into several smaller ones that are
easier parse when they fail.

[Sam Clegg, 2015-10-16 17:06:58]

The CQ bit was checked by sbc@chromium.org

[Sam Clegg, 2015-10-16 17:06:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
bradnelson@google.com, binji@chromium.org
Link to the patchset: https://codereview.chromium.org/1398883004/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-10-16 17:07:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1398883004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1398883004/60001

[commit-bot: I haz the power, 2015-10-16 17:37:36]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-16 17:38:23]

Patchset 4 (id:??) landed as
https://crrev.com/5afc24479ba6b45f12caaa1fb805930004db2596
Cr-Commit-Position: refs/heads/master@{#354536}