Make sanitizer signal handlers work again.

services/service_manager/embedder/main.cc

Index: services/service_manager/embedder/main.cc
diff --git a/services/service_manager/embedder/main.cc b/services/service_manager/embedder/main.cc
index c636bd2cef8d2abd36cb635130c4adfbe2e732d1..1312a4d4028c403bd48622577053af25f1f4f2e9 100644
--- a/services/service_manager/embedder/main.cc
+++ b/services/service_manager/embedder/main.cc
@@ -108,6 +108,9 @@ class ServiceProcessLauncherDelegateImpl
 
 // Setup signal-handling state: resanitize most signals, ignore SIGPIPE.
 void SetupSignalHandlers() {
+#if !defined(ADDRESS_SANITIZER) && !defined(LEAK_SANITIZER) &&  \

[jam, 2017/04/28 00:24:50]

can we use the same switch as in content? i..e move it from content/ to
services/service_manager/switches.h, and then have this code and content use it?

that way, knowledge about what breaks asan is on asan bot (through command line
flag) instead of here which is quite removed from it

[Oliver Chang, 2017/04/28 17:05:06]

Done. I put it in service_manager/embedder/switches.h instead since otherwise
there's a dependency from service_manager/embedder to service_manager/ and GN
complains. I hope this is fine?

I kept the sanitizer #ifdef checks in so that in a non-sanitizer build there's
no unintended side effects from disabling in process stack traces.

[jam, 2017/05/01 15:37:24]

Even content doesn't have these checks, so I don't think we have anything to
worry about. Let's remove them, since keeping them there means that this code
has knowledge about what breaks *san which is what the switch is trying to
avoid.

[Oliver Chang, 2017/05/01 16:05:14]

Sure, removed.

+    !defined(MEMORY_SANITIZER) && !defined(THREAD_SANITIZER) && \
+    !defined(UNDEFINED_SANITIZER)
   // Sanitise our signal handling state. Signals that were ignored by our
   // parent will also be ignored by us. We also inherit our parent's sigmask.
   sigset_t empty_signal_set;
@@ -126,6 +129,7 @@ void SetupSignalHandlers() {
 
   // Always ignore SIGPIPE.  We check the return value of write().
   CHECK_NE(SIG_ERR, signal(SIGPIPE, SIG_IGN));
+#endif
 }
 
 void PopulateFDsFromCommandLine() {