Android: Make webview builds more convenient.

Android: Make webview builds more convenient.

Allow the top level Android.mk to include GypAndroid.mk if the relevant
environment variable is set, to avoid having to edit this file to build
in the Android tree without breaking other Android builds. Also, provide
a gyp wrapper for webview which passes the correct options.

BUG=

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

[Torne, 2012-10-11 15:07:26]

+yfriedman

Yaron, are you okay with this top level Android.mk change? It saves us having to
modify this downstream in the Android tree to test building libwebview.

[mnaganov (inactive), 2012-10-11 15:11:05]

lgtm

[Yaron, 2012-10-12 00:15:17]

lgtm

FYI: Bo was able to remove all the existing Android.mk files

https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
File android_webview/tools/gyp_webview (right):

https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
android_webview/tools/gyp_webview:17: android_gyp
Did you want to set the relevant gyp generator or is that supposed to be handled
outside the script?

[Torne, 2012-10-12 10:00:52]

On 2012/10/12 00:15:17, Yaron wrote:
> lgtm
> 
> FYI: Bo was able to remove all the existing Android.mk files

Well, we still need an Android.mk in the top level anyway, since a bunch of the
third_party directories have them and otherwise things will go wrong when
chromium is checked out inside an android tree. :)

>
https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
> File android_webview/tools/gyp_webview (right):
> 
>
https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
> android_webview/tools/gyp_webview:17: android_gyp
> Did you want to set the relevant gyp generator or is that supposed to be
handled
> outside the script?

[Torne, 2012-10-12 10:01:01]

https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
File android_webview/tools/gyp_webview (right):

https://codereview.chromium.org/11090069/diff/1/android_webview/tools/gyp_web...
android_webview/tools/gyp_webview:17: android_gyp
On 2012/10/12 00:15:17, Yaron wrote:
> Did you want to set the relevant gyp generator or is that supposed to be
handled
> outside the script?

envsetup handles the webview build, when CHROME_ANDROID_BUILD_WEBVIEW is set.

[boliu, 2012-10-12 14:10:54]

On 2012/10/12 10:00:52, Torne wrote:
> On 2012/10/12 00:15:17, Yaron wrote:
> > lgtm
> > 
> > FYI: Bo was able to remove all the existing Android.mk files
> 
> Well, we still need an Android.mk in the top level anyway, since a bunch of
the
> third_party directories have them and otherwise things will go wrong when
> chromium is checked out inside an android tree. :)
> 
I didn't delete the top level one downstream, so should be ok.