Chromium Code Reviews Chromium Code Reviews
Issue 7850023:
Add a command-line option about status email in the layout test analyzer. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| 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 3033a1daed49f31fc51069276f8a25cf3d7826bb..7fe90b0fb265fd0d68f0c856cb8c7f9ef9c9631f 100644 |
| --- a/media/tools/layout_tests/layouttest_analyzer.py |
| +++ b/media/tools/layout_tests/layouttest_analyzer.py |
| @@ -84,6 +84,13 @@ def parse_option(): |
| '(default to %default and no text is ' |
| 'appended in that case.)'), |
| default=None) |
| + option_parser.add_option('-c', '--email-only-change-mode', |
| + dest='email_only_change_mode', |
| + help=('With this mode, email is only sent out ' |
|
dennis_jeffrey
2011/09/09 00:56:29
remove 'only' in this line; it's already stated in
imasaki1
2011/09/09 03:03:45
Done.
|
| + 'only when there is a change in the ' |
| + 'analyzer result comparing to the previous ' |
|
dennis_jeffrey
2011/09/09 00:56:29
'comparing' --> 'compared'
imasaki1
2011/09/09 03:03:45
Done.
|
| + 'result (off by default)'), |
| + action='store_true', default=False) |
| return option_parser.parse_args()[0] |
| @@ -139,7 +146,8 @@ def main(): |
| anno_map, |
| options.receiver_email_address, |
| options.test_group_name, |
| - appended_text_to_email) |
| + appended_text_to_email, |
| + options.email_only_change_mode) |
| if not options.debug: |
| # Save the current result. |
| date = start_time.strftime('%Y-%m-%d-%H') |
