[Mac] Call dump_syms with the -g flag when building with mac_breakpad=1.

chrome/tools/build/mac/dump_product_syms

 #!/bin/bash
 
 # Copyright (c) 2012 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.
 
 # This script expects the following environment variables to be set.  Xcode
 # normally sets them:
 #
 # CONFIGURATION - Release or Debug; this script only operates when Release.
@@ skipping 89 matching lines @@
   # SRC_STEM is the name of the file within the DWARF directory of the .dSYM
   # bundle, which comes from the on-disk name of an executable or dylib within
   # its enclosing .app, .framework or .plugin bundle.  This is the bundle name
   # without .app, .framework or .plugin appended.  For non-bundled types, the
   # stem is just the name of the singular file on disk.
   SRC_STEM=$(echo "${SRC_NAME}" | sed -Ee 's/\.(app|framework|plugin)$//')
   DSYM_NAME="${SRC_NAME}.dSYM"
   DSYM_PATH="${BUILT_PRODUCTS_DIR}/${DSYM_NAME}"
   DWARF_PATH="${DSYM_PATH}/Contents/Resources/DWARF/${SRC_STEM}"
 
+  # SRC_PATH is the path to the Mach-O file to which the DSYM_PATH corresponds.
+  # If this is a directory, it is a bundle, and the path to the actual image
+  # needs to be computed for dump_syms.
+  SRC_PREFIX="${BUILT_PRODUCTS_DIR}/${SRC_NAME}"
+  SRC_PATH=""
+  if [[ -d "${SRC_PREFIX}" ]]; then
+    BUNDLED_OPTIONS=(
+        # Executables and plugins:
+        "${SRC_PREFIX}/Contents/MacOS/${SRC_STEM}"
+        # Frameworks:
+        "${SRC_PREFIX}/Versions/Current/${SRC_STEM}"
+    )
+    for BUNDLED_OPTION in "${BUNDLED_OPTIONS[@]}"; do
+      if [[ -e "${BUNDLED_OPTION}" ]]; then

[Mark Mentovai, 2014/07/25 22:28:43]

Use a combination of -f and -x, not -e.

[Robert Sesek, 2014/07/25 22:37:36]

Done.

+        SRC_PATH="${BUNDLED_OPTION}"
+        break
+      fi
+    done
+
+    if [[ -z "${SRC_PATH}" ]]; then
+      echo "${0}: Could not find bundled Mach-O file for ${SRC_NAME}"
+      exit 1
+    fi
+  else
+    # The Mach-O file is not a bundle.
+    SRC_PATH="${SRC_PREFIX}"
+  fi
+
   ARCHS=$(file "${DWARF_PATH}" | sed -Ene 's/^.*(i386|x86_64)$/\1/p')
   if [[ -z "${ARCHS}" ]]; then
     echo "${0}: expected something dumpable in ${DWARF_PATH}" >& 2
     exit 1
   fi
 
   for ARCH in ${ARCHS}; do
     BPAD_SYM_NAME="${SRC_NAME}-${FULL_VERSION}-${ARCH}.breakpad"
     BPAD_SYM_PATH="${BUILT_PRODUCTS_DIR}/${BPAD_SYM_NAME}"
 
-    # Only run dump_syms if the file has changed since the last dump. Use -c
-    # to avoid dumping CFI, because the Breakpad stackwalk is incompatible
-    # with CFI produced by clang.
-    # http://code.google.com/p/google-breakpad/issues/detail?id=443
+    # Only run dump_syms if the file has changed since the last dump. Use -g
+    # to dump data from the dSYM and CFI data from the Mach-O library or
+    # executable.
     if [ "${DWARF_PATH}" -nt "${BPAD_SYM_PATH}" ] ; then

[Mark Mentovai, 2014/07/25 22:28:43]

You should also check if SRC_PATH is newer than BPAD_SYM_PATH.

[Robert Sesek, 2014/07/25 22:37:36]

Done.

-      "${BREAKPAD_DUMP_SYMS}" -a "${ARCH}" -c "${DWARF_PATH}" > \
+      "${BREAKPAD_DUMP_SYMS}" -a "${ARCH}" -g "${DWARF_PATH}" "${SRC_PATH}" > \
           "${BPAD_SYM_PATH}"
     fi
 
     # Some executables will show up with variant names. The Breakpad symbol
     # server looks up modules based on a combination of the module name and
     # identifier (UUID). Produce symbol files for these variant names so that
     # the Breakpad symbol server will have something to return for stacks that
     # travel through these modules.
     case "${SRC_NAME}" in
       "${SRC_APP_NAME}.app")
@@ skipping 24 matching lines @@
   # the .dSYM archive if a new one is needed
   DSYMS[${#DSYMS[@]}]="${DSYM_NAME}"
 done
 
 # Create the archive of .dSYM bundles.
 if [ ! -e "${DSYM_TAR_PATH}" ] ; then
   # Change directory so that absolute paths aren't included in the archive.
   (cd "${BUILT_PRODUCTS_DIR}" &&
    tar -jcf "${DSYM_TAR_PATH}" "${DSYMS[@]}")
 fi