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

Unified Diff: media/tools/layout_tests/layouttest_analyzer_helpers.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_helpers.py
diff --git a/media/tools/layout_tests/layouttest_analyzer_helpers.py b/media/tools/layout_tests/layouttest_analyzer_helpers.py
index ef5012cd23af6c2d1fde64d2a4d75c8878cdbe71..55ccf59f1b31fabfc2759b2e24d68fed20a9bc7f 100644
--- a/media/tools/layout_tests/layouttest_analyzer_helpers.py
+++ b/media/tools/layout_tests/layouttest_analyzer_helpers.py
@@ -265,7 +265,8 @@ class AnalyzerResultMap:
def SendStatusEmail(prev_time, analyzer_result_map, prev_analyzer_result_map,
- bug_anno_map, receiver_email_address, test_group_name):
+ bug_anno_map, receiver_email_address, test_group_name,
+ appended_text_to_email):
"""Send status email.
Args:
@@ -278,6 +279,8 @@ def SendStatusEmail(prev_time, analyzer_result_map, prev_analyzer_result_map,
stored.
dennis_jeffrey 2011/09/06 19:25:55 nit: indent this line by 1 fewer space
imasaki1 2011/09/06 20:26:10 Done.
receiver_email_address: receiver's email address.
test_group_name: string representing the test group name (e.g., 'media').
+ appended_text_to_email: a text which is appended at the end of the status
+ email.
"""
diff_map = analyzer_result_map.CompareToOtherResultMap(
prev_analyzer_result_map)
@@ -315,7 +318,7 @@ def SendStatusEmail(prev_time, analyzer_result_map, prev_analyzer_result_map,
localtime)
SendEmail('imasaki@chromium.org', 'Kenji Imasaki',
[receiver_email_address], ['Layout Test Analyzer Result'], subject,
- str)
+ str + appended_text_to_email)
def SendEmail(sender_email_address, sender_name, receivers_email_addresses,

Powered by Google App Engine
This is Rietveld 408576698