Add app-as-shared-library flag in java_apk.gypi

Add app-as-shared-library flag in java_apk.gypi

So it could be used for building apk

BUG=532598
Committed: https://crrev.com/915104dacd3dca9594e9aadf0e196af387025d63
Cr-Commit-Position: refs/heads/master@{#349492}

[michaelbai, 2015-09-16 19:53:54]

michaelbai@chromium.org changed reviewers:
+ jbudorick@chromium.org

[Yaron, 2015-09-16 20:57:37]

On 2015/09/16 19:54:19, michaelbai wrote:

drive-by: please add GN support as well. we're trying to get everything up and
running in gn so we need help in order to not chase/re-implement all features

[michaelbai, 2015-09-16 21:01:30]

On 2015/09/16 20:57:37, Yaron wrote:
> On 2015/09/16 19:54:19, michaelbai wrote:
> 
> drive-by: please add GN support as well. we're trying to get everything up and
> running in gn so we need help in order to not chase/re-implement all features

I were thinking about this, but I can't verify the change of GN, because the
target will use this flag dependents on WebView.

[Yaron, 2015-09-16 21:16:26]

On 2015/09/16 21:01:30, michaelbai wrote:
> On 2015/09/16 20:57:37, Yaron wrote:
> > On 2015/09/16 19:54:19, michaelbai wrote:
> > 
> > drive-by: please add GN support as well. we're trying to get everything up
and
> > running in gn so we need help in order to not chase/re-implement all
features
> 
> I were thinking about this, but I can't verify the change of GN, because the
> target will use this flag dependents on WebView.

Can someone from webview try adding targets for GN? You're missing out on a nice
speed boost. GN is working fine for chrome now.

[jbudorick, 2015-09-16 21:19:46]

On 2015/09/16 at 21:16:26, yfriedman wrote:
> On 2015/09/16 21:01:30, michaelbai wrote:
> > On 2015/09/16 20:57:37, Yaron wrote:
> > > On 2015/09/16 19:54:19, michaelbai wrote:
> > > 
> > > drive-by: please add GN support as well. we're trying to get everything up
and
> > > running in gn so we need help in order to not chase/re-implement all
features
> > 
> > I were thinking about this, but I can't verify the change of GN, because the
> > target will use this flag dependents on WebView.
> 
> Can someone from webview try adding targets for GN? You're missing out on a
nice speed boost. GN is working fine for chrome now.

gyp parts lgtm, but ^

[michaelbai, 2015-09-16 21:21:47]

On 2015/09/16 21:16:26, Yaron wrote:
> On 2015/09/16 21:01:30, michaelbai wrote:
> > On 2015/09/16 20:57:37, Yaron wrote:
> > > On 2015/09/16 19:54:19, michaelbai wrote:
> > > 
> > > drive-by: please add GN support as well. we're trying to get everything up
> and
> > > running in gn so we need help in order to not chase/re-implement all
> features
> > 
> > I were thinking about this, but I can't verify the change of GN, because the
> > target will use this flag dependents on WebView.
> 
> Can someone from webview try adding targets for GN? You're missing out on a
nice
> speed boost. GN is working fine for chrome now.

+ torne

[michaelbai, 2015-09-16 21:21:49]

On 2015/09/16 21:16:26, Yaron wrote:
> On 2015/09/16 21:01:30, michaelbai wrote:
> > On 2015/09/16 20:57:37, Yaron wrote:
> > > On 2015/09/16 19:54:19, michaelbai wrote:
> > > 
> > > drive-by: please add GN support as well. we're trying to get everything up
> and
> > > running in gn so we need help in order to not chase/re-implement all
> features
> > 
> > I were thinking about this, but I can't verify the change of GN, because the
> > target will use this flag dependents on WebView.
> 
> Can someone from webview try adding targets for GN? You're missing out on a
nice
> speed boost. GN is working fine for chrome now.

+ torne

[Torne, 2015-09-17 10:28:54]

On 2015/09/16 21:21:49, michaelbai wrote:
> On 2015/09/16 21:16:26, Yaron wrote:
> > On 2015/09/16 21:01:30, michaelbai wrote:
> > > On 2015/09/16 20:57:37, Yaron wrote:
> > > > On 2015/09/16 19:54:19, michaelbai wrote:
> > > > 
> > > > drive-by: please add GN support as well. we're trying to get everything
up
> > and
> > > > running in gn so we need help in order to not chase/re-implement all
> > features
> > > 
> > > I were thinking about this, but I can't verify the change of GN, because
the
> > > target will use this flag dependents on WebView.
> > 
> > Can someone from webview try adding targets for GN? You're missing out on a
> nice
> > speed boost. GN is working fine for chrome now.
> 
> + torne

You can build and run all the android stuff in GN? If so then yes, we should add
webview. Can someone file a bug for this?

[Yaron, 2015-09-17 12:31:48]

On 2015/09/17 10:28:54, Torne wrote:
> On 2015/09/16 21:21:49, michaelbai wrote:
> > On 2015/09/16 21:16:26, Yaron wrote:
> > > On 2015/09/16 21:01:30, michaelbai wrote:
> > > > On 2015/09/16 20:57:37, Yaron wrote:
> > > > > On 2015/09/16 19:54:19, michaelbai wrote:
> > > > > 
> > > > > drive-by: please add GN support as well. we're trying to get
everything
> up
> > > and
> > > > > running in gn so we need help in order to not chase/re-implement all
> > > features
> > > > 
> > > > I were thinking about this, but I can't verify the change of GN, because
> the
> > > > target will use this flag dependents on WebView.
> > > 
> > > Can someone from webview try adding targets for GN? You're missing out on
a
> > nice
> > > speed boost. GN is working fine for chrome now.
> > 
> > + torne
> 
> You can build and run all the android stuff in GN? If so then yes, we should
add
> webview. Can someone file a bug for this?

Ya, I'm using it for local development. Bug filed:
https://code.google.com/p/chromium/issues/detail?id=532905

[michaelbai, 2015-09-17 19:17:36]

The CQ bit was checked by michaelbai@chromium.org

[commit-bot: I haz the power, 2015-09-17 19:18:33]

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

[commit-bot: I haz the power, 2015-09-17 20:55:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-17 20:56:41]

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