[Dashboard] The revision rows for aggregate results should no longer include Chromium...
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2013 08:28:10 +0000 (08:28 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2013 08:28:10 +0000 (08:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114760

Reviewed by Benjamin Poulain.

With the Chromium builders no longer present there's no need to display the Chromium revision range under the
aggregate results. Only the first half of the htmlForRevisionRows method (the one returning the table row listing the
WebKit revisions) is now relevant - the method is therefor removed and its callsites updated to directly call the
htmlForTableRow method that returns the table row containing all the relevant WebKit revisions.

* TestResultServer/static-dashboards/aggregate_results.js:
(htmlForSummaryTable):
(htmlForTestType):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@148725 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestResultServer/static-dashboards/aggregate_results.js

index 70920e0..f39a929 100644 (file)
@@ -1,5 +1,21 @@
 2013-04-19  Zan Dobersek  <zdobersek@igalia.com>
 
 2013-04-19  Zan Dobersek  <zdobersek@igalia.com>
 
+        [Dashboard] The revision rows for aggregate results should no longer include Chromium revisions
+        https://bugs.webkit.org/show_bug.cgi?id=114760
+
+        Reviewed by Benjamin Poulain.
+
+        With the Chromium builders no longer present there's no need to display the Chromium revision range under the
+        aggregate results. Only the first half of the htmlForRevisionRows method (the one returning the table row listing the
+        WebKit revisions) is now relevant - the method is therefor removed and its callsites updated to directly call the
+        htmlForTableRow method that returns the table row containing all the relevant WebKit revisions.
+
+        * TestResultServer/static-dashboards/aggregate_results.js:
+        (htmlForSummaryTable):
+        (htmlForTestType):
+
+2013-04-19  Zan Dobersek  <zdobersek@igalia.com>
+
         [Dashboard] Remove Chromium-specific cases from the construction of the chart HTML for aggregate results
         https://bugs.webkit.org/show_bug.cgi?id=114759
 
         [Dashboard] Remove Chromium-specific cases from the construction of the chart HTML for aggregate results
         https://bugs.webkit.org/show_bug.cgi?id=114759
 
index 99baefc..0bec99a 100644 (file)
@@ -184,12 +184,6 @@ function chart(title, values, revisionLabel, startRevision, endRevision)
     return '<img src="' + url + '">';
 }
 
     return '<img src="' + url + '">';
 }
 
-function htmlForRevisionRows(results, numColumns)
-{
-    return htmlForTableRow('WebKit Revision', results[WEBKIT_REVISIONS_KEY].slice(0, numColumns)) +
-        htmlForTableRow('Chrome Revision', results[CHROME_REVISIONS_KEY].slice(0, numColumns));
-}
-
 function wrapHTMLInTable(description, html)
 {
     return '<h3>' + description + '</h3><table><tbody>' + html + '</tbody></table>';
 function wrapHTMLInTable(description, html)
 {
     return '<h3>' + description + '</h3><table><tbody>' + html + '</tbody></table>';
@@ -205,7 +199,7 @@ function htmlForSummaryTable(results, numColumns)
         // Round to the nearest tenth of a percent.
         percent.push(Math.round(percentage * 10) / 10 + '%');
     }
         // Round to the nearest tenth of a percent.
         percent.push(Math.round(percentage * 10) / 10 + '%');
     }
-    var html = htmlForRevisionRows(results, numColumns) +
+    var html = htmlForTableRow('WebKit Revision', results[WEBKIT_REVISIONS_KEY].slice(0, numColumns)) +
         htmlForTableRow('Percent passed', percent) +
         htmlForTableRow('Failures (deduped)', fixable) +
         htmlForTableRow('Fixable Tests', allFixable);
         htmlForTableRow('Percent passed', percent) +
         htmlForTableRow('Failures (deduped)', fixable) +
         htmlForTableRow('Fixable Tests', allFixable);
@@ -231,7 +225,7 @@ function valuesPerExpectation(counts, numColumns)
 function htmlForTestType(results, key, description, numColumns)
 {
     var counts = results[key];
 function htmlForTestType(results, key, description, numColumns)
 {
     var counts = results[key];
-    var html = htmlForRevisionRows(results, numColumns);
+    var html = htmlForTableRow('WebKit Revision', results[WEBKIT_REVISIONS_KEY].slice(0, numColumns));
     var values = valuesPerExpectation(counts, numColumns);
     for (var expectation in values)
         html += htmlForTableRow(expectationsMap()[expectation], values[expectation]);
     var values = valuesPerExpectation(counts, numColumns);
     for (var expectation in values)
         html += htmlForTableRow(expectationsMap()[expectation], values[expectation]);