Update Crashpad to 3c573b54ae138c81c0224f22b38f5d439b56f9a1

9ed829054711 Bring up skeleton crashpad_client_ios
22701bb0e448 fuchsia: Use zx_system_get_version_string
f79cba47ba0d [lsan][crashpad] Suppress leak sanitizer on crashpad_test
0c20aeabd37a [UBSan] Temporarily disable UBSan for snapshot and minidump
d632f625a2f2 Fix a Wrange-loop-analysis warning in crashpad
8bad6f140e0b [UBSan] Temporarily disable UBSan for gtest and file
11b8eb10d4ad Don’t use the disappearing MSVC_PUSH_DISABLE_WARNING()
             macro
3c573b54ae13 [ios] Fix iOS device build

Change-Id: Icaa0526b9d2b070ac0a70f6f3bfeca319b077ce2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2081530
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#746235}

base/BUILD.gn

@@ -2017,6 +2017,8 @@ jumbo_component("base") {
     } else {
       sources += [ "time/time_exploded_ios.cc" ]
     }
+
+    libs += [ "UIKit.framework" ]
   }
 
   if (dep_libevent) {

third_party/crashpad/README.chromium

@@ -2,7 +2,7 @@ Name: Crashpad
 Short Name: crashpad
 URL: https://crashpad.chromium.org/
 Version: unknown
-Revision: faed21a2862529b4ebead8cf178427b56fcacdca
+Revision: 3c573b54ae138c81c0224f22b38f5d439b56f9a1
 License: Apache 2.0
 License File: crashpad/LICENSE
 Security Critical: yes

third_party/crashpad/crashpad/BUILD.gn

@@ -20,6 +20,19 @@ config("crashpad_config") {
   include_dirs = [ "." ]
 }
 
+# TODO(fuchsia:46805): Remove this once instances of UB have been cleaned up.
+config("disable_ubsan") {
+  if (crashpad_is_in_fuchsia) {
+    cflags = [ "-fno-sanitize=undefined" ]
+  }
+  visibility = [
+    "snapshot:snapshot",
+    "minidump:minidump_test",
+    "third_party/gtest:gtest",
+    "util:util",
+  ]
+}
+
 if (crashpad_is_in_chromium || crashpad_is_in_fuchsia) {
   test("crashpad_tests") {
     deps = [
@@ -35,6 +48,10 @@ if (crashpad_is_in_chromium || crashpad_is_in_fuchsia) {
         "util:util_test",
       ]
     }
+    if (crashpad_is_in_fuchsia) {
+      # TODO(fuchsia:46559): Fix the leaks and remove this.
+      deps += [ "//build/config/sanitizers:suppress-lsan.DO-NOT-USE-THIS" ]
+    }
   }
 
   if (crashpad_is_in_fuchsia) {

third_party/crashpad/crashpad/client/BUILD.gn

@@ -43,6 +43,10 @@ static_library("client") {
     ]
   }
 
+  if (crashpad_is_ios) {
+    sources += [ "crashpad_client_ios.cc" ]
+  }
+
   if (crashpad_is_linux || crashpad_is_android) {
     set_sources_assignment_filter([])
     sources += [
@@ -137,7 +141,9 @@ source_set("client_test") {
     "../util",
   ]
 
-  data_deps = [ "../handler:crashpad_handler" ]
+  if (!crashpad_is_ios) {
+    data_deps = [ "../handler:crashpad_handler" ]
+  }
 
   if (crashpad_is_win) {
     data_deps += [ "../handler:crashpad_handler_console" ]

third_party/crashpad/crashpad/client/crashpad_client.h

@@ -423,9 +423,21 @@ class CrashpadClient {
   //!
   //! \param[in] unhandled_signals The set of unhandled signals
   void SetUnhandledSignals(const std::set<int>& unhandled_signals);
-
 #endif  // OS_LINUX || OS_ANDROID || DOXYGEN
 
+#if defined(OS_IOS) || DOXYGEN
+  //! \brief Configures the process to direct its crashes to the iOS in-process
+  //! Crashpad handler.
+  //!
+  //! This method is only defined on iOS.
+  //!
+  //! \return `true` on success, `false` on failure with a message logged.
+  //!
+  //! TODO(justincohen): This method will need to take database, metrics_dir,
+  //! url and annotations eventually.
+  bool StartCrashpadInProcessHandler();
+#endif
+
 #if defined(OS_MACOSX) || DOXYGEN
   //! \brief Sets the process’ crash handler to a Mach service registered with
   //!     the bootstrap server.

third_party/crashpad/crashpad/client/crashpad_client_ios.cc

+// Copyright 2020 The Crashpad Authors. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include "client/crashpad_client.h"
+
+#include <unistd.h>
+
+#include "base/logging.h"
+#include "base/strings/stringprintf.h"
+#include "client/client_argv_handling.h"
+#include "util/posix/signals.h"
+
+namespace crashpad {
+
+namespace {
+
+// A base class for Crashpad signal handler implementations.
+class SignalHandler {
+ public:
+  // Returns the currently installed signal hander.
+  static SignalHandler* Get() {
+    static SignalHandler* instance = new SignalHandler();
+    return instance;
+  }
+
+  bool Install(const std::set<int>* unhandled_signals) {
+    return Signals::InstallCrashHandlers(
+        HandleSignal, 0, &old_actions_, unhandled_signals);
+  }
+
+ private:
+  SignalHandler() = default;
+
+  // The base implementation for all signal handlers, suitable for calling
+  // directly to simulate signal delivery.
+  void HandleCrash(int signo, siginfo_t* siginfo, void* context) {
+    // Do Something.
+
+    // Always call system handler.
+    Signals::RestoreHandlerAndReraiseSignalOnReturn(
+        siginfo, old_actions_.ActionForSignal(signo));
+  }
+
+  // The signal handler installed at OS-level.
+  static void HandleSignal(int signo, siginfo_t* siginfo, void* context) {
+    Get()->HandleCrash(signo, siginfo, context);
+  }
+
+  Signals::OldActions old_actions_ = {};
+
+  DISALLOW_COPY_AND_ASSIGN(SignalHandler);
+};
+
+}  // namespace
+
+CrashpadClient::CrashpadClient() {}
+
+CrashpadClient::~CrashpadClient() {}
+
+bool CrashpadClient::StartCrashpadInProcessHandler() {
+  return SignalHandler::Get()->Install(nullptr);
+}
+
+}  // namespace crashpad

third_party/crashpad/crashpad/compat/BUILD.gn

@@ -19,7 +19,7 @@ config("compat_config") {
 
   if (crashpad_is_mac) {
     include_dirs += [ "mac" ]
-  } else {
+  } else if (!crashpad_is_ios) {
     include_dirs += [ "non_mac" ]
   }
 
@@ -43,7 +43,7 @@ config("compat_config") {
 }
 
 template("compat_target") {
-  if (crashpad_is_mac) {
+  if (crashpad_is_mac || crashpad_is_ios) {
     # There are no sources to compile, which doesn’t mix will with a
     # static_library.
     group(target_name) {
@@ -67,7 +67,7 @@ compat_target("compat") {
       "mac/mach/mach.h",
       "mac/sys/resource.h",
     ]
-  } else {
+  } else if (!crashpad_is_ios) {
     sources += [
       "non_mac/mach-o/loader.h",
       "non_mac/mach/mach.h",

third_party/crashpad/crashpad/handler/BUILD.gn

@@ -140,25 +140,27 @@ source_set("handler_test") {
   }
 }
 
-crashpad_executable("crashpad_handler") {
-  sources = [ "main.cc" ]
+if (!crashpad_is_ios) {
+  crashpad_executable("crashpad_handler") {
+    sources = [ "main.cc" ]
 
-  deps = [
-    ":handler",
-    "../build:default_exe_manifest_win",
-    "../compat",
-    "../third_party/mini_chromium:base",
-  ]
+    deps = [
+      ":handler",
+      "../build:default_exe_manifest_win",
+      "../compat",
+      "../third_party/mini_chromium:base",
+    ]
 
-  if (crashpad_is_win) {
-    if (crashpad_is_in_chromium || crashpad_is_in_dart) {
-      remove_configs = [ "//build/config/win:console" ]
-      configs = [ "//build/config/win:windowed" ]
-    } else {
-      remove_configs =
-          [ "//third_party/mini_chromium/mini_chromium/build:win_console" ]
-      configs =
-          [ "//third_party/mini_chromium/mini_chromium/build:win_windowed" ]
+    if (crashpad_is_win) {
+      if (crashpad_is_in_chromium || crashpad_is_in_dart) {
+        remove_configs = [ "//build/config/win:console" ]
+        configs = [ "//build/config/win:windowed" ]
+      } else {
+        remove_configs =
+            [ "//third_party/mini_chromium/mini_chromium/build:win_console" ]
+        configs =
+            [ "//third_party/mini_chromium/mini_chromium/build:win_windowed" ]
+      }
     }
   }
 }
@@ -192,19 +194,21 @@ if (crashpad_is_android) {
   }
 }
 
-crashpad_executable("crashpad_handler_test_extended_handler") {
-  testonly = true
+if (!crashpad_is_ios) {
+  crashpad_executable("crashpad_handler_test_extended_handler") {
+    testonly = true
 
-  sources = [ "crashpad_handler_test_extended_handler.cc" ]
+    sources = [ "crashpad_handler_test_extended_handler.cc" ]
 
-  deps = [
-    ":handler",
-    "../build:default_exe_manifest_win",
-    "../compat",
-    "../minidump:test_support",
-    "../third_party/mini_chromium:base",
-    "../tools:tool_support",
-  ]
+    deps = [
+      ":handler",
+      "../build:default_exe_manifest_win",
+      "../compat",
+      "../minidump:test_support",
+      "../third_party/mini_chromium:base",
+      "../tools:tool_support",
+    ]
+  }
 }
 
 if (crashpad_is_win) {

third_party/crashpad/crashpad/minidump/BUILD.gn

@@ -164,4 +164,6 @@ source_set("minidump_test") {
   if (crashpad_is_win) {
     cflags = [ "/wd4201" ]  # nonstandard extension used : nameless struct/union
   }
+
+  configs += [ "..:disable_ubsan" ]
 }

third_party/crashpad/crashpad/minidump/minidump_extensions.h

@@ -25,14 +25,15 @@
 #include "util/misc/pdb_structures.h"
 #include "util/misc/uuid.h"
 
+#if defined(COMPILER_MSVC)
 // C4200 is "nonstandard extension used : zero-sized array in struct/union".
 // We would like to globally disable this warning, but unfortunately, the
 // compiler is buggy and only supports disabling it with a pragma, so we can't
-// disable it with other silly warnings in build/common.gypi. See:
+// disable it with other silly warnings in the build files. See:
 //   https://connect.microsoft.com/VisualStudio/feedback/details/1114440
-MSVC_PUSH_DISABLE_WARNING(4200)
+#pragma warning(push)
+#pragma warning(disable: 4200)
 
-#if defined(COMPILER_MSVC)
 #define PACKED
 #pragma pack(push, 1)
 #else
@@ -498,11 +499,10 @@ struct ALIGNAS(4) PACKED MinidumpCrashpadInfo {
 
 #if defined(COMPILER_MSVC)
 #pragma pack(pop)
+#pragma warning(pop)  // C4200
 #endif  // COMPILER_MSVC
 #undef PACKED
 
-MSVC_POP_WARNING()  // C4200
-
 }  // namespace crashpad
 
 #endif  // CRASHPAD_MINIDUMP_MINIDUMP_EXTENSIONS_H_

third_party/crashpad/crashpad/snapshot/BUILD.gn

@@ -234,6 +234,8 @@ static_library("snapshot") {
     cflags = [ "/wd4201" ]  # nonstandard extension used : nameless struct/union
     libs = [ "powrprof.lib" ]
   }
+
+  configs += [ "..:disable_ubsan" ]
 }
 
 if (crashpad_is_linux && use_fuzzing_engine) {
@@ -329,7 +331,7 @@ source_set("snapshot_test") {
       "sanitized/process_snapshot_sanitized_test.cc",
       "sanitized/sanitization_information_test.cc",
     ]
-  } else {
+  } else if (!crashpad_is_ios) {
     sources += [ "crashpad_info_client_options_test.cc" ]
   }
 

third_party/crashpad/crashpad/snapshot/fuchsia/system_snapshot_fuchsia.cc

@@ -38,10 +38,7 @@ void SystemSnapshotFuchsia::Initialize(const timeval* snapshot_time) {
   // garnet/bin/uname/uname.c, however, this information isn't provided by
   // uname(). Additionally, uname() seems to hang if the network is in a bad
   // state when attempting to retrieve the nodename, so avoid it for now.
-  char kernel_version[256] = {};
-  zx_status_t status =
-      zx_system_get_version(kernel_version, sizeof(kernel_version));
-  ZX_LOG_IF(ERROR, status != ZX_OK, status) << "zx_system_get_version";
+  std::string kernel_version = zx_system_get_version_string();
 
 #if defined(ARCH_CPU_X86_64)
   static constexpr const char kArch[] = "x86_64";
@@ -50,8 +47,8 @@ void SystemSnapshotFuchsia::Initialize(const timeval* snapshot_time) {
 #else
   static constexpr const char kArch[] = "unknown";
 #endif
-  os_version_full_ =
-      base::StringPrintf("Zircon prerelease %s %s", kernel_version, kArch);
+  os_version_full_ = base::StringPrintf(
+      "Zircon prerelease %s %s", kernel_version.c_str(), kArch);
 
   INITIALIZATION_STATE_SET_VALID(initialized_);
 }

third_party/crashpad/crashpad/test/ios/BUILD.gn

@@ -40,6 +40,7 @@ source_set("google_test_runner") {
   deps = [
     "../../build:ios_enable_arc",
     "../../build:ios_xctest",
+    "../../test/ios:google_test_runner_shared_headers",
     "../../third_party/mini_chromium:base",
   ]
   libs = [ "UIKit.framework" ]

third_party/crashpad/crashpad/test/ios/crash_type_xctest.mm

@@ -14,26 +14,135 @@
 
 #import <XCTest/XCTest.h>
 
+#include <objc/runtime.h>
 #import "Service/Sources/EDOClientService.h"
-#import "test/ios/host/edo_placeholder.h"
+#import "test/ios/host/cptest_shared_object.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
-@interface CPTestTestCase : XCTestCase
+@interface CPTestTestCase : XCTestCase {
+  XCUIApplication* _app;
+}
+
 @end
 
 @implementation CPTestTestCase
 
+- (void)handleCrashUnderSymbol:(id)arg1 {
+  // For now, do nothing.  In the future this can be something testable.
+}
+
++ (void)setUp {
+  // Swizzle away the handleCrashUnderSymbol callback.  Without this, any time
+  // the host app is intentionally crashed, the test is immediately failed.
+  SEL originalSelector = NSSelectorFromString(@"handleCrashUnderSymbol:");
+  SEL swizzledSelector = @selector(handleCrashUnderSymbol:);
+
+  Method originalMethod = class_getInstanceMethod(
+      objc_getClass("XCUIApplicationImpl"), originalSelector);
+  Method swizzledMethod =
+      class_getInstanceMethod([self class], swizzledSelector);
+
+  method_exchangeImplementations(originalMethod, swizzledMethod);
+
+  // Override EDO default error handler.  Without this, the default EDO error
+  // handler will throw an error and fail the test.
+  [EDOClientService setErrorHandler:^(NSError* error){
+      // Do nothing.
+  }];
+}
+
 - (void)setUp {
-  [[[XCUIApplication alloc] init] launch];
+  _app = [[XCUIApplication alloc] init];
+  [_app launch];
 }
 
 - (void)testEDO {
-  EDOPlaceholder* rootObject = [EDOClientService rootObjectWithPort:12345];
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
   NSString* result = [rootObject testEDO];
   XCTAssertEqualObjects(result, @"crashpad");
 }
 
+- (void)testSegv {
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+
+  // Crash the app.
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
+  [rootObject crashSegv];
+
+  // Confirm the app is not running.
+  XCTAssertTrue([_app waitForState:XCUIApplicationStateNotRunning timeout:15]);
+  XCTAssertTrue(_app.state == XCUIApplicationStateNotRunning);
+
+  // TODO: Query the app for crash data
+  [_app launch];
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+}
+
+- (void)testKillAbort {
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+
+  // Crash the app.
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
+  [rootObject crashKillAbort];
+
+  // Confirm the app is not running.
+  XCTAssertTrue([_app waitForState:XCUIApplicationStateNotRunning timeout:15]);
+  XCTAssertTrue(_app.state == XCUIApplicationStateNotRunning);
+
+  // TODO: Query the app for crash data
+  [_app launch];
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+}
+
+- (void)testTrap {
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+
+  // Crash the app.
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
+  [rootObject crashTrap];
+
+  // Confirm the app is not running.
+  XCTAssertTrue([_app waitForState:XCUIApplicationStateNotRunning timeout:15]);
+  XCTAssertTrue(_app.state == XCUIApplicationStateNotRunning);
+
+  // TODO: Query the app for crash data
+  [_app launch];
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+}
+
+- (void)testAbort {
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+
+  // Crash the app.
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
+  [rootObject crashAbort];
+
+  // Confirm the app is not running.
+  XCTAssertTrue([_app waitForState:XCUIApplicationStateNotRunning timeout:15]);
+  XCTAssertTrue(_app.state == XCUIApplicationStateNotRunning);
+
+  // TODO: Query the app for crash data
+  [_app launch];
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+}
+
+- (void)testBadAccess {
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+
+  // Crash the app.
+  CPTestSharedObject* rootObject = [EDOClientService rootObjectWithPort:12345];
+  [rootObject crashBadAccess];
+
+  // Confirm the app is not running.
+  XCTAssertTrue([_app waitForState:XCUIApplicationStateNotRunning timeout:15]);
+  XCTAssertTrue(_app.state == XCUIApplicationStateNotRunning);
+
+  // TODO: Query the app for crash data
+  [_app launch];
+  XCTAssertTrue(_app.state == XCUIApplicationStateRunningForeground);
+}
+
 @end

third_party/crashpad/crashpad/test/ios/host/BUILD.gn

@@ -22,13 +22,9 @@ if (crashpad_is_in_chromium) {
 
 source_set("app_shared_sources") {
   testonly = true
-  sources = [
-    "edo_placeholder.h",
-  ]
+  sources = [ "cptest_shared_object.h" ]
   configs += [ "../../..:crashpad_config" ]
-  deps = [
-    "../../../build:ios_enable_arc",
-  ]
+  deps = [ "../../../build:ios_enable_arc" ]
   libs = [ "UIKit.framework" ]
 }
 
@@ -45,6 +41,7 @@ static_library("app_host_sources") {
   deps = [
     ":app_shared_sources",
     "../../../build:ios_enable_arc",
+    "../../../client",
     "../../../third_party/edo",
   ]
   libs = [
@@ -56,7 +53,5 @@ static_library("app_host_sources") {
 ios_app_bundle("ios_crash_xcuitests") {
   info_plist = "Info.plist"
   testonly = true
-  deps = [
-    ":app_host_sources",
-  ]
+  deps = [ ":app_host_sources" ]
 }

third_party/crashpad/crashpad/test/ios/host/application_delegate.mm

@@ -16,8 +16,9 @@
 
 #import "Service/Sources/EDOHostNamingService.h"
 #import "Service/Sources/EDOHostService.h"
+#include "client/crashpad_client.h"
+#import "test/ios/host/cptest_shared_object.h"
 #import "test/ios/host/crash_view_controller.h"
-#import "test/ios/host/edo_placeholder.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
@@ -28,6 +29,10 @@
 
 - (BOOL)application:(UIApplication*)application
     didFinishLaunchingWithOptions:(NSDictionary*)launchOptions {
+  // Start up crashpad.
+  crashpad::CrashpadClient client;
+  client.StartCrashpadInProcessHandler();
+
   self.window = [[UIWindow alloc] initWithFrame:[[UIScreen mainScreen] bounds]];
   [self.window makeKeyAndVisible];
   self.window.backgroundColor = UIColor.greenColor;
@@ -37,17 +42,37 @@
 
   // Start up EDO.
   [EDOHostService serviceWithPort:12345
-                       rootObject:[[EDOPlaceholder alloc] init]
+                       rootObject:[[CPTestSharedObject alloc] init]
                             queue:dispatch_get_main_queue()];
-  [EDOHostNamingService.sharedService start];
-
   return YES;
 }
 
 @end
 
-@implementation EDOPlaceholder
+@implementation CPTestSharedObject
 - (NSString*)testEDO {
   return @"crashpad";
 }
+
+- (void)crashBadAccess {
+  strcpy(0, "bla");
+}
+
+- (void)crashKillAbort {
+  kill(getpid(), SIGABRT);
+}
+
+- (void)crashSegv {
+  long zero = 0;
+  *(long*)zero = 0xC045004d;
+}
+
+- (void)crashTrap {
+  __builtin_trap();
+}
+
+- (void)crashAbort {
+  abort();
+}
+
 @end

third_party/crashpad/crashpad/test/ios/host/edo_placeholder.h → third_party/crashpad/crashpad/test/ios/host/cptest_shared_object.h

@@ -12,13 +12,29 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef CRASHPAD_TEST_IOS_HOST_EDO_PLACEHOLDER_H_
-#define CRASHPAD_TEST_IOS_HOST_EDO_PLACEHOLDER_H_
+#ifndef CRASHPAD_TEST_IOS_HOST_SHARED_OBJECT_H_
+#define CRASHPAD_TEST_IOS_HOST_SHARED_OBJECT_H_
 
 #import <UIKit/UIKit.h>
 
-@interface EDOPlaceholder : NSObject
+@interface CPTestSharedObject : NSObject
+// Returns the string "crashpad" for testing EDO.
 - (NSString*)testEDO;
+
+// Triggers an EXC_BAD_ACCESS exception and crash.
+- (void)crashBadAccess;
+
+// Triggers a crash with a call to kill(SIGABRT).
+- (void)crashKillAbort;
+
+// Triggers a segfault crash.
+- (void)crashSegv;
+
+// Trigger a crash with a __builtin_trap.
+- (void)crashTrap;
+
+// Trigger a crash with an abort().
+- (void)crashAbort;
 @end
 
-#endif  // CRASHPAD_TEST_IOS_HOST_EDO_PLACEHOLDER_H_
+#endif  // CRASHPAD_TEST_IOS_HOST_SHARED_OBJECT_H_

third_party/crashpad/crashpad/third_party/gtest/BUILD.gn

@@ -34,6 +34,7 @@ if (crashpad_is_in_chromium) {
     public_deps = [
       "//third_party/googletest:gtest",
     ]
+    public_configs = [ "../..:disable_ubsan" ]
   }
   group("gmock") {
     testonly = true

third_party/crashpad/crashpad/tools/BUILD.gn

@@ -25,29 +25,31 @@ source_set("tool_support") {
   deps = [ "../third_party/mini_chromium:base" ]
 }
 
-crashpad_executable("crashpad_database_util") {
-  sources = [ "crashpad_database_util.cc" ]
+if (!crashpad_is_ios) {
+  crashpad_executable("crashpad_database_util") {
+    sources = [ "crashpad_database_util.cc" ]
 
-  deps = [
-    ":tool_support",
-    "../build:default_exe_manifest_win",
-    "../client",
-    "../compat",
-    "../third_party/mini_chromium:base",
-    "../util",
-  ]
-}
+    deps = [
+      ":tool_support",
+      "../build:default_exe_manifest_win",
+      "../client",
+      "../compat",
+      "../third_party/mini_chromium:base",
+      "../util",
+    ]
+  }
 
-crashpad_executable("crashpad_http_upload") {
-  sources = [ "crashpad_http_upload.cc" ]
+  crashpad_executable("crashpad_http_upload") {
+    sources = [ "crashpad_http_upload.cc" ]
 
-  deps = [
-    ":tool_support",
-    "../build:default_exe_manifest_win",
-    "../compat",
-    "../third_party/mini_chromium:base",
-    "../util",
-  ]
+    deps = [
+      ":tool_support",
+      "../build:default_exe_manifest_win",
+      "../compat",
+      "../third_party/mini_chromium:base",
+      "../util",
+    ]
+  }
 }
 
 crashpad_executable("base94_encoder") {
@@ -60,7 +62,7 @@ crashpad_executable("base94_encoder") {
   ]
 }
 
-if (!crashpad_is_fuchsia) {
+if (!crashpad_is_fuchsia && !crashpad_is_ios) {
   crashpad_executable("generate_dump") {
     sources = [ "generate_dump.cc" ]
 

third_party/crashpad/crashpad/util/BUILD.gn

@@ -514,6 +514,8 @@ static_library("util") {
   if (crashpad_is_android || crashpad_is_linux) {
     deps += [ "../third_party/lss" ]
   }
+
+  configs += [ "..:disable_ubsan" ]
 }
 
 if (!crashpad_is_android) {

third_party/crashpad/crashpad/util/misc/capture_context.h

@@ -17,7 +17,9 @@
 
 #include "build/build_config.h"
 
-#if defined(OS_MACOSX)
+#if defined(OS_IOS)
+#include <sys/ucontext.h>
+#elif defined(OS_MACOSX)
 #include <mach/mach.h>
 #elif defined(OS_WIN)
 #include <windows.h>
@@ -29,7 +31,9 @@
 
 namespace crashpad {
 
-#if defined(OS_MACOSX)
+#if defined(OS_IOS)
+using NativeCPUContext = ucontext_t;
+#elif defined(OS_MACOSX)
 #if defined(ARCH_CPU_X86_FAMILY)
 using NativeCPUContext = x86_thread_state;
 #endif

third_party/crashpad/crashpad/util/stdlib/strnlen.cc

@@ -14,7 +14,8 @@
 
 #include "util/stdlib/strnlen.h"
 
-#if defined(OS_MACOSX) && MAC_OS_X_VERSION_MIN_REQUIRED < MAC_OS_X_VERSION_10_7
+#if defined(OS_MACOSX) && !defined(OS_IOS) && \
+    MAC_OS_X_VERSION_MIN_REQUIRED < MAC_OS_X_VERSION_10_7
 
 #if MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_7
 // Redeclare a method only available on Mac OS X 10.7 and later to suppress a

third_party/crashpad/crashpad/util/stdlib/strnlen.h

@@ -20,7 +20,7 @@
 
 #include "build/build_config.h"
 
-#if defined(OS_MACOSX)
+#if defined(OS_MACOSX) && !defined(OS_IOS)
 #include <AvailabilityMacros.h>
 #endif
 
@@ -38,7 +38,7 @@ namespace crashpad {
 //!     and not all systems’ standard libraries provide an implementation.
 size_t strnlen(const char* string, size_t max_length);
 
-#if !defined(OS_MACOSX) || \
+#if !defined(OS_MACOSX) || defined(OS_IOS) || \
     MAC_OS_X_VERSION_MIN_REQUIRED >= MAC_OS_X_VERSION_10_7
 inline size_t strnlen(const char* string, size_t max_length) {
   return ::strnlen(string, max_length);