Add support for shared library resources to java_apk build rules for real.

Add support for shared library resources to java_apk build rules for real.

In https://crrev.com/f90b7151cb809d8ec9ab31f7be9e9f8f575fd4d0 I added support
for the process_resources rule but support in package_resources is needed to
generate a compatible .apk file.

BUG=430997
Committed: https://crrev.com/e5c9c76ba67333b430debcaaa5d4880e801114d5
Cr-Commit-Position: refs/heads/master@{#304097}

[mkosiba (inactive), 2014-11-13 20:49:00]

mkosiba@chromium.org changed reviewers:
+ cjhopman@chromium.org

[mkosiba (inactive), 2014-11-13 20:49:01]

PTAL

[cjhopman, 2014-11-13 21:11:48]

Does package_resources still need the flag? The files generated at that step are
only used for compilation, the ones by apk_package are the ones actually built
into the apk.

[cjhopman, 2014-11-13 21:12:35]

On 2014/11/13 21:11:48, cjhopman wrote:
> Does package_resources still need the flag? The files generated at that step
are
> only used for compilation, the ones by apk_package are the ones actually built
> into the apk.

ugh, s/package_resources/process_resources and s/apk_package/package_resources

[mkosiba (inactive), 2014-11-13 21:13:06]

On 2014/11/13 21:11:48, cjhopman wrote:
> Does package_resources still need the flag? The files generated at that step
are
> only used for compilation, the ones by apk_package are the ones actually built
> into the apk.

yes, flipping that flag in package_resoruces causes aapt to generate extra code
in the R.java class

[mkosiba (inactive), 2014-11-13 21:15:20]

On 2014/11/13 21:13:06, mkosiba wrote:
> On 2014/11/13 21:11:48, cjhopman wrote:
> > Does package_resources still need the flag? The files generated at that step
> are
> > only used for compilation, the ones by apk_package are the ones actually
built
> > into the apk.
> 
> yes, flipping that flag in package_resoruces causes aapt to generate extra
code
> in the R.java class

I mean flipping that flag in *process_resources* causes aapt to generate extra
code
in the R.java class

[cjhopman, 2014-11-13 21:18:46]

lgtm

[mkosiba (inactive), 2014-11-13 21:20:40]

The CQ bit was checked by mkosiba@chromium.org

[commit-bot: I haz the power, 2014-11-13 21:22:26]

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

[commit-bot: I haz the power, 2014-11-13 22:38:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-11-13 22:38:45]

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