Added support for trace params to MakeURL. This fixes an issue where the trace= url parameter was s…

dashboard/ui/js/common.js

Index: dashboard/ui/js/common.js
diff --git a/dashboard/ui/js/common.js b/dashboard/ui/js/common.js
index d2daebb509a00ffcd3c8ebdb0ef0b8d1b1acd185..43d9d04c85499ca5834ea90a4420d62f19e8e60e 100644
--- a/dashboard/ui/js/common.js
+++ b/dashboard/ui/js/common.js
@@ -90,7 +90,19 @@ function MakeURL(params) {
   for (p in params) {
     if (!p)
       continue;
-    url = url + sep + p + '=' + params[p];
+    var new_param = '';

[tonyg, 2012/10/16 18:28:29]

nit: newParam

+    if (typeof params[p] == "object") {
+      // Repeated param
+      var repeated = []

[tonyg, 2012/10/16 18:28:29]

nit: end with semicolon

+      for (r in params[p]) {
+        repeated.push(r + '=' + params[p][r]);
+      }
+      new_param = repeated.join("&");
+    }
+    else {

[tonyg, 2012/10/16 18:28:29]

Bring this up a line

+      new_param = p + '=' + params[p];
+    }
+    url = url + sep + new_param;
     sep = '&';
   }
   return url;