Fix java handler.

Fix java handler.

The java handler expects a Mojo-Class entry in the manifest of the
application. The latest dx tool strip the manifest to its bare minimum
and removed everything we added. This change the build system to add the
entry to the manifest after running dx.

R=blundell@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/9df4d8e03991ea837b090bb7ce8f6fbdff849c53

[blundell, 2015-05-11 14:49:14]

lgtm

https://codereview.chromium.org/1126393005/diff/1/services/android/add_manife...
File services/android/add_manifest_entry.py (right):

https://codereview.chromium.org/1126393005/diff/1/services/android/add_manife...
services/android/add_manifest_entry.py:7: """Add an entry into the manigest file
of a jar file."""
nit: s/manigest/manifest everywhere (including cl description)

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni
File services/android/rules.gni (left):

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni#...
services/android/rules.gni:27: manifest_entries = [ "Mojo-Class:" +
invoker.mojo_main ]
is there still a point to this?

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni
File services/android/rules.gni (right):

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni#...
services/android/rules.gni:8: servicess_android_path = get_path_info(".",
"abspath")
hmm, if this is intended to be available to consumers it should be in
//mojo/services/android/public instead (pre-existing issue not to be fixed in
this cl obviously).

[qsr, 2015-05-11 14:54:52]

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

[qsr, 2015-05-11 14:56:27]

https://codereview.chromium.org/1126393005/diff/1/services/android/add_manife...
File services/android/add_manifest_entry.py (right):

https://codereview.chromium.org/1126393005/diff/1/services/android/add_manife...
services/android/add_manifest_entry.py:7: """Add an entry into the manigest file
of a jar file."""
On 2015/05/11 14:49:13, blundell wrote:
> nit: s/manigest/manifest everywhere (including cl description)

Done.

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni
File services/android/rules.gni (left):

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni#...
services/android/rules.gni:27: manifest_entries = [ "Mojo-Class:" +
invoker.mojo_main ]
On 2015/05/11 14:49:14, blundell wrote:
> is there still a point to this?

Removed

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni
File services/android/rules.gni (right):

https://codereview.chromium.org/1126393005/diff/1/services/android/rules.gni#...
services/android/rules.gni:8: servicess_android_path = get_path_info(".",
"abspath")
On 2015/05/11 14:49:14, blundell wrote:
> hmm, if this is intended to be available to consumers it should be in
> //mojo/services/android/public instead (pre-existing issue not to be fixed in
> this cl obviously).

Acknowledged.