Android WebView: add empty jarjar ruleset.

Android WebView: add empty jarjar ruleset.

We don't have any jarjar rules yet, but add the empty file now in order
that the downstream build can enable jarjar. This way, when rules are
added later it will just work, instead of breaking the downstream build
temporarily.

Also, make sure that when a jarjar rule file is being used, it's
considered as an input to the JNI generator step, to make sure the JNI
header files get regenerated when the rules change.

BUG=

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

[bulach, 2013-05-30 10:50:35]

lgtm, just nits:

https://codereview.chromium.org/15888011/diff/1/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/15888011/diff/1/build/common.gypi#newcode1318
build/common.gypi:1318: 'jni_generator_jarjar_file':
'../android_webview/build/jarjar-rules.txt',
does it need a % suffix?

https://codereview.chromium.org/15888011/diff/1/build/jni_generator.gypi
File build/jni_generator.gypi (right):

https://codereview.chromium.org/15888011/diff/1/build/jni_generator.gypi#newc...
build/jni_generator.gypi:59: ['jni_generator_jarjar_file!=""', {
nit: space around !=

[Torne, 2013-05-30 11:08:31]

https://codereview.chromium.org/15888011/diff/1/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/15888011/diff/1/build/common.gypi#newcode1318
build/common.gypi:1318: 'jni_generator_jarjar_file':
'../android_webview/build/jarjar-rules.txt',
On 2013/05/30 10:50:36, bulach wrote:
> does it need a % suffix?

Not really; that just allows it to be overridden in GYP_DEFINES which shouldn't
be happening here (we need to agree about the jarjar rules with the Android
build system). I tried it and found a bug in gyp, though; it doesn't invoke
pathname relativization if you have the % suffix because it's checking to see if
the variable name looks like a path *before* removing the %. So, I'll leave it
as-is? :)

https://codereview.chromium.org/15888011/diff/1/build/jni_generator.gypi
File build/jni_generator.gypi (right):

https://codereview.chromium.org/15888011/diff/1/build/jni_generator.gypi#newc...
build/jni_generator.gypi:59: ['jni_generator_jarjar_file!=""', {
On 2013/05/30 10:50:36, bulach wrote:
> nit: space around !=

Done.

[commit-bot: I haz the power, 2013-05-30 11:28:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/torne@chromium.org/15888011/6001

[commit-bot: I haz the power, 2013-05-30 14:16:23]

Change committed as 203141