[android-webview] Skip signal handler registration on x86/64

android_webview/common/aw_crash_handler.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "android_webview/common/aw_crash_handler.h"
 
 #include <android/log.h>
 #include <signal.h>
 #include <sys/prctl.h>
 #include <sys/syscall.h>
 #include <unistd.h>
 
 #include "base/logging.h"
+#include "build/build_config.h"
 
 namespace {
 
 const int kExceptionSignals[] = {
   SIGSEGV, SIGABRT, SIGFPE, SIGILL, SIGBUS
 };
 
 struct sigaction old_handlers[arraysize(kExceptionSignals)];
 
 bool crash_handler_registered;
@@ skipping 35 matching lines @@
     }
   }
 }
 
 }  // namespace
 
 namespace android_webview {
 namespace crash_handler {
 
 void RegisterCrashHandler(const std::string& version) {
+#if defined(ARCH_CPU_X86_FAMILY)
+  // Don't install signal handler on X86/64 because this breaks binary
+  // translators that handle SIGSEGV in userspace and get chained after our
+  // handler. See crbug.com/477444
+  return;
+#endif
+
   if (crash_handler_registered) {
     NOTREACHED();
     return;
   }
 
   g_crash_msg = "### WebView " + version;
   g_crash_msg_ptr = g_crash_msg.c_str();
 
   // Fail if unable to store all the old handlers.
   for (uint32_t i = 0; i < arraysize(kExceptionSignals); ++i) {
@@ skipping 22 matching lines @@
       LOG(ERROR) << "Error while overriding handler for signal "
                  << kExceptionSignals[i];
     }
   }
 
   crash_handler_registered = true;
 }
 
 }  // namespace crash_handler
 }  // namespace android_webview