Add gyp flag to specify whether we should optimize JNI generation.

Add gyp flag to specify whether we should optimize JNI generation.

Do not regenerate JNI files (and subsequently the .so file) if they have
not changed.  This will happen if you edit a java file that has a native
counterpart, but the native signatures were not touched.

We do not enable this all the time as some build systems require that
when you modify an input, the output should also be modified.  This
also will result in more command line output, so we'll keep it behind
a developer flag for now.

BUG=


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

[Ted C, 2013-02-21 02:31:23]

My old branch got deleted, so starting this anew.

[Torne, 2013-02-21 09:48:29]

On 2013/02/21 02:31:23, Ted C wrote:
> My old branch got deleted, so starting this anew.

Just for reference, you can "git cl patch issuenumber" to download the patch for
an old issue onto the current branch and "git cl issue issuenumber" to set the
current branch's issue number such that uploading makes it a new patch for the
same CL. :)

[Torne, 2013-02-21 09:49:17]

lgtm

[Yaron, 2013-02-21 18:57:33]

lgtm

https://chromiumcodereview.appspot.com/12314025/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/12314025/diff/1/build/common.gypi#newc...
build/common.gypi:1154: # the outputs should always change as well.
Maybe add: "This is meant purely for developer builds, to avoid spurious
re-linking of native files." ?

[Ted C, 2013-02-21 19:01:52]

https://chromiumcodereview.appspot.com/12314025/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/12314025/diff/1/build/common.gypi#newc...
build/common.gypi:1154: # the outputs should always change as well.
On 2013/02/21 18:57:33, Yaron wrote:
> Maybe add: "This is meant purely for developer builds, to avoid spurious
> re-linking of native files." ?

Done.

[commit-bot: I haz the power, 2013-02-21 19:04:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedchoc@chromium.org/12314025/2003

[commit-bot: I haz the power, 2013-02-21 22:13:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedchoc@chromium.org/12314025/2003

[commit-bot: I haz the power, 2013-02-22 00:37:34]

Change committed as 183936