Allow to exlude jar in android_standalone_library.

Allow to exlude jar in android_standalone_library.

R=cjhopman@chromium.org

Committed: https://crrev.com/cd6ee6d5ce5a42c50d0284daeaa7bd664f58b8fb
Cr-Commit-Position: refs/heads/master@{#318214}

[cjhopman, 2015-02-24 21:23:04]

https://codereview.chromium.org/951353003/diff/1/build/android/gyp/dex.py
File build/android/gyp/dex.py (right):

https://codereview.chromium.org/951353003/diff/1/build/android/gyp/dex.py#new...
build/android/gyp/dex.py:53: parser.add_option('--excluded-paths-file',
You could change this to instead just take a gn/gyp list instead of a file path.

Then the new gn stuff can just pass the list and the old gyp one can use a
FileArg (i.e. the gyp one becomes "--excluded-paths-files',
'@FileArg(>(tested_apk_dex_path).inputs)'").

[qsr, 2015-02-25 16:26:19]

On 2015/02/24 21:23:04, cjhopman wrote:
> https://codereview.chromium.org/951353003/diff/1/build/android/gyp/dex.py
> File build/android/gyp/dex.py (right):
> 
>
https://codereview.chromium.org/951353003/diff/1/build/android/gyp/dex.py#new...
> build/android/gyp/dex.py:53: parser.add_option('--excluded-paths-file',
> You could change this to instead just take a gn/gyp list instead of a file
path.
> 
> Then the new gn stuff can just pass the list and the old gyp one can use a
> FileArg (i.e. the gyp one becomes "--excluded-paths-files',
> mailto:).

 I like the idea, but I don't understand how to inplement it.

 If I use FileArg, I need to take a gyp list (if I understand correctly), and if
I take a gyp list, I don't know how to pass a gn list, so how to use this in gn
(outside of using a FileArg myself, which is kind of what I'm doing in this CL).
What am I missing?

[qsr, 2015-02-25 18:29:47]

Thanks for the explanation -> Update dex.py to take a list, and used FileArg for
gyp. PTAL.

[cjhopman, 2015-02-25 18:33:19]

lgtm

[qsr, 2015-02-26 10:00:16]

The CQ bit was checked by qsr@chromium.org

[qsr, 2015-02-26 10:00:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
cjhopman@chromium.org
Link to the patchset: https://codereview.chromium.org/951353003/#ps40001 (title:
"Rebase path before passing it to external command.")

[commit-bot: I haz the power, 2015-02-26 10:00:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/951353003/40001

[commit-bot: I haz the power, 2015-02-26 10:53:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-26 10:53:57]

Patchset 3 (id:??) landed as
https://crrev.com/cd6ee6d5ce5a42c50d0284daeaa7bd664f58b8fb
Cr-Commit-Position: refs/heads/master@{#318214}