Include the WebKit targets on all Android builds

Include the WebKit targets on all Android builds

Promote DumpRenderTree and webkit_unit_tests to targets being part of All (thus
out of experimental), while also adding TestWebKitAPI by correctly referring to
WebKit's All.gyp instead of (the seemingly Windows-oriented) webkit/webkit.gyp.

After this change, three new targets will be build for Android on the main
waterfall. The compile time for new required files (they all rely heavily on
webkit.a) is negligible, link time is more significant, namely ~40 seconds per
target, but this will be parallelized on the bot with other targets. Increase
in build time is ~50 seconds on my T3500.

BUG=
TEST=DumpRenderTree, webkit_unit_tests and TestWebKitAPI should be build on the
Android buildbots.


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

[Yaron, 2012-03-15 17:33:32]

lgtm but up to jrg

[John Grabowski, 2012-03-15 17:35:08]

LGTM!

[Peter Beverloo, 2012-03-15 17:35:17]

+ some more people

[bulach, 2012-03-15 17:44:23]

lgtm

http://codereview.chromium.org/9701070/diff/1/build/all_android.gyp
File build/all_android.gyp (right):

http://codereview.chromium.org/9701070/diff/1/build/all_android.gyp#newcode36
build/all_android.gyp:36: # From here down: not added to run_tests.py yet.
because of this entry, maybe put the new target at the top, as it's not a test?

[Peter Beverloo, 2012-03-15 17:54:08]

On 2012/03/15 17:44:23, bulach wrote:
> lgtm
> 
> http://codereview.chromium.org/9701070/diff/1/build/all_android.gyp
> File build/all_android.gyp (right):
> 
> http://codereview.chromium.org/9701070/diff/1/build/all_android.gyp#newcode36
> build/all_android.gyp:36: # From here down: not added to run_tests.py yet.
> because of this entry, maybe put the new target at the top, as it's not a
test?

While TestWebKitAPI and webkit_unit_tests are tests and can run through
run_tests.py, we indeed won't run these on the Chromium bots. I'll move it to
above the content. Good catch!

Cheers guys! try-bot should be here shortly.

[commit-bot: I haz the power, 2012-03-19 10:42:32]

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

[commit-bot: I haz the power, 2012-03-19 13:53:30]

Change committed as 127459