add yankencodedurl option

Author: Mine02C4

common/content/browser.js

@@ -96,13 +96,21 @@ const Browser = Module("browser", {
         options.add(["urlseparator"],
             "Set the separator regex used to separate multiple URL args",
             "string", ",\\s");
+
+        options.add(["yankencodedurl"],
+            "Set the yank mode copying encoded URL",
+            "boolean", false);
     },
 
     mappings: function () {
         mappings.add([modes.NORMAL],
             ["y"], "Yank current location to the clipboard",
             function () {
-                var url = services.get("textToSubURI").unEscapeURIForUI(buffer.charset, buffer.URL);
+                var url = buffer.URL;
+                if (options.get("yankencodedurl").value)
+                    url = services.get("io").newURI(url, buffer.charser, null).asciiSpec;
+                else
+                    url = services.get("textToSubURI").unEscapeURIForUI(buffer.charset, url).replace(/ /g, "%20");
                 util.copyToClipboard(url, true);
             });
 

common/content/services.js

@@ -70,6 +70,10 @@ const Services = Module("services", {
                 class_: "@mozilla.org/intl/texttosuburi;1",
                 iface: Ci.nsITextToSubURI
             },
+            "io": {
+                class_: "@mozilla.org/network/io-service;1",
+                iface: Ci.nsIIOService
+            },
         };
 
         this.addClass("file",       "@mozilla.org/file/local;1",                 Ci.nsILocalFile);

common/locale/en-US/options.xml

@@ -1395,6 +1395,16 @@
     </description>
 </item>
 
+<item>
+    <tags>'yankencodedurl'</tags>
+    <spec>'yankencodedurl'</spec>
+    <type>boolean</type>
+    <default>false</default>
+    <description>
+        <p>Set the yank mode for copying URLs. By default, only whitespace charaters are replaced by %20 to make pasting in other apps easy. If yankencodedurl is <str>true</str>, any non-ASCII character is URL encoded as well.</p>
+    </description>
+</item>
+
 <item>
     <tags> 'ani' 'animations'</tags>
     <spec> 'animations' 'ani'</spec>

vimperator/NEWS

@@ -1,3 +1,8 @@
+XXXX-XX-XX
+    * When you copy the URL with y, copy it as-is with only space encoded as %20
+    * Add 'yankencodedurl' option, so you can get the old behavior to encode any
+      any non-ASCII character with the encoded character
+
 2013-11-22
     * version 3.8.1
     * Fix arrow keys in menus