Chromium Code Reviews| Index: media/tools/layout_tests/layouttest_analyzer.py |
| diff --git a/media/tools/layout_tests/layouttest_analyzer.py b/media/tools/layout_tests/layouttest_analyzer.py |
| index b7045a689b41fc996e0f92b8e2264034b23e9608..0ccd437b8ef67902a3506b7a220303a5a75dd7e2 100644 |
| --- a/media/tools/layout_tests/layouttest_analyzer.py |
| +++ b/media/tools/layout_tests/layouttest_analyzer.py |
| @@ -79,6 +79,12 @@ def parse_option(): |
| help=('Name of result directory location ' |
| '(default to %default)'), |
| default=DEFAULT_RESULT_DIR) |
| + option_parser.add_option('-b', '--email-insert-text-file-location', |
| + dest='email_insert_text_file_location', |
| + help=('File location of the email insert text. ' |
| + 'The text is included in the status email. ' |
| + '(default to %default)'), |
|
dennis_jeffrey
2011/09/06 19:25:55
You might want to clarify that in the default case
imasaki1
2011/09/06 20:26:10
Done.
|
| + default=None) |
| return option_parser.parse_args()[0] |
| @@ -111,11 +117,18 @@ def main(): |
| anno_map[row[0]] = row[1] |
| file_object.close() |
| + appended_text_to_email = '' |
| + if options.email_insert_text_file_location: |
| + file_object = open(options.email_insert_text_file_location) |
|
dennis_jeffrey
2011/09/06 19:25:55
We should check to make sure the file exists befor
imasaki1
2011/09/06 20:26:10
Done. Also changed annotation file reading part.
|
| + appended_text_to_email = ''.join(file_object.readlines()) |
| + file_object.close() |
| + |
| layouttest_analyzer_helpers.SendStatusEmail(prev_time, analyzer_result_map, |
| prev_analyzer_result_map, |
| anno_map, |
| options.receiver_email_address, |
| - options.test_group_name) |
| + options.test_group_name, |
| + appended_text_to_email) |
| if not options.debug: |
| # Save the current result. |
| date = start_time.strftime('%Y-%m-%d-%H') |