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

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

Issue 9476021: Updating Layout test analyzer to add control to show issue detail or not. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Modification based on CR comments. Created 8 years, 10 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_runner.py
diff --git a/media/tools/layout_tests/layouttest_analyzer_runner.py b/media/tools/layout_tests/layouttest_analyzer_runner.py
old mode 100755
new mode 100644
index 868c746fe50b0ae23bbab6e96f3cd0e976c42519..eec0f4d73bca0d2d187d4a7caabac7a8e46756cf
--- a/media/tools/layout_tests/layouttest_analyzer_runner.py
+++ b/media/tools/layout_tests/layouttest_analyzer_runner.py
@@ -1,5 +1,5 @@
#!/usr/bin/env python
-# Copyright (c) 2011 The Chromium Authors. All rights reserved.
+# Copyright (c) 2012 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.
@@ -75,6 +75,12 @@ def ParseOption():
'analyzer result compared to the previous '
'result (off by default)'),
action='store_true', default=False)
+ option_parser.add_option('-z', '--issue-detail-mode',
+ dest='issue_detail_mode',
+ help=('With this mode, email includes issue details'
+ ' including links to the flakiness dashboard'
+ ' (off by default)'),
+ action='store_true', default=False)
return option_parser.parse_args()[0]
@@ -194,6 +200,8 @@ def main():
cmd += ' -b ' + options.email_appended_text_file_location
if options.email_only_change_mode:
cmd += ' -c '
+ if options.issue_detail_mode:
+ cmd += ' -z '
print 'Running ' + cmd
proc = Popen(cmd, shell=True)
proc.communicate()
« no previous file with comments | « media/tools/layout_tests/layouttest_analyzer_helpers_unittest.py ('k') | media/tools/layout_tests/layouttests.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698