Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols

Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols (patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail when running GN: 
> https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8b91d2a0d28c06a9
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed: https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b17ad2ebd64e9edd

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
Review-Url: https://codereview.webrtc.org/2697133002
Cr-Commit-Position: refs/heads/master@{#16636}
Committed: https://chromium.googlesource.com/external/webrtc/+/1c3548c67c27063171453fc25766f600c7d498a4

[kjellander_webrtc, 2017-02-15 21:05:59]

Created Reland of iOS: Use JSON for GN configuration instead of MB + remove
symbols

[kjellander_webrtc, 2017-02-16 06:34:55]

Description was changed from

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #1 id:1 of https://codereview.webrtc.org/2694423002/ )

Reason for revert:
Preparing reland CL

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
==========

to

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #1 id:1 of https://codereview.webrtc.org/2694423002/ )

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
==========

[kjellander_webrtc, 2017-02-16 06:37:58]

Description was changed from

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #1 id:1 of https://codereview.webrtc.org/2694423002/ )

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
==========

to

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
==========

[kjellander_webrtc, 2017-02-16 06:38:10]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2017-02-16 06:38:13]

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

[commit-bot: I haz the power, 2017-02-16 06:38:22]

CQ is committing da patch.

Bot data: {"patchset_id": 260001, "attempt_start_ts": 1487227090520020,
"parent_rev": "8714b8f1aecbf04eec90d1fc86f151072d629120", "commit_rev":
"1c3548c67c27063171453fc25766f600c7d498a4"}

[commit-bot: I haz the power, 2017-02-16 06:38:27]

Description was changed from

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161
==========

to

==========
Reland of iOS: Use JSON for GN configuration instead of MB + remove symbols

Reason for revert:
MB is now updated to handle spaces in the buildername properly, something that
only affected our commit bots (not trybots).

Original issue's description:
> Revert of iOS: Use JSON for GN configuration instead of MB + remove symbols
(patchset #4 id:80001 of https://codereview.webrtc.org/2688103002/ )
> 
> Reason for revert:
> Something is different from trybots vs the commit bots, causing it to fail
when running GN: 
>
https://build.chromium.org/p/client.webrtc/builders/iOS32%20Release/builds/10151
> 
> Original issue's description:
> > iOS: Use JSON for GN configuration instead of MB + remove symbols.
> > 
> > This aligns with how the ios recipe module is used in Chromium.
> > It should prevent breakages like one we had recently.
> > 
> > It also means we're no longer setting symbol_level=1 explicitly. 
> > The default is 0 (no symbols), which is now what's being used.
> > 
> > Also move all the directories containing JSON files into
> > tools-webrtc/ios/bots to make it clearer (and more similar to
> > Chromium).
> > 
> > BUG=webrtc:7140, webrtc:7161
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2688103002
> > Cr-Commit-Position: refs/heads/master@{#16633}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/73f01de4edda30c419bf6daf8...
> 
> TBR=ehmaldonado@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7140, webrtc:7161
> 
> Review-Url: https://codereview.webrtc.org/2694423002
> Cr-Commit-Position: refs/heads/master@{#16634}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/68ab366547b7494b69e28047b...

TBR=ehmaldonado@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7140, webrtc:7161

Review-Url: https://codereview.webrtc.org/2697133002
Cr-Commit-Position: refs/heads/master@{#16636}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1c3548c67c27063171453fc25...
==========

[commit-bot: I haz the power, 2017-02-16 06:38:28]

Committed patchset #2 (id:260001) as
https://chromium.googlesource.com/external/webrtc/+/1c3548c67c27063171453fc25...