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

Unified Diff: media/tools/layout_tests/layouttest_analyzer.py

Issue 7789008: Added an option to append text at the end of the status email n layout test analyzer tool. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove graph/graph.html Created 9 years, 3 months 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 side-by-side diff with in-line comments
Download patch
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')

Powered by Google App Engine
This is Rietveld 408576698