Specify jar name for apks in java_apk.gypi

Specify jar name for apks in java_apk.gypi

Previously, we had to keep this name in sync in multiple places, the
"fake" xxx_java target and in sdk-targets.xml. Instead, it should be set
only in java_apk.gypi and then passed along to each of the places that
we were specifying it before.

Also, delete unused 'output' block and unneeded xxx_java dependencies.

BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=168148

[Yaron, 2012-11-08 02:22:56]

lgtm

http://codereview.chromium.org/11363131/diff/1/content/content_shell.gypi
File content/content_shell.gypi (right):

http://codereview.chromium.org/11363131/diff/1/content/content_shell.gypi#new...
content/content_shell.gypi:589: # its jar path in the variable
'apk_output_jar_path'.
Isn't this "All dependents"?

[commit-bot: I haz the power, 2012-11-12 17:26:30]

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

[commit-bot: I haz the power, 2012-11-12 17:26:34]

Presubmit check for 11363131-4001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit Messages **
If this change has an associated bug, add BUG=[bug number].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content
    chrome

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[cjhopman, 2012-11-14 23:32:53]

brettw@ for owners

http://codereview.chromium.org/11363131/diff/1/content/content_shell.gypi
File content/content_shell.gypi (right):

http://codereview.chromium.org/11363131/diff/1/content/content_shell.gypi#new...
content/content_shell.gypi:589: # its jar path in the variable
'apk_output_jar_path'.
On 2012/11/08 02:22:57, Yaron wrote:
> Isn't this "All dependents"?

Done.

[Isaac (away), 2012-11-15 06:20:47]

Now that OWNERS works on a per-file basis, seems reasonable to add a couple
android members to chrome/*_android.gypi

[brettw, 2012-11-15 20:34:37]

LGTM, I'd ask Ben about the owners change.

[commit-bot: I haz the power, 2012-11-15 20:51:43]

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

[commit-bot: I haz the power, 2012-11-15 21:37:57]

Sorry for I got bad news for ya.
Compile failed with a clobber build on android_dbg.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-11-15 21:50:19]

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

[commit-bot: I haz the power, 2012-11-16 02:37:47]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-11-16 02:38:37]

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

[commit-bot: I haz the power, 2012-11-16 07:31:50]

Change committed as 168148