diff options
author | glider <glider@chromium.org> | 2015-02-06 09:42:27 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-02-06 17:43:30 +0000 |
commit | 9d9193404a307ba7cf79db460a0f988bc349bc4e (patch) | |
tree | 777b6f6f18ece035c45a965fa87d8d0a1f58f935 /chrome | |
parent | 1490217011f83c5b46253d24d913e9bfb98f8005 (diff) | |
download | chromium_src-9d9193404a307ba7cf79db460a0f988bc349bc4e.zip chromium_src-9d9193404a307ba7cf79db460a0f988bc349bc4e.tar.gz chromium_src-9d9193404a307ba7cf79db460a0f988bc349bc4e.tar.bz2 |
Fix report symbolization on swarming bots.
According to issue 444835 there were at least two problems with debug info on
the swarming bots:
- the .dSYM bundles for the test binaries and other applications weren't
uploaded to swarming servers;
- for some binaries (e.g. Chromium Framework) their names were replaced with
SHA-1 hashes that confused llvm-symbolizer and made finding the .dSYM bundles
impossible.
This CL adds the necessary .dSYM paths to .isolate files for the tests
currently being ran on swarming bots (for other tests their .dSYM paths will be
added as needed).
It also extends tools/valgrind/asan/asan_symbolize.py with a binary name filter
that replaces the hashes with the corresponding file names in the product
directory.
BUG=444835
R=earthdok@chromium.org,maruel@chromium.org,csharp@chromium.org
Review URL: https://codereview.chromium.org/859293002
Cr-Commit-Position: refs/heads/master@{#315059}
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser_tests.isolate | 7 | ||||
-rw-r--r-- | chrome/chrome.isolate | 11 | ||||
-rw-r--r-- | chrome/interactive_ui_tests.isolate | 7 | ||||
-rw-r--r-- | chrome/unit_tests.isolate | 7 |
4 files changed, 32 insertions, 0 deletions
diff --git a/chrome/browser_tests.isolate b/chrome/browser_tests.isolate index 912fbca..2624378c 100644 --- a/chrome/browser_tests.isolate +++ b/chrome/browser_tests.isolate @@ -208,6 +208,13 @@ ], }, }], + ['OS=="mac" and asan==1', { + 'variables': { + 'files': [ + '<(PRODUCT_DIR)/browser_tests.dSYM/', + ], + }, + }], ['OS=="win" and libpeer_target_type=="loadable_module"', { 'variables': { 'files': [ diff --git a/chrome/chrome.isolate b/chrome/chrome.isolate index 64ae230..061cc23 100644 --- a/chrome/chrome.isolate +++ b/chrome/chrome.isolate @@ -77,6 +77,17 @@ ], }, }], + ['OS=="mac" and asan==1', { + 'variables': { + 'files': [ + '<(PRODUCT_DIR)/<(mac_product_name) Framework.framework.dSYM/', + '<(PRODUCT_DIR)/<(mac_product_name) Helper.app.dSYM/', + '<(PRODUCT_DIR)/<(mac_product_name).app.dSYM/', + '<(PRODUCT_DIR)/exif.so.dSYM/', + '<(PRODUCT_DIR)/ffmpegsumo.so.dSYM/', + ], + }, + }], ['OS=="win"', { 'variables': { 'files': [ diff --git a/chrome/interactive_ui_tests.isolate b/chrome/interactive_ui_tests.isolate index d968a7d..1d78389 100644 --- a/chrome/interactive_ui_tests.isolate +++ b/chrome/interactive_ui_tests.isolate @@ -70,6 +70,13 @@ ], }, }], + ['OS=="mac" and asan==1', { + 'variables': { + 'files': [ + '<(PRODUCT_DIR)/interactive_ui_tests.dSYM/', + ], + }, + }], ['OS=="mac" or OS=="win"', { 'variables': { 'command': [ diff --git a/chrome/unit_tests.isolate b/chrome/unit_tests.isolate index a03f8a5..3c12a33 100644 --- a/chrome/unit_tests.isolate +++ b/chrome/unit_tests.isolate @@ -80,6 +80,13 @@ ], }, }], + ['OS=="mac" and asan==1', { + 'variables': { + 'files': [ + '<(PRODUCT_DIR)/unit_tests.dSYM/', + ], + }, + }], ['OS=="linux" or OS=="mac"', { 'variables': { 'read_only': 1, |