[Dashboard] Remove Chromium-specific cases from the construction of the chart HTML...
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2013 08:25:44 +0000 (08:25 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2013 08:25:44 +0000 (08:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114759

Reviewed by Benjamin Poulain.

Aggregate results are not viewable for Chromium builders anymore, meaning the WebKit revisions are shown by default.
This means the code in the chartHTML method can be simplified, defaulting the revision key to WEBKIT_REVISIONS_KEY
and the revision label to 'WebKit Revision'.

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

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

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

index dc77a59..70920e0 100644 (file)
@@ -1,3 +1,17 @@
+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
+
+        Reviewed by Benjamin Poulain.
+
+        Aggregate results are not viewable for Chromium builders anymore, meaning the WebKit revisions are shown by default.
+        This means the code in the chartHTML method can be simplified, defaulting the revision key to WEBKIT_REVISIONS_KEY
+        and the revision label to 'WebKit Revision'.
+
+        * TestResultServer/static-dashboards/aggregate_results.js:
+        (chartHTML):
+
 2013-04-18  Xuefei Ren  <xren@blackberry.com>
 
         remove build warning(unused parameter)
index 5fb7e91..99baefc 100644 (file)
@@ -104,11 +104,10 @@ function rawValuesHTML(results, numColumns)
 
 function chartHTML(results, numColumns)
 {
-    var shouldShowWebKitRevisions = isTipOfTreeWebKitBuilder();
-    var revisionKey = shouldShowWebKitRevisions ? WEBKIT_REVISIONS_KEY : CHROME_REVISIONS_KEY;
+    var revisionKey = WEBKIT_REVISIONS_KEY;
+    var revisionLabel = "WebKit Revision";
     var startRevision = results[revisionKey][numColumns - 1];
     var endRevision = results[revisionKey][0];
-    var revisionLabel = shouldShowWebKitRevisions ? "WebKit Revision" : "Chromium Revision";
 
     var fixable = results[FIXABLE_COUNT_KEY].slice(0, numColumns);
     var html = chart("Total failing", {"": fixable}, revisionLabel, startRevision, endRevision);