jar_file_jni_generator accepts jar_input_file optionally

jar_file_jni_generator accepts jar_input_file optionally


BUG=


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

[ycheo (away), 2013-03-25 05:42:32]

Hi Marcus,

With the recent CL, input_jar_file was removed.
But, in my project, the variable is needed to specify GoogleTV SDK jar file.
So I added the variable it again and set its default value as the Android SDK
jar, not to break the current build.

Thanks,
Yuncheol

[bulach, 2013-03-25 13:46:00]

lgtm, but please let torne verify his project too :)

[Torne, 2013-03-25 14:27:47]

lgtm

https://codereview.chromium.org/12710012/diff/1/build/jar_file_jni_generator....
File build/jar_file_jni_generator.gypi (right):

https://codereview.chromium.org/12710012/diff/1/build/jar_file_jni_generator....
build/jar_file_jni_generator.gypi:19: # Optional varibales:
nit: spelling (variables) :)

[ycheo (away), 2013-03-26 00:23:48]

https://codereview.chromium.org/12710012/diff/1/build/jar_file_jni_generator....
File build/jar_file_jni_generator.gypi (right):

https://codereview.chromium.org/12710012/diff/1/build/jar_file_jni_generator....
build/jar_file_jni_generator.gypi:19: # Optional varibales:
On 2013/03/25 14:27:47, Torne wrote:
> nit: spelling (variables) :)

Done.

[commit-bot: I haz the power, 2013-03-26 00:26:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ycheo@chromium.org/12710012/7001

[commit-bot: I haz the power, 2013-03-26 07:01:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ycheo@chromium.org/12710012/7001

[commit-bot: I haz the power, 2013-03-26 08:03:55]

Change committed as 190585