WebRTC: Remove build_with_chromium=false flag for GN

WebRTC: Remove build_with_chromium=false flag for GN

After https://codereview.webrtc.org/1403453003/ we no longer need
to pass build_with_chromium=false for standalone WebRTC builds.

It's possible all bots will need a clobber build after landing this,
since they'll have args.gn files in the out/ tree containing the
variable previously passed via --args.

BUG=webrtc:5070
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=297312

[kjellander_chromium, 2015-10-14 10:58:46]

kjellander@chromium.org changed reviewers:
+ machenbach@chromium.org

[kjellander_chromium, 2015-10-14 10:58:47]

https://codereview.chromium.org/1400283004/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/webrtc/chromium_config.py (right):

https://codereview.chromium.org/1400283004/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/webrtc/chromium_config.py:51:
c.compile_py.default_targets = ['all']
The config is still needed because without it, GN on Windows will get the 'All'
target, which fails to build (only 'all' exists for GN).

I considered cleaning up our webrtc_standalone config, but I don't think it's
useful since we'll be specifying individual targets when we start using analyze
anyway, so I'm punting that work.

[Michael Achenbach, 2015-10-14 11:15:12]

lgtm - do you support landmines for the clobber? Otherwise it'll be tedious
clobbering all the shared directories on slaves from slave pools.

[kjellander_chromium, 2015-10-27 10:30:43]

On 2015/10/14 11:15:12, Michael Achenbach wrote:
> lgtm - do you support landmines for the clobber? Otherwise it'll be tedious
> clobbering all the shared directories on slaves from slave pools.

Delayed update: landmines support was added in
https://code.google.com/p/webrtc/issues/detail?id=5070 so I will land this now.

[kjellander_chromium, 2015-10-27 10:30:56]

The CQ bit was checked by kjellander@chromium.org

[kjellander_chromium, 2015-10-27 10:30:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
machenbach@chromium.org
Link to the patchset: https://codereview.chromium.org/1400283004/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2015-10-27 10:31:25]

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

[commit-bot: I haz the power, 2015-10-27 10:33:53]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=297312