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

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

Issue 7778048: Add README file and parametrize result directory in layout test analzyer tool. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Correct mistake in the last upload. Created 9 years, 4 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 c39fff88343d3e8e347d454a74465fc5b11a2dbf..9261562028eecaa4f5480205f04d5ce26cdeb41b 100644
--- a/media/tools/layout_tests/trend_graph.py
+++ b/media/tools/layout_tests/trend_graph.py
@@ -45,6 +45,11 @@ class TrendGraph(object):
as values for graph annotation.
"""
joined_str = ''
+ # For a date format in GViz, month is shifted (e.g., '2008,1,1' means
dennis_jeffrey 2011/09/01 00:29:13 Maybe pick a slightly different example than "2008
imasaki1 2011/09/01 03:08:23 Done.
+ # Feb. 1st, 2008).
+ str_list = datetime_string.split(',')
dennis_jeffrey 2011/09/01 00:29:13 Just to clarify: if we have 'Jan. 1, 2008', does t
imasaki1 2011/09/01 03:08:23 The input string for 'Jan. 1, 2008' is '2008,0,1'
dennis_jeffrey 2011/09/01 16:56:42 Ok, if that is true, then won't line 51 below subt
imasaki1 2011/09/03 19:35:13 Sorry I was not clear. Input string ranges from (
+ str_list[1] = str(int(str_list[1])-1) # month
+ datetime_string = ','.join(str_list)
for key in ['whole', 'skip', 'nonskip']:
joined_str += ','.join(data_map[key]) + ','
new_line_for_numbers = ' [new Date(%s),%s],\n' % (datetime_string,
« media/tools/layout_tests/README ('K') | « media/tools/layout_tests/layouttest_analyzer.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698