improvements in :echo object, :echo getBrowser() should work now reliable

content/util.js

@@ -41,6 +41,9 @@ vimperator.util = {
     {
         var type = typeof(arg);
 
+        // some objects like window.JSON or getBrowsers()._browsers need the try/catch
+        try
+        {
             if (type == "number")
             {
                 return "<span style=\"color: red;\">" + arg + "</span>";
@@ -73,8 +76,13 @@ vimperator.util = {
                 else
                     return "undefined";
             }
+        }
+        catch (e)
+        {
+            return "&lt;unknown&gt;";
+        }
 
-        return arg;
+        return "&lt;unknown type&gt;";
     }
 }
 

content/vimperator.js

@@ -360,19 +360,18 @@ const vimperator = (function() //{{{
             else
                 string += obj + "::\n";
 
+            try // window.content often does not want to be queried with "var i in object"
+            {
                 for (var i in object)
                 {
                     var value;
                     try
                     {
-                    if (i == "JSON") // without this ugly hack, ":echo window" does not work
-                        value = "[object JSON]";
-                    else
                         value = object[i];
                     }
                     catch (e)
                     {
-                    value = "";
+                        value = "<no value>";
                     }
 
                     if (color)
@@ -383,6 +382,9 @@ const vimperator = (function() //{{{
                     else
                         string += i + ": " + value + "\n";
                 }
+            }
+            catch (e) { };
+
             return string;
         },