Mojo: Make the top-level mojo target build more.

Mojo: Make the top-level mojo target build more.

("more" = "everything but the Android-only targets")

R=sky@chromium.org, sky

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

[sky, 2013-11-06 23:29:41]

LGTM

https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp
File mojo/mojo.gyp (right):

https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp#newcode18
mojo/mojo.gyp:18: 'mojo_system_unittests',
Can you sort these?

[abarth-chromium, 2013-11-06 23:30:17]

https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp
File mojo/mojo.gyp (right):

https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp#newcode26
mojo/mojo.gyp:26: 'native_viewport',
Can we add the android ones in a condition block?

[viettrungluu, 2013-11-07 00:03:55]

On 2013/11/06 23:30:17, abarth wrote:
> https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp
> File mojo/mojo.gyp (right):
> 
> https://codereview.chromium.org/63393002/diff/1/mojo/mojo.gyp#newcode26
> mojo/mojo.gyp:26: 'native_viewport',
> Can we add the android ones in a condition block?

I'll do that separately, once I have an android build up and running (or you can
do that if you want and it's convenient). I'm hesitant to do the change blindly
(since I can never tell what's going to work with gyp).

[commit-bot: I haz the power, 2013-11-07 00:07:09]

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

[abarth-chromium, 2013-11-07 00:08:00]

Ok.  I can take care of it.

[commit-bot: I haz the power, 2013-11-07 05:39:03]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[viettrungluu, 2013-11-07 06:42:26]

Committed patchset #2 manually as r233508 (presubmit successful).