Commit 92d69f77 authored by Scott Graham's avatar Scott Graham

Add dbghelp.dll that works on XP, and include in base.isolate

This fixes the hang that caused the addition of the early out in
stack_trace_win.cc.

Removing that in turn fixes symbolization of stack traces on XP bots.

R=cpu@chromium.org
BUG=460506

Review URL: https://codereview.chromium.org/943933005

Cr-Commit-Position: refs/heads/master@{#317651}
parent 9e7cf706
...@@ -30,6 +30,14 @@ ...@@ -30,6 +30,14 @@
], ],
}, },
}], }],
['OS=="win"', {
# Required for base/stack_trace_win.cc to symbolize correctly.
'variables': {
'files': [
'../build/win/dbghelp_xp/dbghelp.dll',
],
},
}],
['OS=="win" and asan==1 and component=="shared_library"', { ['OS=="win" and asan==1 and component=="shared_library"', {
'variables': { 'variables': {
'files': [ 'files': [
......
...@@ -152,10 +152,6 @@ class SymbolContext { ...@@ -152,10 +152,6 @@ class SymbolContext {
init_error_ = ERROR_SUCCESS; init_error_ = ERROR_SUCCESS;
// Work around a mysterious hang on Windows XP.
if (base::win::GetVersion() < base::win::VERSION_VISTA)
return;
// When transferring the binaries e.g. between bots, path put // When transferring the binaries e.g. between bots, path put
// into the executable will get off. To still retrieve symbols correctly, // into the executable will get off. To still retrieve symbols correctly,
// add the directory of the executable to symbol search path. // add the directory of the executable to symbol search path.
......
This dbghelp.dll is the redistributable version from the Windows 7 SDK, the
last one to work on Windows XP.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment