Downgrade Mac GN arg save_unstripped_output to just a variable.

Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052
Committed: https://crrev.com/1b53b935fb9febdbc52456d8643b02a39c83b737
Cr-Commit-Position: refs/heads/master@{#406338}

[Dirk Pranke, 2016-07-19 00:13:02]

Description was changed from

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685
==========

to

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052
==========

[Dirk Pranke, 2016-07-19 00:13:36]

I don't fell that strongly about it not being a declare_arg(), but it seems like
almost no one will ever want to toggle this ...

[Robert Sesek, 2016-07-19 14:13:09]

LGTM

[Dirk Pranke, 2016-07-19 17:58:16]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2016-07-19 17:59:06]

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, 2016-07-19 19:17:05]

Description was changed from

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052
==========

to

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052
==========

[commit-bot: I haz the power, 2016-07-19 19:17:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-19 19:20:25]

Description was changed from

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052
==========

to

==========
Downgrade Mac GN arg save_unstripped_output to just a variable.

It's not clear that we really need the GN arg `save_unstripped_output`
to be a user-configurable thing, so this change removes it from the
declare_arg() block in the build file and instead becomes a shorthand
for (enable_stripping && !enable_dsyms).

R=rsesek@chromium.org
BUG=623685, 628052

Committed: https://crrev.com/1b53b935fb9febdbc52456d8643b02a39c83b737
Cr-Commit-Position: refs/heads/master@{#406338}
==========

[commit-bot: I haz the power, 2016-07-19 19:20:26]

Patchset 1 (id:??) landed as
https://crrev.com/1b53b935fb9febdbc52456d8643b02a39c83b737
Cr-Commit-Position: refs/heads/master@{#406338}