Chromium Code Reviews Chromium Code Reviews
Issue 7693018:
Intial checkin of layout test analyzer. (Closed)
Base URL: http://git.chromium.org/git/chromium.git@trunk| Index: media/tools/layout_tests/graph/graph.html |
| diff --git a/media/tools/layout_tests/graph/graph.html b/media/tools/layout_tests/graph/graph.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..c9908098a74fd715c961c0a48a9c1c804ca38c4d |
| --- /dev/null |
| +++ b/media/tools/layout_tests/graph/graph.html |
| @@ -0,0 +1,50 @@ |
| +<html> |
| + <head> |
| + <script type='text/javascript' src='https://www.google.com/jsapi'></script> |
| + <script type='text/javascript'> |
| + google.load('visualization', '1', {'packages':['annotatedtimeline']}); |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: put a space after the ':'
imasaki1
2011/08/25 23:57:03
Done.
|
| + google.setOnLoadCallback(drawChart); |
| + function drawChart() { |
| + var data = new google.visualization.DataTable(); |
| + data.addColumn('date', 'Date'); |
| + data.addColumn('number', '#Tests'); |
| + data.addColumn('string', 'title1'); |
| + data.addColumn('string', 'text1'); |
| + data.addColumn('number', '#Skipped tests'); |
| + data.addColumn('string', 'title2'); |
| + data.addColumn('string', 'text2'); |
| + data.addColumn('number', '#Non-skipped tests'); |
| + data.addColumn('string', 'title2'); |
| + data.addColumn('string', 'text2'); |
| + data.addRows([ |
| + // insert 1 |
|
dennis_jeffrey
2011/08/24 17:40:47
what do you mean by this comment? Same at line 30
dennis_jeffrey
2011/08/25 00:36:46
Wait a sec, I just realized why you need this in t
imasaki1
2011/08/25 23:57:03
This is the insert point of the data. The result o
imasaki1
2011/08/25 23:57:03
Well, I am not sure about it. Embedding these HTML
dennis_jeffrey
2011/08/26 19:01:26
Maybe we could check this file in as something lik
|
| + ]); |
| + var data2 = new google.visualization.DataTable(); |
| + data2.addColumn('date', 'Date'); |
| + data2.addColumn('number', 'Passing Rate '+ |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: put a space right before the '+'
imasaki1
2011/08/25 23:57:03
Done.
|
| + '(100-(#Non-skipped tests)/'+ |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: put a space right before the '+'
imasaki1
2011/08/25 23:57:03
Done.
|
| + '((#Tests)-(#Skipped tests))*100)'); |
| + data2.addColumn('string', 'title1'); |
| + data2.addColumn('string', 'text1'); |
| + data2.addRows([ |
| + // insert 2 |
| + ]); |
| + |
| + var chart = new google.visualization.AnnotatedTimeLine(document.getElementById('chart_div')); |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: have this statement span 2 lines to prevent t
imasaki1
2011/08/25 23:57:03
Done.
|
| + chart.draw(data, {displayAnnotations: true, allowHtml:true}); |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: add a space right after the second ':' in thi
imasaki1
2011/08/25 23:57:03
Done.
|
| + var chart2 = new google.visualization.AnnotatedTimeLine(document.getElementById('chart_div2')); |
| + chart2.draw(data2, {displayAnnotations: true, allowHtml:true, scaleType:'maximized'}); |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: add a space right after the second ':' in thi
imasaki1
2011/08/25 23:57:03
Done.
|
| + |
| + } |
| + </script> |
| + <title>Media Layout Test Analyzer Result</title> |
| + </head> |
| + |
| + <body> |
| + <h2>Current Statistics</h2> |
| + <h3 id="numbers">Numbers</h3> |
|
dennis_jeffrey
2011/08/24 17:40:47
nit: change the double quotes to single quotes her
imasaki1
2011/08/25 23:57:03
Done.
|
| + <div id='chart_div' style='width: 1400px; height: 480px;'></div> |
| + <h3>Test passing rate</h3> |
| + <div id='chart_div2' style='width: 1400px; height: 480px;'></div> |
| + </body> |
| +</html> |
