Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1557)

Side by Side Diff: snapshot/win/end_to_end_test.py

Issue 1430633006: Fix exception location in z7 test on older bots (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 2
3 # Copyright 2015 The Crashpad Authors. All rights reserved. 3 # Copyright 2015 The Crashpad Authors. All rights reserved.
4 # 4 #
5 # Licensed under the Apache License, Version 2.0 (the "License"); 5 # Licensed under the Apache License, Version 2.0 (the "License");
6 # you may not use this file except in compliance with the License. 6 # you may not use this file except in compliance with the License.
7 # You may obtain a copy of the License at 7 # You may obtain a copy of the License at
8 # 8 #
9 # http://www.apache.org/licenses/LICENSE-2.0 9 # http://www.apache.org/licenses/LICENSE-2.0
10 # 10 #
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 # stack entries will not be available. This confirms that we have a mostly 213 # stack entries will not be available. This confirms that we have a mostly
214 # valid dump, but that the stack was omitted. 214 # valid dump, but that the stack was omitted.
215 out.Check(r'self_destroying_program!crashpad::`anonymous namespace\'::' 215 out.Check(r'self_destroying_program!crashpad::`anonymous namespace\'::'
216 r'FreeOwnStackAndBreak.*\nquit:', 216 r'FreeOwnStackAndBreak.*\nquit:',
217 'at correct location, no additional stack entries') 217 'at correct location, no additional stack entries')
218 218
219 if z7_dump_path: 219 if z7_dump_path:
220 out = CdbRun(cdb_path, z7_dump_path, '.ecxr;lm') 220 out = CdbRun(cdb_path, z7_dump_path, '.ecxr;lm')
221 out.Check('This dump file has an exception of interest stored in it', 221 out.Check('This dump file has an exception of interest stored in it',
222 'captured exception in z7 module') 222 'captured exception in z7 module')
223 out.Check(r'z7_test\+0x[0-8a-f]+:', 'exception in z7 at correct location') 223 # Older versions of cdb display relative to exports for /Z7 modules, newer
224 # ones just display the offset.
225 out.Check(r'z7_test(!CrashMe\+0xe|\+0x100e):',
226 'exception in z7 at correct location')
224 out.Check(r'z7_test C \(codeview symbols\) z7_test.dll', 227 out.Check(r'z7_test C \(codeview symbols\) z7_test.dll',
225 'expected non-pdb symbol format') 228 'expected non-pdb symbol format')
226 229
227 230
228 def main(args): 231 def main(args):
229 try: 232 try:
230 if len(args) != 1: 233 if len(args) != 1:
231 print >>sys.stderr, 'must supply binary dir' 234 print >>sys.stderr, 'must supply binary dir'
232 return 1 235 return 1
233 236
(...skipping 30 matching lines...) Expand all
264 RunTests(cdb_path, crashy_dump_path, destroyed_dump_path, z7_dump_path, 267 RunTests(cdb_path, crashy_dump_path, destroyed_dump_path, z7_dump_path,
265 pipe_name) 268 pipe_name)
266 269
267 return 0 270 return 0
268 finally: 271 finally:
269 CleanUpTempDirs() 272 CleanUpTempDirs()
270 273
271 274
272 if __name__ == '__main__': 275 if __name__ == '__main__':
273 sys.exit(main(sys.argv[1:])) 276 sys.exit(main(sys.argv[1:]))
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698