Add a dump-file option to process_dumps_linux.py

chrome/tools/process_dumps_linux.py

 #!/usr/bin/python
 # Copyright (c) 2009 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.
 
 """A tool to collect crash signatures for Chrome builds on Linux."""
 
 import fnmatch
 import optparse
 import os
@@ skipping 204 matching lines @@
   symbol_dir = options.symbol_dir
   if not options.symbol_dir:
     symbol_dir = os.curdir
   symbol_dir = os.path.abspath(os.path.expanduser(symbol_dir))
   symbol_file = os.path.join(symbol_dir, symbol_file)
   if not os.path.exists(symbol_file):
     print 'Cannot find symbols.'
     return 1
   symbol_time = os.path.getmtime(symbol_file)
 
-  dump_dir = options.dump_dir
-  if not dump_dir:
-    dump_dir = GetCrashDumpDir()
-  if not dump_dir:
-    print 'Cannot find dump dir.'
-    return 1
+  dump_files = []
+  if options.dump_file:
+    dump_files.append(options.dump_file)
+  else:
+    dump_dir = options.dump_dir
+    if not dump_dir:
+      dump_dir = GetCrashDumpDir()
+    if not dump_dir:
+      print 'Cannot find dump files.'
+      return 1
+    for dump_file in LocateFiles(pattern='*.dmp', root=dump_dir):
+      file_time = os.path.getmtime(dump_file)
+      if file_time < symbol_time:
+        # Ignore dumps older than symbol file.
+        continue
+      dump_files.append(dump_file)
 
   temp_dir = tempfile.mkdtemp(suffix='chromedump')
   if not VerifySymbolAndCopyToTempDir(symbol_file, temp_dir):
     print 'Cannot parse symbols.'
     shutil.rmtree(temp_dir)
     return 1
 
   dump_count = 0
-  for dump_file in LocateFiles(pattern='*.dmp', root=dump_dir):
-    file_time = os.path.getmtime(dump_file)
-    if file_time < symbol_time:
-      # Ignore dumps older than symbol file.
-      continue
+  for dump_file in dump_files:
     processed_dump_file = ProcessDump(dump_file, temp_dir)
     if not processed_dump_file:
       continue
     print '-------------------------'
     print GetStackTrace(processor_bin, temp_dir, processed_dump_file)
     print
     os.remove(processed_dump_file)
     dump_count += 1
 
   shutil.rmtree(temp_dir)
   print '%s dumps found' % dump_count
   return 0
 
 
 if '__main__' == __name__:
   parser = optparse.OptionParser()
   parser.add_option('', '--processor-dir', type='string', default='',
                     help='The directory where the processor is installed. '
                          'The processor is used to get stack trace from dumps. '
                           'Searches $PATH by default')
+  parser.add_option('', '--dump-file', type='string', default='',
+                    help='The path of the dump file to be processed.'
+                         'Overwrites dump-path')
   parser.add_option('', '--dump-dir', type='string', default='',
                     help='The directory where dump files are stored. '
-                         'Searches the Chromium crash directory by default.')
+                         'Searches this directory if dump-file is not'

[Lei Zhang, 2009/10/28 23:17:25]

nit, you need an extra space after not.

+                         'specified. Default is the Chromium crash directory.')
   parser.add_option('', '--symbol-dir', default='',
                     help='The directory with the symbols file. [Required]')
   parser.add_option('', '--architecture', type='int', default=None,
                     help='Override automatic x86/x86-64 detection. '
                          'Valid values are 32 and 64')
 
   (options, args) = parser.parse_args()
 
   if sys.platform == 'linux2':
     sys.exit(main_linux(options, args))
   else:
     sys.exit(1)