_visiblePositionForPoint lacks a check for the case when renderer->layer()->hitTest() doesn't find any node. This causes a crash in "run-webkit-tests editing/input/text-input-controller.html" with a fix to bug 5230 being applied. Used to work fine before - apparently, some recent checkin has changed the behavior of hitTest().
Created attachment 4674 [details] naive fix I couldn't find the reason why the behavior has changed - but the check for nil seems necessary here nonetheless.
Comment on attachment 4674 [details] naive fix As long as it's correct for hitTest to not find a node in this case, r=me.
Landed