Fix javascript: URIs.

This is @mrmr9393's suggestion from #1636. It mimic's Chrome's behaviour
when a javascript: URI is enetered into the omnibox (or clicked).

Fixes #1611.

1 files changed, 12 insertions, 0 deletions

diff --git a/tests/unit_tests/utils_test.coffee b/tests/unit_tests/utils_test.coffee
index 88e9a15b..ac3e4d85 100644
--- a/tests/unit_tests/utils_test.coffee
+++ b/tests/unit_tests/utils_test.coffee
@@ -42,6 +42,7 @@ context "convertToUrl",
     assert.equal "http://127.0.0.1:8080", Utils.convertToUrl("127.0.0.1:8080")
     assert.equal "http://[::]:8080", Utils.convertToUrl("[::]:8080")
     assert.equal "view-source:    0.0.0.0", Utils.convertToUrl("view-source:    0.0.0.0")
+    assert.equal "javascript:alert('25 % 20 25 ');", Utils.convertToUrl "javascript:alert('25 % 20 25%20');"
 
   should "convert non-URL terms into search queries", ->
     assert.equal "http://www.google.com/search?q=google", Utils.convertToUrl("google")
@@ -62,6 +63,17 @@ context "hasChromePrefix",
     assert.isFalse Utils.hasChromePrefix "data"
     assert.isFalse Utils.hasChromePrefix "data :foobar"
 
+context "hasJavascriptPrefix",
+  should "detect javascript: URLs", ->
+    assert.isTrue Utils.hasJavascriptPrefix "javascript:foobar"
+    assert.isFalse Utils.hasJavascriptPrefix "http:foobar"
+
+context "decodeJavascriptURI",
+  should "decode javascript: URLs", ->
+    assert.equal "foobar", Utils.decodeJavascriptURI "foobar"
+    assert.equal " ", Utils.decodeJavascriptURI "%20"
+    assert.equal "25 % 20 25 ", Utils.decodeJavascriptURI "25 % 20 25%20"
+
 context "isUrl",
   should "identify URLs as URLs", ->
     assert.isTrue Utils.isUrl "http://www.example.com/blah"