android: Remove unneeded md5sum class for jar.py and jar_toc.py.

build/java_apk.gypi

Index: build/java_apk.gypi
diff --git a/build/java_apk.gypi b/build/java_apk.gypi
index b4dbb72d1b7af3d4220c080d7910a8eb68f9560f..92e1d2a9890d1648bf6c99bdefc71e5ce75c177f 100644
--- a/build/java_apk.gypi
+++ b/build/java_apk.gypi
@@ -592,10 +592,10 @@
       'action_name': 'jar_<(_target_name)',
       'message': 'Creating <(_target_name) jar',
       'inputs': [
-        '<(instr_stamp)',
         '<(DEPTH)/build/android/gyp/util/build_utils.py',
         '<(DEPTH)/build/android/gyp/util/md5_check.py',
         '<(DEPTH)/build/android/gyp/jar.py',
+        '<(instr_stamp)',
       ],
       'outputs': [
         '<(jar_stamp)',
@@ -606,9 +606,6 @@
         '--jar-path=<(jar_path)',
         '--excluded-classes=<(jar_excluded_classes)',
         '--stamp=<(jar_stamp)',

[Nico, 2014/03/06 20:48:02]

I noticed that nothing depends on jar_stamp (and that there's no jar_toc in this
file). It looks like dex_action.py uses classes_final_dir directly.

What are the jar files written by this action used for?

[cjhopman, 2014/03/06 21:25:28]

The jar is used by dependents via `apk_output_jar_path` above, which just
happens to be the same as `jar_path` (this should be fixed to actually use
`jar_path` there). There is then a target (like content_shell_apk_java) that
depends on the apk target and acts like a normal java library target to its
dependents. This is needed so that the tests can refer to the classes included
in the apk here.

Really though, it's a total hack. Instead it should be inverted and the
java_apk.gypi shouldn't deal directly with java files or resources. Instead
java+resources should be extracted out to a seperate library target and then the
apk and test targets can both just depend on that (and then java_apk.gypi really
does just package stuff together into an apk).

-
-        # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
-        '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
     {