Enabled test case filter for rankings computation.

2 files changed, 14 insertions, 5 deletions

diff --git a/web/getstats/main.js b/web/getstats/main.js
index ac9dc88..78b5ca5 100644
--- a/web/getstats/main.js
+++ b/web/getstats/main.js
@@ -185,9 +185,8 @@ function updateActions() {
             encodeURIComponent(context1_["platform"]);
         url_rankings += "&branch=" + encodeURIComponent(context1_["branch"]);
         url_rankings += "&sha1=" + context1_["sha1"];
+        url_rankings += "&testcasefilter=" + testCaseFilter;
         url_rankings += "&maxsize=" + $("#max_rank_size option:selected").val();
-        // ### Test case filter unsupported for now:
-        //url_rankings += "&testcasefilter=" + testCaseFilter;
     } else {
         url_rankings = "";
     }
diff --git a/web/getstats/rankings.js b/web/getstats/rankings.js
index 18861c8..59a0f2b 100644
--- a/web/getstats/rankings.js
+++ b/web/getstats/rankings.js
@@ -58,7 +58,11 @@ function populateRankingTable(
         } else {
             html += "<td style=\"color:red\">n/a</td>";
         }
-        html += "<td style=\"text-align:right\">" + val + "</td>";
+        if (val >= 0) {
+            html += "<td style=\"text-align:right\">" + val + "</td>";
+        } else {
+            html += "<td style=\"color:red\">n/a</td>";
+        }
         html += "<td class=\"metric\">" + metric + "</td>";
         html += "<td class=\"benchmark\">" + benchmark + "</td>";
         html += "</tr>";
@@ -67,7 +71,8 @@ function populateRankingTable(
     $(tableSel + " > tbody:last").append(html);
 }
 
-function fetchRankings(database, host, platform, branch, sha1, maxsize) {
+function fetchRankings(
+    database, host, platform, branch, sha1, testCaseFilter, maxsize) {
     updateStatus("fetching rankings ...", true);
 
     query = "?db=" + database +
@@ -77,6 +82,8 @@ function fetchRankings(database, host, platform, branch, sha1, maxsize) {
         "&branch=" + encodeURIComponent(branch) +
         "&sha1=" + sha1 +
         "&maxsize=" + maxsize;
+    if (testCaseFilter != "")
+        query += "&testcasefilter=" + encodeURIComponent(testCaseFilter);
 
     url = "http://" + location.host + "/cgi-bin/getstatswrapper" + query;
     //alert("url: >" + url + "<");
@@ -245,9 +252,12 @@ $(document).ready(function() {
         return;
     }
 
+    var testCaseFilter = extractArg(args, "testcasefilter"); // optional
+
     $("#div_tsbm_border").css("display", "none");
     $("#div_tsbm").css("display", "none");
     $("#div_rankings").css("display", "none");
 
-    fetchRankings(database, host, platform, branch, sha1, maxsize);
+    fetchRankings(
+        database, host, platform, branch, sha1, testCaseFilter, maxsize);
 });