Don't blow up if the user hasn't yet clicked the content when moving around in caret mode.

common/content/editor.js

@@ -424,6 +424,11 @@ const Editor = Module("editor", {
                         count = 1;
 
                     let controller = buffer.selectionController;
+                    let sel = controller.getSelection(controller.SELECTION_NORMAL);
+                    if (!sel.rangeCount) // Hack.
+                        sel.addRange(RangeFind.endpoint(
+                            RangeFind.nodeRange(tabs.localStore.focusedFrame.document.documentElement),
+                            true));
                     while (count--)
                         controller[caretModeMethod](caretModeArg, false);
                 },

pentadactyl/TODO

@@ -25,8 +25,6 @@ BUGS:
 (recent Mercurial regressions):
 9 about:pentadactyl is currently about:undefined
 
-- visual caret mode is broken, requires a manual page focus first anyway or
-  else it chucks, I haven't investigated --djk
 - messages is still broken in several ways - needs testing.
   => :ls | :echomsg "Foobar" doesn't add "Foobar" to the already open MOW.
   (NOTE: an intentional design decision by MS - Boo!)