Convert apk-package-resources.xml to python

Convert apk-package-resources.xml to python

This ant script is basically a simple wrapper around an aapt call (the
only thing the large AaptExecTask custom ant task adds is a bunch of
dependency checking that we don't actually want anyway). This
makes it use python instead.

BUG=359249, 375324, 375431
NOTRY=true

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

[cjhopman, 2014-05-19 20:18:18]

yfriedman: *

[Yaron, 2014-05-20 01:43:37]

lgtm

Seems like you're nuking our dependencies on Ant? Is that for gn? Either way,
can you have a tracking bug for this and your other change

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi
File build/java_apk.gypi (left):

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi#oldc...
build/java_apk.gypi:442: 'resource_additional_options': [],
Just remove from https://codereview.chromium.org/295473002/

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi
File build/java_apk.gypi (right):

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi#newc...
build/java_apk.gypi:650: 'extra_package_input_paths': [
Is this not going to resolve 351928?

[cjhopman, 2014-05-20 21:01:51]

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi
File build/java_apk.gypi (left):

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi#oldc...
build/java_apk.gypi:442: 'resource_additional_options': [],
On 2014/05/20 01:43:37, Yaron wrote:
> Just remove from https://codereview.chromium.org/295473002/

Done.

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi
File build/java_apk.gypi (right):

https://codereview.chromium.org/291963002/diff/20001/build/java_apk.gypi#newc...
build/java_apk.gypi:650: 'extra_package_input_paths': [
On 2014/05/20 01:43:37, Yaron wrote:
> Is this not going to resolve 351928?
It's not. One easy way for these inputs to be incorrect (and for this step to
not be rerun when it should) is to have some foo_dir as <(asset_location) and
then not list any files in foo_dir in <(additional_input_paths) (or
<(package_input_paths)).

I think that this should probably get its assets from a single archive instead
of just grabbing everything in a single directory. Once that is done for assets
and dependency resources, 351928 will be resolved (except for the unlisted
python dependencies which is a harder problem in gyp and applies to basically
all of our actions).

[cjhopman, 2014-05-28 17:53:18]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-05-28 17:54:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cjhopman@chromium.org/291963002/60001

[cjhopman, 2014-05-28 19:47:27]

The CQ bit was unchecked by cjhopman@chromium.org

[cjhopman, 2014-05-28 19:47:28]

The CQ bit was checked by cjhopman@chromium.org

[cjhopman, 2014-05-28 19:47:44]

NOTRY to bypass presubmit, all android bots have passed

[cjhopman, 2014-05-28 19:47:49]

The CQ bit was unchecked by cjhopman@chromium.org

[cjhopman, 2014-05-28 19:47:50]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-05-28 19:48:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cjhopman@chromium.org/291963002/60001

[commit-bot: I haz the power, 2014-05-28 19:50:11]

Change committed as 273326