Revert 234207 "Set the default ASan options for executables buil..."

Revert 234207 "Set the default ASan options for executables buil..."

Seems to cause two Chrome Memory bots to fail consistently with the error
"Failed cbuildbot failed buildpackages failed report"

See these links:
http://build.chromium.org/p/chromium.memory/builders/Chromium%20OS%20(x86)%20ASAN
http://build.chromium.org/p/chromium.memory/builders/Chromium%20OS%20(amd64)%20ASAN

If those links break, mpearson@ has a copy of the failure output
(too long to paste here).


> Set the default ASan options for executables built with ASan on Linux.
> 
> This CL introduces a module, base/debug/sanitizer_options.cc, which will override
> the defaults for various dynamic tools (only ASan at this moment). For every executable
> built with a dynamic tool this module will be linked into that executable, providing
> weak functions to be called by the tool.
> 
> The existing declaration of __asan_default_options() in chrome/app/chrome_exe_main_gtk.cc
> has been moved into sanitizer_options.cc (now every binary built with GOOGLE_CHROME_BUILD=1
> will have the same options as google-chrome-asan.
> The existing declaration of __asan_default_options() in chrome/nacl/nacl_helper_linux.cc
> has been kept as is, but we had to remove -Wl,-u_sanitizer_options_link_helper to avoid picking sanitizer_options.cc.
> 
> GYP changes for OSX and iOS will be committed separately.
> 
> BUG=302040
> R=thakis@chromium.org
> 
> Review URL: https://codereview.chromium.org/25687005

TBR=glider@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=234232

[Mark P, 2013-11-11 17:25:37]

Committed patchset #1 manually as r234232.

[Alexander Potapenko, 2013-11-12 06:12:23]

LGTM, thanks!
On Nov 11, 2013 9:25 PM, <mpearson@chromium.org> wrote:

> Committed patchset #1 manually as r234232.
>
> https://codereview.chromium.org/68843003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.