Import some fixes from Vimperator. Thanks anekos, teramoko, and stepnem.

common/content/commandline.js

@@ -1142,8 +1142,9 @@ const CommandLine = Module("commandline", {
             // Change the completion text.
             // The second line is a hack to deal with some substring
             // preview corner cases.
-            commandline.widgets.command.value = this.prefix + completion + this.suffix;
-            this.editor.selection.focusNode.textContent = commandline.widgets.command.value;
+            let value = this.prefix + completion + this.suffix;
+            commandline.widgets.command.value = value;
+            this.editor.selection.focusNode.textContent = value;
 
             // Reset the caret to one position after the completion.
             this.caret = this.prefix.length + completion.length;

common/content/commands.js

@@ -973,11 +973,11 @@ const Commands = Module("commands", {
             bookmark: "bookmark", buffer: "buffer", color: "colorScheme",
             command: "command", dialog: "dialog", dir: "directory",
             environment: "environment", event: "autocmdEvent", file: "file",
-            help: "help", highlight: "highlightGroup", javascript: "javascript",
-            macro: "macro", mapping: "userMapping", menu: "menuItem",
-            option: "option", preference: "preference", search: "search",
-            shellcmd: "shellCommand", sidebar: "sidebar", url: "url",
-            usercommand: "userCommand"
+            help: "help", highlight: "highlightGroup", history: "history",
+            javascript: "javascript", macro: "macro", mapping: "userMapping",
+            menu: "menuItem", option: "option", preference: "preference",
+            search: "search", shellcmd: "shellCommand", sidebar: "sidebar",
+            url: "url", usercommand: "userCommand"
         };
 
         // TODO: Vim allows commands to be defined without {rep} if there are {attr}s
@@ -1018,7 +1018,7 @@ const Commands = Module("commands", {
                             };
                         }
                         else
-                            completeFunc = completion[completeOptionMap[completeOpt]];
+                            completeFunc = completion.closure[completeOptionMap[completeOpt]];
                     }
 
                     let added = commands.addUserCommand([cmd],

common/content/editor.js

@@ -554,7 +554,7 @@ const Editor = Module("editor", {
     getAbbreviations: function (filter, lhs) {
         // ! -> list all, on c or i ! matches too
         let searchFilter = (filter == "!") ? "!ci" : filter + "!";
-        return list = [[mode, left, right] for ([left, [mode, right]] in this.abbrevs())
+        return [[mode, left, right] for ([left, [mode, right]] in this.abbrevs())
                 if (searchFilter.indexOf(mode) >= 0 && left.indexOf(lhs || "") == 0)];
     },
 
@@ -968,11 +968,8 @@ const Editor = Module("editor", {
                     editor.executeCommand("cmd_copy");
                     modes.set(modes.TEXTAREA);
                 }
-                else {
-                    let sel = window.content.document.getSelection();
-                    dactyl.assert(sel);
-                    dactyl.clipboardWrite(sel, true);
-                }
+                else
+                    dactyl.clipboardWrite(buffer.getCurrentWord(), true);
             });
 
         mappings.add([modes.VISUAL, modes.TEXTAREA],

common/content/events.js

@@ -194,7 +194,7 @@ const Events = Module("events", {
      * Replays a macro.
      *
      * @param {string} The name of the macro to replay.
-     * @return {boolean}
+     * @returns {boolean}
      */
     playMacro: function (macro) {
         let res = false;
@@ -411,7 +411,7 @@ const Events = Module("events", {
      * <S-@> where @ is a non-case-changable, non-space character.
      *
      * @param {string} keys The string to parse.
-     * @return {Array[Object]}
+     * @returns {Array[Object]}
      */
     fromString: function (input) {
         let out = [];

common/content/tabs.js

@@ -283,8 +283,8 @@ const Tabs = Module("tabs", {
     /**
      * Removes the specified <b>tab</b> from the tab list.
      *
-     * @param {Object} tab
-     * @param {number} count
+     * @param {Object} tab The tab to remove.
+     * @param {number} count How many tabs to remove.
      * @param {boolean} focusLeftTab Focus the tab to the left of the removed tab.
      * @param {number} quitOnLastTab Whether to quit if the tab being
      *     deleted is the only tab in the tab list:

common/modules/base.jsm

@@ -487,7 +487,7 @@ function memoize(obj, key, getter) {
  *     function foo(a, b, c) [a, b, c].join(" ");
  *     curry(foo)(1, 2, 3) -> "1 2 3";
  *     curry(foo)(4)(5, 6) -> "4 5 6";
- *     curry(foo)(4)(8)(9) -> "7 8 9";
+ *     curry(foo)(7)(8)(9) -> "7 8 9";
  *
  * @param {function} fn The function to curry.
  * @param {integer} length The number of arguments expected.
@@ -720,7 +720,7 @@ Class.prototype = {
  * @param {string} name The name of the instance.
  * @param {Object} prototype The instance prototype.
  * @param {Object} classProperties Properties to be applied to the class constructor.
- * @return {Class}
+ * @returns {Class}
  */
 function Module(name, prototype) {
     let init = callable(prototype) ? 4 : 3;
@@ -755,7 +755,7 @@ else
  * a fixed set of named members. Each argument should be the name of
  * a member in the resulting objects. These names will correspond to
  * the arguments passed to the resultant constructor. Instances of
- * the new struct may be treated vary much like arrays, and provide
+ * the new struct may be treated very much like arrays, and provide
  * many of the same methods.
  *
  *     const Point = Struct("x", "y", "z");

common/modules/highlight.jsm

@@ -302,10 +302,10 @@ const Highlights = Module("Highlight", {
                 serialize: function () [
                     {
                         command: this.name,
-                        arguments: [k],
-                        literalArg: v
+                        arguments: [v.class],
+                        literalArg: v.value
                     }
-                    for ([k, v] in Iterator(highlight))
+                    for (v in Iterator(highlight))
                     if (v.value != v.default)
                 ]
             });

common/modules/util.jsm

@@ -451,6 +451,16 @@ const Util = Module("Util", {
         return idx > -1 && idx + domain.length == host.length && (idx == 0 || host[idx-1] == ".");
     },
 
+    /**
+     * Returns true if the given DOM node is currently visible.
+     *
+     * @param {Node} node
+     */
+    isVisible: function (node) {
+        let style = util.computedStyle(node);
+        return style.visibility == "visible" && style.display != "none";
+    },
+
     /**
      * Returns an XPath union expression constructed from the specified node
      * tests. An expression is built with node tests for both the null and

pentadactyl/content/config.js

@@ -45,7 +45,7 @@ const Config = Module("config", ConfigBase, {
         checkupdates: ["Check for updates",
             function () { window.checkForUpdates(); }],
         cleardata: ["Clear private data",
-            function () { Cc[GLUE_CID].getService(Ci.nsIBrowserGlue).sanitize(window || null); }],
+            function () { Cc["@mozilla.org/browser/browserglue;1"].getService(Ci.nsIBrowserGlue).sanitize(window || null); }],
         cookies: ["List your cookies",
             function () { window.toOpenWindowByType("Browser:Cookies", "chrome://browser/content/preferences/cookies.xul", "chrome,dialog=no,resizable"); }],
         console: ["JavaScript console",