diff options
| author | imasaki@google.com <imasaki@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-07 17:27:16 +0000 |
|---|---|---|
| committer | imasaki@google.com <imasaki@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-07 17:27:16 +0000 |
| commit | c366498bdd1969210e22e2bb0ef2ed2a8d669c34 (patch) | |
| tree | 7d6e70c1e35dc26c67230c587f5a4f1faff38c64 /media | |
| parent | fad554f49bb42f2b91fbf557ed24767a013c6d83 (diff) | |
| download | chromium_src-c366498bdd1969210e22e2bb0ef2ed2a8d669c34.zip chromium_src-c366498bdd1969210e22e2bb0ef2ed2a8d669c34.tar.gz chromium_src-c366498bdd1969210e22e2bb0ef2ed2a8d669c34.tar.bz2 | |
[Layout Test Analzyer] show changed test names all the time (even when the
total number of tests does not change from previous ).
BUG=126301
TEST=make sure layout test analzyer still runs as before.
Review URL: https://chromiumcodereview.appspot.com/10383018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135664 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media')
| -rw-r--r-- | media/tools/layout_tests/layouttest_analyzer_helpers.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/media/tools/layout_tests/layouttest_analyzer_helpers.py b/media/tools/layout_tests/layouttest_analyzer_helpers.py index bddb862..d4cda6d 100644 --- a/media/tools/layout_tests/layouttest_analyzer_helpers.py +++ b/media/tools/layout_tests/layouttest_analyzer_helpers.py @@ -87,10 +87,10 @@ class AnalyzerResultMap: a string in HTML format (with colors) to show difference between two analyzer results. """ - diff = len(diff_map_element[0]) - len(diff_map_element[1]) - if diff == 0: + if not diff_map_element[0] and not diff_map_element[1]: return 'No Change' color = '' + diff = len(diff_map_element[0]) - len(diff_map_element[1]) if diff > 0 and type_str != 'whole': color = 'red' else: @@ -98,7 +98,10 @@ class AnalyzerResultMap: diff_sign = '' if diff > 0: diff_sign = '+' - whole_str = '<font color="%s">%s%d</font>' % (color, diff_sign, diff) + if not diff: + whole_str = 'No Change' + else: + whole_str = '<font color="%s">%s%d</font>' % (color, diff_sign, diff) colors = ['red', 'green'] if type_str == 'whole': # Bug 107773 - when we increase the number of tests, |