Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as needed"

Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.

Committed: https://crrev.com/19adc908726781b299b441d3142f29143f234753
Cr-Commit-Position: refs/heads/master@{#418640}

[antz1, 2016-09-14 00:37:02]

Description was changed from

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archieve libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

to

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archieve libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

[antz1, 2016-09-14 00:37:02]

antz@chromium.org changed reviewers:
+ bcf@chromium.org, brettw@chromium.org, dpranke@chromium.org,
lcwu@chromium.org, wzhong@chromium.org

[antz1, 2016-09-14 00:38:15]

Description was changed from

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archieve libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

to

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

[Dirk Pranke, 2016-09-14 17:09:11]

lgtm

[bcf, 2016-09-14 17:15:41]

lgtm

[antz1, 2016-09-14 18:08:30]

The CQ bit was checked by antz@chromium.org

[commit-bot: I haz the power, 2016-09-14 18:09:15]

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

[wzhong, 2016-09-14 18:18:00]

lgtm

[commit-bot: I haz the power, 2016-09-14 18:18:04]

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

[commit-bot: I haz the power, 2016-09-14 18:18:05]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[antz1, 2016-09-14 18:37:51]

antz@chromium.org changed reviewers:
+ halliwell@chromium.org, slan@chromium.org

[slan, 2016-09-14 18:42:23]

lgtm

[antz1, 2016-09-14 18:44:10]

The CQ bit was checked by antz@chromium.org

[commit-bot: I haz the power, 2016-09-14 18:44:44]

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-09-14 19:36:45]

Description was changed from

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

to

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

[commit-bot: I haz the power, 2016-09-14 19:36:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-14 19:39:49]

Description was changed from

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.
==========

to

==========
Revert "[Chromecast] Only include all libstdc++/libgcc symbols in executable as
needed"

Reason for revert:
The original fix is opt-in, which causes unit test binaries to miss symbols. I
will
reimplement an opt-out for binaries that we do not want to whole-archive libs.

Original issue description:
>> All chromecast executable include all libstdc++/libgcc symbols by default
>> due to b/25566835. Added a config for --whole-archive for only ones that
>> depend on shared libs. ~700kb saving per executable.

BUG=internal b/31417380

This reverts commit 92947f235fcf22912990029925b4154e7ea08f74.

Committed: https://crrev.com/19adc908726781b299b441d3142f29143f234753
Cr-Commit-Position: refs/heads/master@{#418640}
==========

[commit-bot: I haz the power, 2016-09-14 19:39:50]

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