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

Unified Diff: media/tools/layout_tests/trend_graph.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: 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/trend_graph.py
diff --git a/media/tools/layout_tests/trend_graph.py b/media/tools/layout_tests/trend_graph.py
index a2bcafa888fabc1b31bf6fc989d3d1c4cacc7213..fb94d03ce708897641abe7f917376be0ce4e8cfc 100644
--- a/media/tools/layout_tests/trend_graph.py
+++ b/media/tools/layout_tests/trend_graph.py
@@ -5,7 +5,6 @@
"""A module for manipulating trend graph with analyzer result history."""
import os
-import sys
import layouttest_analyzer_helpers
@@ -16,7 +15,7 @@ LINE_INSERT_POINT_FOR_NUMBERS = r'// insert 1'
LINE_INSERT_POINT_FOR_PASSING_RATE = r'// insert 2'
-class TrendGraph(object):
+class TrendGraph:
dennis_jeffrey 2012/02/29 17:50:32 curious: why did you remove the "(object)" from he
imasaki1 2012/03/02 19:12:07 Mistake. I bring it back
"""A class to manage trend graph which is using Google Visualization APIs.
Google Visualization API (http://code.google.com/apis/chart/interactive/docs/
@@ -52,7 +51,7 @@ class TrendGraph(object):
# After the below conversion, for example, in the case of the year 2008,
# |datetime_string| ranges from '2008,0,1,0,0,00' to '2008,11,31,23,59,99'.
str_list = datetime_string.split(',')
- str_list[1] = str(int(str_list[1])-1) # month
+ str_list[1] = str(int(str_list[1])-1) # Month
datetime_string = ','.join(str_list)
for key in ['whole', 'skip', 'nonskip']:
joined_str += str(len(data_map[key][0])) + ','

Powered by Google App Engine
This is Rietveld 408576698