Moved multiple implementations to common file.

3 files changed, 21 insertions, 42 deletions

diff --git a/web/getstats/rankings.js b/web/getstats/rankings.js
index 845449e..bda3eed 100644
--- a/web/getstats/rankings.js
+++ b/web/getstats/rankings.js
@@ -11,27 +11,6 @@ function selectRankingTable() {
     $("#rankingTable_nrows").text(nrows + ((nrows == 1) ? " row" : " rows"));
 }
 
-// ### Same function in tsstats.js -> move both to a common place? tsbmbody.js?
-// 2 B DONE!
-function clickBenchmarkCheckbox(
-    cb, tableSel, database, host, platform, branch, sha11, sha12, benchmark,
-    metric, difftol, durtolmin, durtolmax) {
-
-    if (cb.checked) {
-        // Deselect all other benchmarks:
-        $(tableSel + " input").attr("checked", false);
-        cb.checked = true;
-        index = cb.id.match(/bm_cb:(\d+)/)[1];
-
-        // Fetch and plot time series:
-        fetchTimeSeries(
-            database, host, platform, branch, sha11, sha12, benchmark, metric,
-            difftol, durtolmin, durtolmax, true);
-    } else {
-        clearPlot();
-    }
-}
-
 function populateRankingTable(
     tableSel, rankings, database, host, platform, branch, sha11, sha12,
     difftol, durtolmin, durtolmax, bmarkId2Name, metricId2Name) {
diff --git a/web/getstats/tsbmbody.js b/web/getstats/tsbmbody.js
index cdec563..758c607 100644
--- a/web/getstats/tsbmbody.js
+++ b/web/getstats/tsbmbody.js
@@ -1272,3 +1272,24 @@ function setSnapshots(sshots) {
     updateMainContextInterval(
         sshots[0], sshots[sshots.length - 1], sshots.length);
 }
+
+// Handles clicking a benchmark checkbox. At most one benchmark may
+// be selected at any time.
+function clickBenchmarkCheckbox(
+    cb, tableSel, database, host, platform, branch, sha11, sha12, benchmark,
+    metric, difftol, durtolmin, durtolmax) {
+
+    if (cb.checked) {
+        // Deselect all other benchmarks:
+        $(tableSel + " input").attr("checked", false);
+        cb.checked = true;
+        index = cb.id.match(/bm_cb:(\d+)/)[1];
+
+        // Fetch and plot time series:
+        fetchTimeSeries(
+            database, host, platform, branch, sha11, sha12, benchmark, metric,
+            difftol, durtolmin, durtolmax, true);
+    } else {
+        clearPlot();
+    }
+}
diff --git a/web/getstats/tsstats.js b/web/getstats/tsstats.js
index af847bd..1ee26f4 100644
--- a/web/getstats/tsstats.js
+++ b/web/getstats/tsstats.js
@@ -12,27 +12,6 @@ function selectPbmTable() {
     $("#pbmTable_nrows").text(nrows + ((nrows == 1) ? " row" : " rows"));
 }
 
-// Handles clicking a benchmark checkbox. At most one benchmark may
-// be selected at any time.
-function clickBenchmarkCheckbox(
-    cb, tableSel, database, host, platform, branch, sha11, sha12, benchmark,
-    metric, difftol, durtolmin, durtolmax) {
-
-    if (cb.checked) {
-        // Deselect all other benchmarks:
-        $(tableSel + " input").attr("checked", false);
-        cb.checked = true;
-        index = cb.id.match(/bm_cb:(\d+)/)[1];
-
-        // Fetch and plot time series:
-        fetchTimeSeries(
-            database, host, platform, branch, sha11, sha12, benchmark, metric,
-            difftol, durtolmin, durtolmax, true);
-    } else {
-        clearPlot();
-    }
-}
-
 function changeMagnitudeScore(change) {
     var maxChange = 2.0;
     var absChange = (change < 1.0) ? (1.0 / change) : change;