Save .dSYMs and Breakpad symbol files for ppGoogleNaClPluginChrome.plugin and,

if present, PDF.plugin.

BUG=58310
TEST=none
Review URL: http://codereview.chromium.org/7659007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97212 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 20 insertions, 8 deletions

diff --git a/chrome/tools/build/mac/dump_product_syms b/chrome/tools/build/mac/dump_product_syms
index 477429c..58d28e8 100755
--- a/chrome/tools/build/mac/dump_product_syms
+++ b/chrome/tools/build/mac/dump_product_syms
@@ -49,14 +49,26 @@ DSYM_TAR_PATH="${BUILT_PRODUCTS_DIR}/${SRC_APP_NAME}.dSYM.tar.bz2"
 
 declare -a DSYMS
 
-for SRC_NAME in "${SRC_APP_NAME}.app" \
-                "${SRC_APP_NAME} Framework.framework" \
-                "${SRC_APP_NAME} Helper.app" \
-                "crash_inspector" \
-                "crash_report_sender.app" \
-                "ffmpegsumo.so" \
-                "libplugin_carbon_interpose.dylib" \
-                "remoting_host_plugin.plugin" ; do
+# Everything in SRC_NAMES is required. It's an error for any of these files
+# to be missing.
+SRC_NAMES=(
+    "${SRC_APP_NAME}.app"
+    "${SRC_APP_NAME} Framework.framework"
+    "${SRC_APP_NAME} Helper.app"
+    "crash_inspector"
+    "crash_report_sender.app"
+    "ffmpegsumo.so"
+    "libplugin_carbon_interpose.dylib"
+    "ppGoogleNaClPluginChrome.plugin"
+    "remoting_host_plugin.plugin"
+)
+
+# PDF.plugin is optional. Only include it if present.
+if [[ -e "${BUILT_PRODUCTS_DIR}/PDF.plugin" ]]; then
+  SRC_NAMES[${#SRC_NAMES[@]}]="PDF.plugin"
+fi
+
+for SRC_NAME in "${SRC_NAMES[@]}"; do
   # 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