Reland https://codereview.chromium.org/298333007/: Enable mac_strip_release under ASan on OSX. Remo…

Reland https://codereview.chromium.org/298333007/: Enable mac_strip_release under ASan on OSX. Remove the .saves files.

The reasons for reverting the CL hadn't been documented. I'm going to reland and update the bugs.

Unce upon a time the ASan runtime library used to be statically linked into
the executables on OSX.
Because that library provided a number of API functions needed by the shared
libraries, we had custom ASan-specific .saves files that told the `strip`
utility to preserve those functions when stripping the executables.

Then ASan switched to using dynamic runtime library, which instantly broke the
stripping step, because the executables weren't necessarily referencing all the
ASan API functions (issue 242503).
As a result, stripping has been disabled, and we haven't had .dSYM debug info
for ASan builds for more than a year now (issue 148383).

Because the ASan API functions are now undefined in the executables, it's
actually senseless to use the .saves files to preserve those functions.
This CL deletes the .saves files and turns mac_strip_release on for ASan
builds.

BUG=148383, 242503, 170739, 166857
TBR=mark@chromium.org

Committed: https://crrev.com/5750a97fd179642548e6add77aafc1cdb0eec7e8
Cr-Commit-Position: refs/heads/master@{#292872}

[Alexander Potapenko, 2014-09-01 12:04:06]

The CQ bit was checked by glider@chromium.org

[commit-bot: I haz the power, 2014-09-01 12:05:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/glider@chromium.org/517803004/1

[Alexander Potapenko, 2014-09-01 12:57:45]

TBR

The patch appears to work locally, and we don't have any info about the reasons
for which it didn't work last time.
If someone's going to revert this patch, please consider posting the logs of
failing ASan bots.

[commit-bot: I haz the power, 2014-09-01 12:59:14]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[commit-bot: I haz the power, 2014-09-01 13:44:46]

Committed patchset #1 (id:1) as d5319a149964f953e7506c1ff2b33f90974d19ea

[Marc Treib, 2014-09-01 17:06:00]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/522743003/ by treib@chromium.org.

The reason for reverting is: Breaks *lots* of browser_tests on Mac ASan. See
e.g.:
http://build.chromium.org/p/chromium.memory/buildstatus?builder=Mac%20ASan%20...
http://build.chromium.org/p/chromium.memory/buildstatus?builder=Mac%20ASan%20...
http://build.chromium.org/p/chromium.memory/buildstatus?builder=Mac%20ASan%20....

[commit-bot: I haz the power, 2014-09-10 03:16:52]

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