fold both_gn_and_gyp and gn_only groups into gn_all

fold both_gn_and_gyp and gn_only groups into gn_all

It is not necessary to have them anymore now that GYP build is gone.

BUG=695174
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2734303002
Cr-Commit-Position: refs/heads/master@{#455566}
Committed: https://chromium.googlesource.com/chromium/src/+/fbc99249d5d02c81d0f42ffb9f6af28b1451f442

[tfarina, 2017-03-08 01:43:24]

The CQ bit was checked by tfarina@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 01:44:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tfarina, 2017-03-08 01:54:19]

Description was changed from

==========
ifold both_gn_and_gyp and gn_only groups into gn_all

It is not necessary to have them anymore now that GYP build is gone.

BUG=695174
R=dpranke@chromium.org
==========

to

==========
fold both_gn_and_gyp and gn_only groups into gn_all

It is not necessary to have them anymore now that GYP build is gone.

BUG=695174
R=dpranke@chromium.org
==========

[Dirk Pranke, 2017-03-08 02:13:36]

lgtm

https://codereview.chromium.org/2734303002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/2734303002/diff/1/BUILD.gn#newcode584
BUILD.gn:584: if (!is_ios) {
Do you feel like merging these conditions with the ones above where that's easy
to do so? You could either do that in this CL or in a follow-up one.

[commit-bot: I haz the power, 2017-03-08 02:47:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-08 02:47:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tfarina, 2017-03-08 22:17:54]

https://codereview.chromium.org/2734303002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/2734303002/diff/1/BUILD.gn#newcode584
BUILD.gn:584: if (!is_ios) {
On 2017/03/08 02:13:36, Dirk Pranke wrote:
> Do you feel like merging these conditions with the ones above where that's
easy
> to do so? You could either do that in this CL or in a follow-up one.

I'm up to it. But in a follow up, because that way it will be easier to see what
is being changed.

[tfarina, 2017-03-08 22:18:01]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2017-03-08 22:18:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-08 22:44:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489011481374950, "parent_rev":
"75ee3cee8e33bd1bcf4fb335b20acc04991448e8", "commit_rev":
"fbc99249d5d02c81d0f42ffb9f6af28b1451f442"}

[commit-bot: I haz the power, 2017-03-08 22:44:53]

Description was changed from

==========
fold both_gn_and_gyp and gn_only groups into gn_all

It is not necessary to have them anymore now that GYP build is gone.

BUG=695174
R=dpranke@chromium.org
==========

to

==========
fold both_gn_and_gyp and gn_only groups into gn_all

It is not necessary to have them anymore now that GYP build is gone.

BUG=695174
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2734303002
Cr-Commit-Position: refs/heads/master@{#455566}
Committed:
https://chromium.googlesource.com/chromium/src/+/fbc99249d5d02c81d0f42ffb9f6a...
==========

[commit-bot: I haz the power, 2017-03-08 22:44:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fbc99249d5d02c81d0f42ffb9f6a...