Android: build webview on default bots.

Android: build webview on default bots.

Move the webview targets from the android_experimental target into All
so they are built by default on the main tree and on trybots. The tests
are not run currently; this is just checking that they compile.

The libwebview target is a dependency of android_webview_apk so it
doesn't need to be specified explicitly here.

BUG=


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

[Isaac (away), 2012-09-07 17:52:02]

lgtm.

Note this will cause the targets to be build on all our trybots as well.  From
[http://build.chromium.org/p/chromium.fyi/stats/Chromium%20Linux%20Android%20Builder]
it looks like this step will add about 150 seconds on clean builds w/ similar
spaces to the perf lab machines.

[Isaac (away), 2012-09-07 17:54:23]

fyi I'm not a full committer yet, it's being processed today but CQ prob
won't have it until monday.

On Fri, Sep 7, 2012 at 10:37 AM, <torne@chromium.org> wrote:

> Reviewers: Isaac,
>
> Description:
> Android: build webview on default bots.
>
> Move the webview targets from the android_experimental target into All
> so they are built by default on the main tree and on trybots. The tests
> are not run currently; this is just checking that they compile.
>
> The libwebview target is a dependency of android_webview_apk so it
> doesn't need to be specified explicitly here.
>
> BUG=
>
>
> Please review this at
https://chromiumcodereview.**appspot.com/10918124/<https://chromiumcodereview...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M build/all_android.gyp
>
>
> Index: build/all_android.gyp
> diff --git a/build/all_android.gyp b/build/all_android.gyp
> index c3b103a292724f7f88b37d07fa93bb**fd5317a59a..**
> 1fc43cb47e400b89ada2ffb4586ae7**f131bde1b4 100644
> --- a/build/all_android.gyp
> +++ b/build/all_android.gyp
> @@ -17,6 +17,7 @@
>        'target_name': 'All',
>        'type': 'none',
>        'dependencies': [
> +        '../android_webview/android_**webview.gyp:android_webview_**apk',
>          '../content/content.gyp:**content_shell_apk',
>          '<@(android_app_targets)',
>          'android_builder_tests',
> @@ -34,6 +35,7 @@
>        'target_name': 'android_builder_tests',
>        'type': 'none',
>        'dependencies': [
> +        '../android_webview/android_**webview.gyp:android_webview_**
> test_apk',
>          '../base/android/jni_**generator/jni_generator.gyp:**
> jni_generator_tests',
>          '../base/base.gyp:base_**unittests',
>          '../chrome/chrome.gyp:unit_**tests',
> @@ -94,9 +96,6 @@
>        'target_name': 'android_experimental',
>        'type': 'none',
>        'dependencies': [
> -        '../android_webview/android_**webview.gyp:libwebview',
> -        '../android_webview/android_**webview.gyp:android_webview_**apk',
> -        '../android_webview/android_**webview.gyp:android_webview_**
> test_apk',
>        ],
>      },
>      {
>
>
>

[Yaron, 2012-09-07 18:47:52]

lgtm

[commit-bot: I haz the power, 2012-09-07 19:26:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/torne@chromium.org/10918124/1

[commit-bot: I haz the power, 2012-09-07 21:53:51]

Change committed as 155498

[Isaac (away), 2012-09-09 04:03:49]

I forgot that new folders that we do not need for our tester need to be excluded
in bb_zip_build in buildbot/buildbot_functions.sh.  Preparing CL for that.